git.iko.soy/typeable/oauth2-proxy
on master help sign in
Merge pull request #357 from wrapp/skip-group-lookup-404

Skip 404 errors when looking up Google groups

commitcd0d13e3fbee0d48722069a435b0beb75047c159 authorJehiah Czebotar <jehiah@gmail.com> dateTue 28 Mar 2017, 21:38 (10 years ago) parent86d0832652f43e
1 file changed+7−2
providers/google.go+7−2
providers/google.go+7−2
@@ -17,6 +17,7 @@ import (
1717 "golang.org/x/oauth2"
1818 "golang.org/x/oauth2/google"
1919 "google.golang.org/api/admin/directory/v1"
20+ "google.golang.org/api/googleapi"
2021)
2122
2223type GoogleProvider struct {
@@ -197,8 +198,12 @@ func userInGroup(service *admin.Service, groups []string, email string) bool {
197198 for _, group := range groups {
198199 members, err := fetchGroupMembers(service, group)
199200 if err != nil {
200− log.Printf("error fetching group members: %v", err)
201− return false
201+ if err, ok := err.(*googleapi.Error); ok && err.Code == 404 {
202+ log.Printf("error fetching members for group %s: group does not exist", group)
203+ } else {
204+ log.Printf("error fetching group members: %v", err)
205+ return false
206+ }
202207 }
203208
204209 for _, member := range members {