Bug Fix : Release v2.3.3 (#270)

* Additional logging
* Refining group member handling
* Check USER is ACTIVE
* Bugfix: Do not stop syncing if user was deleted and therefore no longer a member of the group.
* sequence of operations
* #268
* Added additional checks for an empty set of groups returned from an  unreliable SDK.
* Added checks for '/' so don't strip the path.

---------

Co-authored-by: Janne Heinonen <[email protected]>

Author: ChrisPates

internal/google/client.go

@@ -186,6 +186,10 @@ func (c *client) GetGroups(query string) ([]*admin.Group, error) {
 		}); err != nil {
 			return nil, err
 		}
+		// Check we've got some users otherwise something is wrong.
+		if len(g) == 0 {
+			return nil, errors.New("google api return 0 groups?")
+		}
 		return g, err
 	}
 
@@ -204,7 +208,7 @@ func (c *client) GetGroups(query string) ([]*admin.Group, error) {
 
 	// Check we've got some users otherwise something is wrong.
 	if len(g) == 0 {
-		return g, errors.New("google api return 0 groups?")
+		return nil, errors.New("google api return 0 groups?")
 	}
 	return g, err
 }