From 3998441bc1264b360849499223201cdf671b1001 Mon Sep 17 00:00:00 2001 From: Antoni Sawicki Date: Tue, 29 Nov 2022 03:03:49 -0800 Subject: refactor creation of new request --- handlers.go | 29 +++++++++++++++-------------- 1 file changed, 15 insertions(+), 14 deletions(-) diff --git a/handlers.go b/handlers.go index 4fedc58..a3a18c3 100644 --- a/handlers.go +++ b/handlers.go @@ -24,25 +24,26 @@ type wfmRequest struct { } func wfmMain(w http.ResponseWriter, r *http.Request) { - wfm := new(wfmRequest) r.ParseMultipartForm(10 << 20) - wfm.userName, wfm.rwAccess = auth(w, r) - if wfm.userName == "" { + uName, uAccess := auth(w, r) + if uName == "" { return } - go log.Printf("req from=%q user=%q uri=%q form=%v", r.RemoteAddr, wfm.userName, r.RequestURI, noText(r.Form)) + go log.Printf("req from=%q user=%q uri=%q form=%v", r.RemoteAddr, uName, r.RequestURI, noText(r.Form)) - // TODO(tenox): allow fs per user - wfm.fs = wfmFs - wfm.w = w - wfm.remAddr = r.RemoteAddr - wfm.eSort = r.FormValue("sort") - if strings.HasPrefix(r.UserAgent(), "Mozilla/5") { - wfm.modern = true + wfm := &wfmRequest{ + userName: uName, + rwAccess: uAccess, + remAddr: r.RemoteAddr, + w: w, + eSort: r.FormValue("sort"), + modern: strings.HasPrefix(r.UserAgent(), "Mozilla/5"), + fs: wfmFs, // TODO(tenox): per user FS/homedir + uFbn: filepath.Base(r.FormValue("file")), + uDir: filepath.Clean(r.FormValue("dir")), } - wfm.uFbn = filepath.Base(r.FormValue("file")) - wfm.uDir = filepath.Clean(r.FormValue("dir")) - // directory can come from form value or URI Path + + // directory can come either from form value or URI Path if wfm.uDir == "" || wfm.uDir == "." { // TODO(tenox): use url.Parse() instead u, _ := url.PathUnescape(r.URL.Path) -- cgit v1.2.3