From a5d7a8151917a5073595d98e84f99d543c45f079 Mon Sep 17 00:00:00 2001
From: advplyr <advplyr@protonmail.com>
Date: Sat, 30 Mar 2024 14:17:34 -0500
Subject: [PATCH] Clean up formatting of advanced group/permission claims on
 authentication page

---
 client/pages/config/authentication.vue | 15 ++++++++-------
 server/Auth.js                         |  2 +-
 2 files changed, 9 insertions(+), 8 deletions(-)

diff --git a/client/pages/config/authentication.vue b/client/pages/config/authentication.vue
index cecccee4..9f2e71ec 100644
--- a/client/pages/config/authentication.vue
+++ b/client/pages/config/authentication.vue
@@ -82,19 +82,20 @@
               <p class="pl-4 text-sm text-gray-300">{{ $strings.LabelAutoRegisterDescription }}</p>
             </div>
 
-            <div class="flex items-center pt-6 pb-1 px-1 w-full">{{ $strings.LabelOpenIDClaims }}</div>
-            <div class="flex items-center mb-2">
-              <div class="w-96">
+            <p class="pt-6 mb-4 px-1">{{ $strings.LabelOpenIDClaims }}</p>
+
+            <div class="flex mb-4">
+              <div class="w-44 min-w-44">
                 <ui-text-input-with-label ref="openidGroupClaim" v-model="newAuthSettings.authOpenIDGroupClaim" :disabled="savingSettings" :placeholder="'groups'" :label="'Group Claim'" />
               </div>
-              <p class="pl-4 text-sm text-gray-300 mt-5" v-html="$strings.LabelOpenIDGroupClaimDescription"></p>
+              <p class="pl-4 text-sm text-gray-300" v-html="$strings.LabelOpenIDGroupClaimDescription"></p>
             </div>
 
-            <div class="flex mb-2">
-              <div class="w-96 pt-6">
+            <div class="flex mb-4">
+              <div class="w-44 min-w-44">
                 <ui-text-input-with-label ref="openidAdvancedPermsClaim" v-model="newAuthSettings.authOpenIDAdvancedPermsClaim" :disabled="savingSettings" :placeholder="'abspermissions'" :label="'Advanced Permission Claim'" />
               </div>
-              <div class="pl-4 text-sm text-gray-300 mt-5 flex-column">
+              <div class="pl-4 text-sm text-gray-300 flex-column">
                 <p v-html="$strings.LabelOpenIDAdvancedPermsClaimDescription"></p>
                 <pre class="text-pre-wrap mt-2"
                   >{{ newAuthSettings.authOpenIDSamplePermissions }}
diff --git a/server/Auth.js b/server/Auth.js
index b52ee727..8ba87509 100644
--- a/server/Auth.js
+++ b/server/Auth.js
@@ -287,7 +287,7 @@ class Auth {
       throw new Error(`Advanced permissions claim ${absPermissionsClaim} not found in userinfo`)
 
     if (user.updatePermissionsFromExternalJSON(absPermissions)) {
-      Logger.debug(`[Auth] openid callback: Updating advanced perms for user "${user.username}" using "${JSON.stringify(absPermissions)}"`)
+      Logger.info(`[Auth] openid callback: Updating advanced perms for user "${user.username}" using "${JSON.stringify(absPermissions)}"`)
       await Database.userModel.updateFromOld(user)
     }
   }