web/admin: don't show backup error when server can't backup

Signed-off-by: Jens Langhammer <jens.langhammer@beryju.org>
This commit is contained in:
Jens Langhammer 2021-06-06 12:51:23 +02:00
parent 1f35f73c66
commit bedf7fbcaa
3 changed files with 14 additions and 6 deletions

View File

@ -365,8 +365,8 @@ msgid "Backup finished with errors."
msgstr "Backup finished with errors." msgstr "Backup finished with errors."
#: src/pages/admin-overview/cards/BackupStatusCard.ts #: src/pages/admin-overview/cards/BackupStatusCard.ts
msgid "Backup finished with warnings." msgid "Backup finished with warnings/backup not supported."
msgstr "Backup finished with warnings." msgstr "Backup finished with warnings/backup not supported."
#: src/pages/admin-overview/AdminOverviewPage.ts #: src/pages/admin-overview/AdminOverviewPage.ts
msgid "Backup status" msgid "Backup status"

View File

@ -361,7 +361,7 @@ msgid "Backup finished with errors."
msgstr "" msgstr ""
#: #:
msgid "Backup finished with warnings." msgid "Backup finished with warnings/backup not supported."
msgstr "" msgstr ""
#: #:

View File

@ -2,8 +2,9 @@ import { t } from "@lingui/macro";
import { customElement, html, TemplateResult } from "lit-element"; import { customElement, html, TemplateResult } from "lit-element";
import { AdminStatus, AdminStatusCard } from "./AdminStatusCard"; import { AdminStatus, AdminStatusCard } from "./AdminStatusCard";
import { AdminApi, StatusEnum } from "authentik-api"; import { AdminApi, StatusEnum } from "authentik-api";
import { DEFAULT_CONFIG } from "../../../api/Config"; import { config, DEFAULT_CONFIG } from "../../../api/Config";
import { convertToTitle } from "../../../utils"; import { convertToTitle } from "../../../utils";
import { CapabilitiesEnum } from "authentik-api/src";
@customElement("ak-admin-status-card-backup") @customElement("ak-admin-status-card-backup")
export class BackupStatusCard extends AdminStatusCard<StatusEnum> { export class BackupStatusCard extends AdminStatusCard<StatusEnum> {
@ -14,7 +15,14 @@ export class BackupStatusCard extends AdminStatusCard<StatusEnum> {
}).then((value) => { }).then((value) => {
return value.status; return value.status;
}).catch(() => { }).catch(() => {
return StatusEnum.Error; // On error (probably 404), check the config and see if the server
// can even backup
return config().then(c => {
if (c.capabilities.includes(CapabilitiesEnum.Backup)) {
return StatusEnum.Error;
}
return StatusEnum.Warning;
});
}); });
} }
@ -27,7 +35,7 @@ export class BackupStatusCard extends AdminStatusCard<StatusEnum> {
case StatusEnum.Warning: case StatusEnum.Warning:
return Promise.resolve<AdminStatus>({ return Promise.resolve<AdminStatus>({
icon: "fa fa-exclamation-triangle pf-m-warning", icon: "fa fa-exclamation-triangle pf-m-warning",
message: t`Backup finished with warnings.`, message: t`Backup finished with warnings/backup not supported.`,
}); });
case StatusEnum.Error: case StatusEnum.Error:
return Promise.resolve<AdminStatus>({ return Promise.resolve<AdminStatus>({