Unicon / googleapps-grouper-provisioner

This project is an Internet2 Grouper connector that synchronizes Grouper groups and users to Google Apps.
Apache License 2.0
6 stars 5 forks source link

issue 14: ignore extra google group members #16

Closed jtgasper3 closed 10 years ago

macavilca commented 9 years ago

I have deprovisionUser set to false and added a user to a google group that was not in the corresponding grouper group then ran the fullsync. User did not get removed from google, user not added to grouper group (as expected) although there was an error in output from googleAppsFullSync script: "java.lang.NullPointerException at edu.internet2.middleware.changelogconsumer.googleapps.GoogleAppsFullSync.processExtraGroupMembers(GoogleAppsFullSync.java:289) ..." Is that what is being addressed in this issue?

jtgasper3 commented 9 years ago

@macavilca, A few things upfront:

  1. deprovisionUser has no affect at this point. What it will eventually do is check to see if the user is apart of any other Google Groups and if there are no additional memberships, then kill the Google account.
  2. The provisioner is runs in a single direction. Grouper->Google. Nothing updates Grouper; only changes are made to Google. I'll check to make see if I can make this more clear in the documentation.

So to the point at hand, the user could have been removed from Google when the full sync ran, but it depends upon the value of ignoreExtraGoogleMembers. If it is true, the user's group membership should have been removed.

I'm not sure about the exception. Can you please send more of the stack trace?