mirror of
https://github.com/filebrowser/filebrowser.git
synced 2024-06-07 23:00:43 +00:00
Refactor the code
This commit is contained in:
parent
c3a4e33245
commit
030f6607f0
@ -46,14 +46,20 @@ func addConfigFlags(flags *pflag.FlagSet) {
|
|||||||
|
|
||||||
func getAuthentication(flags *pflag.FlagSet, defaults ...interface{}) (settings.AuthMethod, auth.Auther) {
|
func getAuthentication(flags *pflag.FlagSet, defaults ...interface{}) (settings.AuthMethod, auth.Auther) {
|
||||||
method := settings.AuthMethod(mustGetString(flags, "auth.method"))
|
method := settings.AuthMethod(mustGetString(flags, "auth.method"))
|
||||||
|
|
||||||
var defaultAuther map[string]interface{}
|
var defaultAuther map[string]interface{}
|
||||||
for _, arg := range defaults {
|
if len(defaults) > 0 {
|
||||||
switch def := arg.(type) {
|
if hasAuth := defaults[0]; hasAuth != true {
|
||||||
case *settings.Settings:
|
for _, arg := range defaults {
|
||||||
method = settings.AuthMethod(def.AuthMethod)
|
switch def := arg.(type) {
|
||||||
case auth.Auther:
|
case *settings.Settings:
|
||||||
ms, _ := json.Marshal(def)
|
method = settings.AuthMethod(def.AuthMethod)
|
||||||
json.Unmarshal(ms, &defaultAuther)
|
case auth.Auther:
|
||||||
|
ms, err := json.Marshal(def)
|
||||||
|
checkErr(err)
|
||||||
|
json.Unmarshal(ms, &defaultAuther)
|
||||||
|
}
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -3,7 +3,6 @@ package cmd
|
|||||||
import (
|
import (
|
||||||
"strings"
|
"strings"
|
||||||
|
|
||||||
"github.com/filebrowser/filebrowser/v2/auth"
|
|
||||||
"github.com/spf13/cobra"
|
"github.com/spf13/cobra"
|
||||||
"github.com/spf13/pflag"
|
"github.com/spf13/pflag"
|
||||||
)
|
)
|
||||||
@ -63,20 +62,15 @@ you want to change. Other options will remain unchanged.`,
|
|||||||
|
|
||||||
getUserDefaults(flags, &set.Defaults, false)
|
getUserDefaults(flags, &set.Defaults, false)
|
||||||
|
|
||||||
var auther auth.Auther
|
// read the defaults
|
||||||
if hasAuth {
|
auther, err := d.store.Auth.Get(set.AuthMethod)
|
||||||
set.AuthMethod, auther = getAuthentication(flags)
|
checkErr(err)
|
||||||
err = d.store.Auth.Save(auther)
|
|
||||||
checkErr(err)
|
|
||||||
} else {
|
|
||||||
auther, err = d.store.Auth.Get(set.AuthMethod)
|
|
||||||
checkErr(err)
|
|
||||||
// check if there are new flags for existing auth method
|
|
||||||
set.AuthMethod, auther = getAuthentication(flags, set, auther)
|
|
||||||
err = d.store.Auth.Save(auther)
|
|
||||||
checkErr(err)
|
|
||||||
}
|
|
||||||
|
|
||||||
|
// check if there are new flags for existing auth method
|
||||||
|
set.AuthMethod, auther = getAuthentication(flags, hasAuth, set, auther)
|
||||||
|
|
||||||
|
err = d.store.Auth.Save(auther)
|
||||||
|
checkErr(err)
|
||||||
err = d.store.Settings.Save(set)
|
err = d.store.Settings.Save(set)
|
||||||
checkErr(err)
|
checkErr(err)
|
||||||
err = d.store.Settings.SaveServer(ser)
|
err = d.store.Settings.SaveServer(ser)
|
||||||
|
Loading…
Reference in New Issue
Block a user