Skip Menu |

This queue is for tickets about the REST-Google-Apps-Provisioning CPAN distribution.

Report information
The Basics
Id: 70863
Status: resolved
Priority: 0/
Queue: REST-Google-Apps-Provisioning

People
Owner: sschneid [...] gmail.com
Requestors: EVO [...] cpan.org
Cc: simonm [...] cavalletto.org
AdminCc:

Bug Information
Severity: Normal
Broken in: (no value)
Fixed in: (no value)



CC: simonm [...] cavalletto.org
Hello, and thank you for this very useful module! I noticed one unnecessary limitation in the module: the code treats the member parameter to addGroupMember and deleteGroupMember as always a bare username -- but in fact, Google allows you to enable adding non-domain members to a group. I monkey-patched around this and confirmed it works, using code like this: $arg->{'member'} .= '@' . $self->{'domain'} unless ( $arg->{'member'} =~ /[@]/ ); $body .= qq( <apps:property name="memberId" value="$arg->{'member'}" />\n); I hope that's useful in case you want to wish to incorporate a similar flexibility in your distribution. Thanks again, -- -Simon
Subject: Support external group members
Hello again, I found another method which needed to be patched in order to add support for external (non- domain) group members. In getGroupMembers, don't discard external domain names: - $member =~ s/^(.*)\@.*$/$1/g; + $member =~ s/^(.*)\@\Q$self->{'domain'}\E$/$1/g; Thanks again! -Simon -- -Simon