Skip to content

Commit 5bbefff

Browse files
authored
Admin are unable to add access keys hestiacp#4799 (hestiacp#4810)
1 parent 820eabc commit 5bbefff

File tree

4 files changed

+5
-8
lines changed

4 files changed

+5
-8
lines changed

web/add/access-key/index.php

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -11,10 +11,7 @@
1111
!empty($_SESSION["API_SYSTEM"]) && is_numeric($_SESSION["API_SYSTEM"])
1212
? $_SESSION["API_SYSTEM"]
1313
: 0;
14-
if (
15-
($user_plain == $_SESSION["ROOT_USER"] && $api_status < 1) ||
16-
($_SESSION["ROOT_USER"] != "admin" && $api_status < 2)
17-
) {
14+
if ($api_status < 1 || ($user_plain != $_SESSION["ROOT_USER"] && $api_status < 2)) {
1815
header("Location: /edit/user/");
1916
exit();
2017
}
@@ -23,7 +20,7 @@
2320
exec(HESTIA_CMD . "v-list-apis json", $output, $return_var);
2421
$apis = json_decode(implode("", $output), true);
2522
$apis = array_filter($apis, function ($api) use ($user_plain) {
26-
return $user_plain == "admin" || $api["ROLE"] == "user";
23+
return $user_plain == $_SESSION["ROOT_USER"] || $api["ROLE"] == "user";
2724
});
2825
ksort($apis);
2926
unset($output);

web/list/access-key/index.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616
!empty($_SESSION["API_SYSTEM"]) && is_numeric($_SESSION["API_SYSTEM"])
1717
? $_SESSION["API_SYSTEM"]
1818
: 0;
19-
if (($user_plain == "admin" && $api_status < 1) || ($user_plain != "admin" && $api_status < 2)) {
19+
if ($api_status < 1 || ($user_plain != $_SESSION["ROOT_USER"] && $api_status < 2)) {
2020
header("Location: /edit/user/");
2121
exit();
2222
}

web/templates/pages/edit_user.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@
2626
<?php } ?>
2727
<?php
2828
$api_status = (!empty($_SESSION['API_SYSTEM']) && is_numeric($_SESSION['API_SYSTEM'])) ? $_SESSION['API_SYSTEM'] : 0;
29-
if (($user_plain == 'admin' && $api_status > 0) || ($user_plain != 'admin' && $api_status > 1)) { ?>
29+
if (($user_plain == $_SESSION['ROOT_USER'] && $api_status > 0) || ($user_plain != $_SESSION['ROOT_USER'] && $api_status > 1)) { ?>
3030
<a href="<?= $keys_url; ?>" class="button button-secondary js-button-create" title="<?= _("Access Keys") ?>">
3131
<i class="fas fa-key icon-purple"></i><?= _("Access Keys") ?>
3232
</a>

web/templates/pages/list_key.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
<div class="toolbar">
33
<div class="toolbar-inner">
44
<div class="toolbar-buttons">
5-
<?php if ($_SESSION["userContext"] === "admin" && $_SESSION['look'] !== '' && $_GET["user"] !== "admin") { ?>
5+
<?php if ($_SESSION["userContext"] === "admin" && $_SESSION['look'] !== '' && $_GET["user"] !== $_SESSION['ROOT_USER']) { ?>
66
<a href="/edit/user/?user=<?= htmlentities($_SESSION["look"]) ?>&token=<?= $_SESSION["token"] ?>" class="button button-secondary button-back js-button-back">
77
<i class="fas fa-arrow-left icon-blue"></i><?= _("Back") ?>
88
</a>

0 commit comments

Comments
 (0)