astelm/minio
1
0
Fork 0
mirror of https://github.com/pgsty/minio.git synced 2026-07-24 06:26:17 +03:00
Code Issues Packages Projects Releases Wiki Activity

sr: use site replicator creds to verify temp user claims (#19224)

Browse Source 
This PR continues #19209 which did not handle claims verification of
temporary users created by root in site replication scenario.

Fixes: #19217
This commit is contained in:
Poorna
2024-03-07 14:30:00 -08:00
committed by GitHub
parent 233cc3905a
commit 934f6cabf6
1 changed files with 7 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files
cmd/auth-handler.go
+7 -7
View File
@@ -298,15 +298,15 @@ func checkClaimsFromToken(r *http.Request, cred auth.Credentials) (map[string]in
if cred.IsTemp() && cred.IsExpired() { if cred.IsTemp() && cred.IsExpired() {
return nil, toAPIErrorCode(r.Context(), errInvalidAccessKeyID) return nil, toAPIErrorCode(r.Context(), errInvalidAccessKeyID)
} }
secret := globalActiveCred.SecretKey secret := globalActiveCred.SecretKey
var err error
if globalSiteReplicationSys.isEnabled() && cred.AccessKey != siteReplicatorSvcAcc { if globalSiteReplicationSys.isEnabled() && cred.AccessKey != siteReplicatorSvcAcc {
if cred.ParentUser != globalActiveCred.AccessKey { nsecret, err := getTokenSigningKey()
secret, err = getTokenSigningKey() if err != nil {
if err != nil { return nil, toAPIErrorCode(r.Context(), err)
return nil, toAPIErrorCode(r.Context(), err) }
} // sign root's temporary accounts also with site replicator creds
if cred.ParentUser != globalActiveCred.AccessKey || cred.IsTemp() {
secret = nsecret
} }
} }
if cred.IsServiceAccount() { if cred.IsServiceAccount() {