You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@shindig.apache.org by ch...@apache.org on 2010/03/15 19:00:15 UTC

svn commit: r923365 - /shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/oauth/OAuthRequest.java

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");
   }