Tokens take precedence over default user setting

src/handlers/auth_handler.cr

@@ -74,10 +74,17 @@ class AuthHandler < Kemal::Handler
     end
 
     if request_path_startswith env, ["/admin", "/api/admin", "/download"]
-      unless validate_token_admin(env) ||
+      # The token (if exists) takes precedence over the default user option.
-             Storage.default.username_is_admin Config.current.default_username
+      #   this is why we check the default username first before checking the
-        env.response.status_code = 403
+      #   token.
+      should_reject = true
+      if Storage.default.username_is_admin Config.current.default_username
+        should_reject = false
       end
+      if env.session.string? "token"
+        should_reject = !validate_token_admin(env)
+      end
+      env.response.status_code = 403 if should_reject
     end
 
     call_next env

src/util/web.cr

@@ -4,13 +4,16 @@ macro layout(name)
   base_url = Config.current.base_url
   begin
     is_admin = false
-    if token = env.session.string? "token"
+    # The token (if exists) takes precedence over the default user option.
-      is_admin = @context.storage.verify_admin token
+    #   this is why we check the default username first before checking the
-    end
+    #   token.
     if Config.current.disable_login
       is_admin = @context.storage.
         username_is_admin Config.current.default_username
     end
+    if token = env.session.string? "token"
+      is_admin = @context.storage.verify_admin token
+    end
     page = {{name}}
     render "src/views/#{{{name}}}.html.ecr", "src/views/layout.html.ecr"
   rescue e