Author: chirag
Date: Mon Mar 15 18:00:15 2010
New Revision: 923365
URL: http://svn.apache.org/viewvc?rev=923365&view=rev
Log:
Fix SHINDIG-1302 - Make allowParam and isContainerInjectedParameter protected
to ease extendability of OAuthRequest
Code Review: http://codereview.appspot.com/476042/show
Modified:
shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/oauth/OAuthRequest.java
Modified:
shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/oauth/OAuthRequest.java
URL:
http://svn.apache.org/viewvc/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/oauth/OAuthRequest.java?rev=923365&r1=923364&r2=923365&view=diff
==============================================================================
---
shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/oauth/OAuthRequest.java
(original)
+++
shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/oauth/OAuthRequest.java
Mon Mar 15 18:00:15 2010
@@ -392,7 +392,7 @@ public class OAuthRequest {
return list;
}
- private boolean allowParam(String paramName) {
+ protected boolean allowParam(String paramName) {
String canonParamName = paramName.toLowerCase();
return (!(canonParamName.startsWith("oauth") ||
canonParamName.startsWith("xoauth") ||
@@ -913,7 +913,7 @@ public class OAuthRequest {
return result;
}
- private static boolean isContainerInjectedParameter(String key) {
+ protected static boolean isContainerInjectedParameter(String key) {
key = key.toLowerCase();
return key.startsWith("oauth") || key.startsWith("xoauth") ||
key.startsWith("opensocial");
}