Skip to content
Snippets Groups Projects
Commit 8ae2d384 authored by Brian Major's avatar Brian Major
Browse files

s1651 - Changed search parameter name from TYPE to IDTYPE to avoid UWS collision.

parent dc61eb8c
No related branches found
No related tags found
No related merge requests found
...@@ -68,12 +68,14 @@ ...@@ -68,12 +68,14 @@
*/ */
package ca.nrc.cadc.ac.server; package ca.nrc.cadc.ac.server;
import java.util.List;
import org.apache.log4j.Logger;
import ca.nrc.cadc.ac.IdentityType; import ca.nrc.cadc.ac.IdentityType;
import ca.nrc.cadc.ac.Role; import ca.nrc.cadc.ac.Role;
import ca.nrc.cadc.uws.Parameter; import ca.nrc.cadc.uws.Parameter;
import ca.nrc.cadc.uws.ParameterUtil; import ca.nrc.cadc.uws.ParameterUtil;
import java.util.List;
import org.apache.log4j.Logger;
/** /**
* Request Validator. This class extracts and validates the ID, TYPE, ROLE * Request Validator. This class extracts and validates the ID, TYPE, ROLE
...@@ -105,7 +107,7 @@ public class RequestValidator ...@@ -105,7 +107,7 @@ public class RequestValidator
if (paramList == null || paramList.isEmpty()) if (paramList == null || paramList.isEmpty())
{ {
throw new IllegalArgumentException( throw new IllegalArgumentException(
"Missing required parameters: ID and TYPE"); "Missing required parameters: ID and IDTYPE");
} }
// ID // ID
...@@ -118,12 +120,12 @@ public class RequestValidator ...@@ -118,12 +120,12 @@ public class RequestValidator
this.userID = param.trim(); this.userID = param.trim();
log.debug("ID: " + userID); log.debug("ID: " + userID);
// TYPE // IDTYPE
param = ParameterUtil.findParameterValue("TYPE", paramList); param = ParameterUtil.findParameterValue("IDTYPE", paramList);
if (param == null || param.trim().isEmpty()) if (param == null || param.trim().isEmpty())
{ {
throw new IllegalArgumentException( throw new IllegalArgumentException(
"TYPE parameter required but not found"); "IDTYPE parameter required but not found");
} }
this.idType = IdentityType.toValue(param); this.idType = IdentityType.toValue(param);
log.debug("TYPE: " + idType); log.debug("TYPE: " + idType);
......
...@@ -688,7 +688,7 @@ public class GMSClient ...@@ -688,7 +688,7 @@ public class GMSClient
searchGroupURL.append("/search?"); searchGroupURL.append("/search?");
searchGroupURL.append("ID=" + URLEncoder.encode(id, "UTF-8")); searchGroupURL.append("ID=" + URLEncoder.encode(id, "UTF-8"));
searchGroupURL.append("&TYPE=" + URLEncoder.encode(idType, "UTF-8")); searchGroupURL.append("&IDTYPE=" + URLEncoder.encode(idType, "UTF-8"));
searchGroupURL.append("&ROLE=" + URLEncoder.encode(roleString, "UTF-8")); searchGroupURL.append("&ROLE=" + URLEncoder.encode(roleString, "UTF-8"));
log.debug("getMemberships request to " + searchGroupURL.toString()); log.debug("getMemberships request to " + searchGroupURL.toString());
...@@ -801,7 +801,7 @@ public class GMSClient ...@@ -801,7 +801,7 @@ public class GMSClient
searchGroupURL.append("/search?"); searchGroupURL.append("/search?");
searchGroupURL.append("ID=" + URLEncoder.encode(id, "UTF-8")); searchGroupURL.append("ID=" + URLEncoder.encode(id, "UTF-8"));
searchGroupURL.append("&TYPE=" + URLEncoder.encode(idType, "UTF-8")); searchGroupURL.append("&IDTYPE=" + URLEncoder.encode(idType, "UTF-8"));
searchGroupURL.append("&ROLE=" + URLEncoder.encode(roleString, "UTF-8")); searchGroupURL.append("&ROLE=" + URLEncoder.encode(roleString, "UTF-8"));
searchGroupURL.append("&GROUPID=" + URLEncoder.encode(groupName, "UTF-8")); searchGroupURL.append("&GROUPID=" + URLEncoder.encode(groupName, "UTF-8"));
......
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