Skip to content

Commit 5785207

Browse files
authored
Merge pull request #476 from devforth/feature/AdminForth/1193/i-think-we-had-some-feature-wh
fix: separation of private IP and default user
2 parents c104482 + 349a76c commit 5785207

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

adminforth/modules/restApi.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -411,7 +411,8 @@ export default class AdminForthRestAPI implements IAdminForthRestAPI {
411411
globalInjections: this.adminforth.config.customization.globalInjections,
412412
userFullnameField: this.adminforth.config.auth.userFullNameField,
413413
settingPages: settingPages,
414-
hasDefaultUserOnPublicNetwork: defaultUserExists && !isPrivateIP
414+
defaultUserExists: defaultUserExists,
415+
isPrivateIP: isPrivateIP,
415416
}
416417

417418
// translate menu labels

adminforth/spa/src/components/Sidebar.vue

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@
5151
</div>
5252
</div>
5353

54-
<div v-if="coreStore.config.hasDefaultUserOnPublicNetwork" class="p-4 mb-4 text-white rounded-lg bg-red-700/80 fill-white text-sm">
54+
<div v-if="coreStore.config.defaultUserExists && !coreStore.config.isPrivateIP" class="p-4 mb-4 text-white rounded-lg bg-red-700/80 fill-white text-sm">
5555
<IconExclamationCircleOutline class="inline-block align-text-bottom mr-0,5 w-5 h-5" />
5656
Default user <strong>"adminforth"</strong> detected. Delete it and create your own account.
5757
</div>

0 commit comments

Comments
 (0)