authorize: allow missing user for authorization (#3421)

This commit is contained in:
Caleb Doxsey 2022-06-14 05:44:34 -06:00 committed by GitHub
parent ebbb6a7ff2
commit 15e3b3a431
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -61,16 +61,12 @@ func (a *Authorize) Check(ctx context.Context, in *envoy_service_auth_v3.CheckRe
if sessionState != nil {
s, err = a.getDataBrokerSessionOrServiceAccount(ctx, sessionState.ID)
if err != nil {
log.Warn(ctx).Err(err).Msg("clearing session due to force sync failed")
log.Warn(ctx).Err(err).Msg("clearing session due to missing session or service account")
sessionState = nil
}
}
if s != nil {
u, err = a.getDataBrokerUser(ctx, s.GetUserId())
if err != nil {
log.Warn(ctx).Err(err).Msg("clearing session due to force sync failed")
sessionState = nil
}
u, _ = a.getDataBrokerUser(ctx, s.GetUserId()) // ignore any missing user error
}
req, err := a.getEvaluatorRequestFromCheckRequest(in, sessionState)