Merge pull request #357 from wrapp/skip-group-lookup-404

Skip 404 errors when looking up Google groups
This commit is contained in:
Jehiah Czebotar 2017-03-28 21:38:55 -04:00 committed by GitHub
commit cd0d13e3fb

View File

@ -17,6 +17,7 @@ import (
"golang.org/x/oauth2" "golang.org/x/oauth2"
"golang.org/x/oauth2/google" "golang.org/x/oauth2/google"
"google.golang.org/api/admin/directory/v1" "google.golang.org/api/admin/directory/v1"
"google.golang.org/api/googleapi"
) )
type GoogleProvider struct { type GoogleProvider struct {
@ -197,8 +198,12 @@ func userInGroup(service *admin.Service, groups []string, email string) bool {
for _, group := range groups { for _, group := range groups {
members, err := fetchGroupMembers(service, group) members, err := fetchGroupMembers(service, group)
if err != nil { if err != nil {
log.Printf("error fetching group members: %v", err) if err, ok := err.(*googleapi.Error); ok && err.Code == 404 {
return false log.Printf("error fetching members for group %s: group does not exist", group)
} else {
log.Printf("error fetching group members: %v", err)
return false
}
} }
for _, member := range members { for _, member := range members {