@@ -296,27 +296,31 @@ func (p *OAuthProxy) GetRedirectURI(host string) string {
296296 return u.String()
297297}
298298
299−func (p *OAuthProxy) redeemCode(ctx context.Context, host, code string) (s *sessionsapi.SessionState, err error) {
299+func (p *OAuthProxy) redeemCode(ctx context.Context, host, code string) (*sessionsapi.SessionState, error) {
300300 if code == "" {
301301 return nil, errors.New("missing code")
302302 }
303303 redirectURI := p.GetRedirectURI(host)
304− s, err = p.provider.Redeem(ctx, redirectURI, code)
304+ s, err := p.provider.Redeem(ctx, redirectURI, code)
305305 if err != nil {
306− return
306+ return nil, err
307307 }
308308
309309 if s.Email == "" {
310310 s.Email, err = p.provider.GetEmailAddress(ctx, s)
311+ if err != nil && err.Error() != "not implemented" {
312+ return nil, err
313+ }
311314 }
312315
313316 if s.User == "" {
314317 s.User, err = p.provider.GetUserName(ctx, s)
315− if err != nil && err.Error() == "not implemented" {
316− err = nil
318+ if err != nil && err.Error() != "not implemented" {
319+ return nil, err
317320 }
318321 }
319− return
322+
323+ return s, nil
320324}
321325
322326// MakeCSRFCookie creates a cookie for CSRF