Grouper Developers Forum

[Chris Hyzer <>]

Hey,

Thanks a lot for making a patch and sending it back.  I wonder if we should 
restrict event types at all... hmmm (e.g. stem add, privilege revoke, etc).  
Anyways, I didn't want to add a new Jar, so I changed your immutable set to 
this:

Collections.unmodifiableSet(GrouperClientUtils.toSet("MEMBERSHIP_ADD"...

Take a look at this once it shows up there (I think there is some lag), and 
diff the versions and see if you think it will work.

http://anonsvn.internet2.edu/cgi-bin/viewvc.cgi/i2mi/trunk/grouper-misc/grouperClient/src/extMore/edu/internet2/middleware/grouperClientExt/xmpp/GrouperClientXmppMain.java?view=log

Thanks,
Chris


-----Original Message-----
From: 

 
[mailto:]
 On Behalf Of Erik Froese
Sent: Friday, March 11, 2011 10:30 AM
To: 

Subject: [grouper-dev] Supporting GROUP_ADD and GROUP_DELETE in the 
grouperClient XMPP

Hey everyone,

I've been working with the XMPP code in the grouperClient project and
I came up with a patch that helps me out a bit. I've put the
grouperClient project in my github account so I can work with more
easily.

https://github.com/efroese/grouper-client/commit/9e452d053d170860540074a7047193112add4b7b

From what I can see the client code was written with the
MEMBERSHIP_ADD and MEMBERSHIP_DELETE events in mind. For each event
received the GrouperClientXmppMain tries to fetch the membership for
the group listed in the event. Unfortunately for groups that are in
the process of being created or have just been deleted this call
throws an error.

I've added a list of currently supported events SUPPORTED_EVENT_TYPES
and a check in the incrementalRefreshGroup method.

WDYT?

Erik3