diff --git a/CHANGELOG.md b/CHANGELOG.md index c383f8d2..fd46f7c1 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -12,6 +12,7 @@ ## Changes since v5.1.0 +- [#474](https://github.com/oauth2-proxy/oauth2-proxy/pull/474) Always log hasMember request error object (@jbielick) - [#468](https://github.com/oauth2-proxy/oauth2-proxy/pull/468) Implement graceful shutdown and propagate request context (@johejo) - [#464](https://github.com/oauth2-proxy/oauth2-proxy/pull/464) Migrate to oauth2-proxy/oauth2-proxy (@JoelSpeed) - Project renamed from `pusher/oauth2_proxy` to `oauth2-proxy` diff --git a/providers/google.go b/providers/google.go index ec16ded5..804ea345 100644 --- a/providers/google.go +++ b/providers/google.go @@ -198,11 +198,11 @@ func userInGroup(service *admin.Service, groups []string, email string) bool { req := service.Members.HasMember(group, email) r, err := req.Do() if err != nil { - err, ok := err.(*googleapi.Error) + gerr, ok := err.(*googleapi.Error) switch { - case ok && err.Code == 404: + case ok && gerr.Code == 404: logger.Printf("error checking membership in group %s: group does not exist", group) - case ok && err.Code == 400: + case ok && gerr.Code == 400: // It is possible for Members.HasMember to return false even if the email is a group member. // One case that can cause this is if the user email is from a different domain than the group, // e.g. "member@otherdomain.com" in the group "group@mydomain.com" will result in a 400 error