refactor: rename to hide login button

cmd/config.go

@@ -31,7 +31,7 @@ func addConfigFlags(flags *pflag.FlagSet) {
 	addServerFlags(flags)
 	addUserFlags(flags)
 	flags.BoolP("signup", "s", false, "allow users to signup")
-	flags.Bool("public-login", true, "allow users to login from the public page")
+	flags.Bool("hide-login-button", false, "hide login button from public pages")
 	flags.Bool("create-user-dir", false, "generate user's home directory automatically")
 	flags.Uint("minimum-password-length", settings.DefaultMinimumPasswordLength, "minimum password length for new users")
 	flags.String("shell", "", "shell command to which other commands should be appended")
@@ -193,10 +193,10 @@ func printSettings(ser *settings.Server, set *settings.Settings, auther auth.Aut
 	w := tabwriter.NewWriter(os.Stdout, 0, 0, 2, ' ', 0)
 
 	fmt.Fprintf(w, "Sign up:\t%t\n", set.Signup)
-	fmt.Fprintf(w, "Public login:\t%t\n", set.PublicLogin)
+	fmt.Fprintf(w, "Hide Login Button:\t%t\n", set.HideLoginButton)
 	fmt.Fprintf(w, "Create User Dir:\t%t\n", set.CreateUserDir)
 	fmt.Fprintf(w, "Minimum Password Length:\t%d\n", set.MinimumPasswordLength)
-	fmt.Fprintf(w, "Auth method:\t%s\n", set.AuthMethod)
+	fmt.Fprintf(w, "Auth Method:\t%s\n", set.AuthMethod)
 	fmt.Fprintf(w, "Shell:\t%s\t\n", strings.Join(set.Shell, " "))
 	fmt.Fprintln(w, "\nBranding:")
 	fmt.Fprintf(w, "\tName:\t%s\n", set.Branding.Name)

cmd/config_init.go

@@ -41,7 +41,7 @@ override the options.`,
 			return err
 		}
 
-		publicLogin, err := getBool(flags, "public-login")
+		hideLoginButton, err := getBool(flags, "hide-login-button")
 		if err != nil {
 			return err
 		}
@@ -89,7 +89,7 @@ override the options.`,
 		s := &settings.Settings{
 			Key:                   key,
 			Signup:                signup,
-			PublicLogin:           publicLogin,
+			HideLoginButton:       hideLoginButton,
 			CreateUserDir:         createUserDir,
 			MinimumPasswordLength: minLength,
 			Shell:                 convertCmdStrToCmdArray(shell),

cmd/config_set.go

@@ -50,8 +50,8 @@ you want to change. Other options will remain unchanged.`,
 				ser.Port, err = getString(flags, flag.Name)
 			case "log":
 				ser.Log, err = getString(flags, flag.Name)
-			case "publiclogin":
+			case "hide-login-button":
-				set.PublicLogin, err = getBool(flags, flag.Name)
+				set.HideLoginButton, err = getBool(flags, flag.Name)
 			case "signup":
 				set.Signup, err = getBool(flags, flag.Name)
 			case "auth.method":

cmd/root.go

@@ -420,7 +420,7 @@ func quickSetup(flags *pflag.FlagSet, d pythonData) error {
 	set := &settings.Settings{
 		Key:                   generateKey(),
 		Signup:                false,
-		PublicLogin:           true,
+		HideLoginButton:       true,
 		CreateUserDir:         false,
 		MinimumPasswordLength: settings.DefaultMinimumPasswordLength,
 		UserHomeBasePath:      settings.DefaultUsersHomeBasePath,

frontend/src/components/Sidebar.vue

@@ -63,7 +63,7 @@
     </template>
     <template v-else>
       <router-link
-        v-if="publiclogin"
+        v-if="!hideLoginButton"
         class="action"
         to="/login"
         :aria-label="$t('sidebar.login')"
@@ -125,7 +125,7 @@ import * as auth from "@/utils/auth";
 import {
   version,
   signup,
-  publiclogin,
+  hideLoginButton,
   disableExternal,
   disableUsedPercentage,
   noAuth,
@@ -155,7 +155,7 @@ export default {
       return this.currentPromptName === "sidebar";
     },
     signup: () => signup,
-    publiclogin: () => publiclogin,
+    hideLoginButton: () => hideLoginButton,
     version: () => version,
     disableExternal: () => disableExternal,
     disableUsedPercentage: () => disableUsedPercentage,

frontend/src/utils/constants.ts

@@ -18,7 +18,7 @@ const enableExec: boolean = window.FileBrowser.EnableExec;
 const tusSettings = window.FileBrowser.TusSettings;
 const origin = window.location.origin;
 const tusEndpoint = `/api/tus`;
-const publiclogin = window.FileBrowser.PublicLogin;
+const hideLoginButton = window.FileBrowser.HideLoginButton;
 
 export {
   name,
@@ -40,5 +40,5 @@ export {
   tusSettings,
   origin,
   tusEndpoint,
-  publiclogin,
+  hideLoginButton,
 };

http/settings.go

@@ -10,7 +10,7 @@ import (
 
 type settingsData struct {
 	Signup                bool                  `json:"signup"`
-	PublicLogin           bool                  `json:"publiclogin"`
+	HideLoginButton       bool                  `json:"hideLoginButton"`
 	CreateUserDir         bool                  `json:"createUserDir"`
 	MinimumPasswordLength uint                  `json:"minimumPasswordLength"`
 	UserHomeBasePath      string                `json:"userHomeBasePath"`
@@ -25,7 +25,7 @@ type settingsData struct {
 var settingsGetHandler = withAdmin(func(w http.ResponseWriter, r *http.Request, d *data) (int, error) {
 	data := &settingsData{
 		Signup:                d.settings.Signup,
-		PublicLogin:           d.settings.PublicLogin,
+		HideLoginButton:       d.settings.HideLoginButton,
 		CreateUserDir:         d.settings.CreateUserDir,
 		MinimumPasswordLength: d.settings.MinimumPasswordLength,
 		UserHomeBasePath:      d.settings.UserHomeBasePath,

http/static.go

@@ -46,7 +46,7 @@ func handleWithStaticData(w http.ResponseWriter, _ *http.Request, d *data, fSys
 		"ResizePreview":         d.server.ResizePreview,
 		"EnableExec":            d.server.EnableExec,
 		"TusSettings":           d.settings.Tus,
-		"PublicLogin":           d.settings.PublicLogin,
+		"HideLoginButton":       d.settings.HideLoginButton,
 	}
 
 	if d.settings.Branding.Files != "" {

settings/settings.go

@@ -22,7 +22,7 @@ type AuthMethod string
 type Settings struct {
 	Key                   []byte              `json:"key"`
 	Signup                bool                `json:"signup"`
-	PublicLogin           bool                `json:"publiclogin"`
+	HideLoginButton       bool                `json:"hideLoginButton"`
 	CreateUserDir         bool                `json:"createUserDir"`
 	UserHomeBasePath      string              `json:"userHomeBasePath"`
 	Defaults              UserDefaults        `json:"defaults"`