Merge pull request #962 from openziti/fix.is.default.admin.permission.check

fixes admin permission not also checking default admin
This commit is contained in:
Andrew
2022-04-08 10:47:46 -04:00
committed by GitHub
+2 -2
View File
@@ -50,9 +50,9 @@ import (
"github.com/openziti/foundation/common/constants"
"github.com/openziti/foundation/identity/identity"
"github.com/openziti/foundation/metrics"
"github.com/openziti/storage/boltz"
"github.com/openziti/foundation/util/errorz"
"github.com/openziti/sdk-golang/ziti/config"
"github.com/openziti/storage/boltz"
cmap "github.com/orcaman/concurrent-map"
"github.com/xeipuuv/gojsonschema"
"io"
@@ -317,7 +317,7 @@ func (ae *AppEnv) FillRequestContext(rc *response.RequestContext) error {
rc.ActivePermissions = append(rc.ActivePermissions, permissions.AuthenticatedPermission)
}
if rc.Identity.IsAdmin {
if rc.Identity.IsAdmin || rc.Identity.IsDefaultAdmin {
rc.ActivePermissions = append(rc.ActivePermissions, permissions.AdminPermission)
}
}