Skip to content

Commit 818a7d5

Browse files
authored
Merge pull request Place1#121 from halkeye/report-why-oidc-failure
2 parents 216e77a + ca29c6e commit 818a7d5

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

pkg/authnz/authconfig/oidc.go

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -98,11 +98,15 @@ func (c *OIDCConfig) callbackHandler(runtime *authruntime.ProviderRuntime, oauth
9898
}
9999

100100
code := r.FormValue("code")
101-
token, _ := oauthConfig.Exchange(r.Context(), code)
101+
102+
token, err := oauthConfig.Exchange(r.Context(), code)
103+
if err != nil {
104+
panic(errors.Wrap(err, "Unable to exchange tokens"))
105+
}
106+
102107
info, err := provider.UserInfo(r.Context(), oauthConfig.TokenSource(r.Context(), token))
103108
if err != nil {
104-
http.Error(w, err.Error(), http.StatusBadRequest)
105-
return
109+
panic(errors.Wrap(err, "Unable to get user info"))
106110
}
107111

108112
if msg, valid := verifyEmailDomain(c.EmailDomains, info.Email); !valid {

0 commit comments

Comments
 (0)