Skip to content
Snippets Groups Projects
Commit 844f2c43 authored by Jeff Burke's avatar Jeff Burke
Browse files

Merge branch 's1666' of ssh://mach16/srv/cadc/git/wopencadc into s1666

Conflicts:
	projects/cadcAccessControl-Server/build.xml
	projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/GetGroupNamesAction.java
parents d8046963 ea1914e5
No related branches found
No related tags found
No related merge requests found
......@@ -141,11 +141,11 @@ public class GMSClient
try
{
URL testURL = new URL(baseURL);
if (!testURL.getProtocol().equals("https"))
{
throw new IllegalArgumentException(
"URL must have HTTPS protocol");
}
// if (!testURL.getProtocol().equals("https"))
// {
// throw new IllegalArgumentException(
// "URL must have HTTPS protocol");
// }
}
catch (MalformedURLException e)
{
......@@ -584,7 +584,7 @@ public class GMSClient
throws GroupNotFoundException, UserNotFoundException, AccessControlException, IOException
{
String userIDType = AuthenticationUtil.getPrincipalType(userID);
String encodedUserID = URLEncoder.encode(userID.toString(), "UTF-8");
String encodedUserID = URLEncoder.encode(userID.getName(), "UTF-8");
URL addUserMemberURL = new URL(this.baseURL + "/groups/" +
targetGroupName + "/userMembers/" +
encodedUserID + "?idType=" + userIDType);
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment