You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by se...@apache.org on 2012/01/30 23:20:03 UTC
svn commit: r1238046 - in
/cxf/trunk/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth:
data/ filters/ services/ utils/
Author: sergeyb
Date: Mon Jan 30 22:20:02 2012
New Revision: 1238046
URL: http://svn.apache.org/viewvc?rev=1238046&view=rev
Log:
Minor cleanup in the OAuth code
Modified:
cxf/trunk/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/data/RequestTokenRegistration.java
cxf/trunk/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/filters/AbstractAuthFilter.java
cxf/trunk/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/filters/OAuthServletFilter.java
cxf/trunk/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/services/RequestTokenHandler.java
cxf/trunk/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/utils/OAuthConstants.java
Modified: cxf/trunk/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/data/RequestTokenRegistration.java
URL: http://svn.apache.org/viewvc/cxf/trunk/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/data/RequestTokenRegistration.java?rev=1238046&r1=1238045&r2=1238046&view=diff
==============================================================================
--- cxf/trunk/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/data/RequestTokenRegistration.java (original)
+++ cxf/trunk/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/data/RequestTokenRegistration.java Mon Jan 30 22:20:02 2012
@@ -28,7 +28,6 @@ public class RequestTokenRegistration {
private Client client;
private String state;
private String callback;
- private List<String> uris;
private List<String> scopes;
private long lifetime;
private long issuedAt;
@@ -54,12 +53,6 @@ public class RequestTokenRegistration {
public String getState() {
return state;
}
- public void setUris(List<String> uris) {
- this.uris = uris;
- }
- public List<String> getUris() {
- return uris;
- }
public void setScopes(List<String> scopes) {
this.scopes = scopes;
}
Modified: cxf/trunk/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/filters/AbstractAuthFilter.java
URL: http://svn.apache.org/viewvc/cxf/trunk/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/filters/AbstractAuthFilter.java?rev=1238046&r1=1238045&r2=1238046&view=diff
==============================================================================
--- cxf/trunk/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/filters/AbstractAuthFilter.java (original)
+++ cxf/trunk/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/filters/AbstractAuthFilter.java Mon Jan 30 22:20:02 2012
@@ -56,7 +56,6 @@ import org.apache.cxf.security.SecurityC
* Base OAuth filter which can be used to protect end-user endpoints
*/
public class AbstractAuthFilter {
- protected static final String USE_USER_SUBJECT = "org.apache.cxf.rs.security.oauth.use_user_subject";
private static final Logger LOG = LogUtils.getL7dLogger(AbstractAuthFilter.class);
private static final String[] REQUIRED_PARAMETERS =
new String[] {
@@ -74,7 +73,6 @@ public class AbstractAuthFilter {
ALLOWED_OAUTH_PARAMETERS.addAll(Arrays.asList(REQUIRED_PARAMETERS));
ALLOWED_OAUTH_PARAMETERS.add(OAuth.OAUTH_VERSION);
ALLOWED_OAUTH_PARAMETERS.add(OAuthConstants.X_OAUTH_SCOPE);
- ALLOWED_OAUTH_PARAMETERS.add(OAuthConstants.X_OAUTH_URI);
ALLOWED_OAUTH_PARAMETERS.add(OAuthConstants.OAUTH_CONSUMER_SECRET);
}
Modified: cxf/trunk/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/filters/OAuthServletFilter.java
URL: http://svn.apache.org/viewvc/cxf/trunk/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/filters/OAuthServletFilter.java?rev=1238046&r1=1238045&r2=1238046&view=diff
==============================================================================
--- cxf/trunk/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/filters/OAuthServletFilter.java (original)
+++ cxf/trunk/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/filters/OAuthServletFilter.java Mon Jan 30 22:20:02 2012
@@ -43,6 +43,8 @@ import org.apache.cxf.security.SecurityC
* HTTP Servlet filter which can be used to protect end user endpoints
*/
public class OAuthServletFilter extends AbstractAuthFilter implements javax.servlet.Filter {
+ protected static final String USE_USER_SUBJECT = "org.apache.cxf.rs.security.oauth.use_user_subject";
+
public void init(FilterConfig filterConfig) throws ServletException {
ServletContext servletContext = filterConfig.getServletContext();
super.setDataProvider(OAuthUtils.getOAuthDataProvider(servletContext));
Modified: cxf/trunk/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/services/RequestTokenHandler.java
URL: http://svn.apache.org/viewvc/cxf/trunk/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/services/RequestTokenHandler.java?rev=1238046&r1=1238045&r2=1238046&view=diff
==============================================================================
--- cxf/trunk/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/services/RequestTokenHandler.java (original)
+++ cxf/trunk/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/services/RequestTokenHandler.java Mon Jan 30 22:20:02 2012
@@ -56,7 +56,6 @@ public class RequestTokenHandler {
private long tokenLifetime = 3600L;
private String defaultScope;
- private String defaultURI;
public Response handle(MessageContext mc, OAuthDataProvider dataProvider) {
try {
@@ -77,14 +76,11 @@ public class RequestTokenHandler {
List<String> scopes = OAuthUtils.parseParamValue(
oAuthMessage.getParameter(OAuthConstants.X_OAUTH_SCOPE), defaultScope);
- List<String> uris = OAuthUtils.parseParamValue(
- oAuthMessage.getParameter(OAuthConstants.X_OAUTH_URI), defaultURI);
RequestTokenRegistration reg = new RequestTokenRegistration();
reg.setClient(client);
reg.setCallback(callback);
reg.setState(oAuthMessage.getParameter("state"));
- reg.setUris(uris);
reg.setScopes(scopes);
reg.setLifetime(tokenLifetime);
reg.setIssuedAt(System.currentTimeMillis() / 1000);
@@ -144,9 +140,5 @@ public class RequestTokenHandler {
public void setDefaultScope(String defaultScope) {
this.defaultScope = defaultScope;
}
-
- public void setDefaultURI(String defaultURI) {
- this.defaultURI = defaultURI;
- }
}
Modified: cxf/trunk/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/utils/OAuthConstants.java
URL: http://svn.apache.org/viewvc/cxf/trunk/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/utils/OAuthConstants.java?rev=1238046&r1=1238045&r2=1238046&view=diff
==============================================================================
--- cxf/trunk/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/utils/OAuthConstants.java (original)
+++ cxf/trunk/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/utils/OAuthConstants.java Mon Jan 30 22:20:02 2012
@@ -36,7 +36,6 @@ public final class OAuthConstants {
public static final String AUTHORIZATION_DECISION_ALLOW = "allow";
public static final String AUTHORIZATION_DECISION_DENY = "deny";
- public static final String X_OAUTH_URI = "x_oauth_uri";
public static final String X_OAUTH_SCOPE = "x_oauth_scope";
public static final String OAUTH_CONSUMER_SECRET = "oauth_consumer_secret";