You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@oltu.apache.org by to...@apache.org on 2011/12/06 22:07:59 UTC
svn commit: r1211146 - in
/incubator/amber/trunk/oauth-2.0/oauth2-authzserver/src:
main/java/org/apache/amber/oauth2/as/validator/CodeValidator.java
test/java/org/apache/amber/oauth2/as/OAuthRequestTest.java
Author: tommaso
Date: Tue Dec 6 21:07:59 2011
New Revision: 1211146
URL: http://svn.apache.org/viewvc?rev=1211146&view=rev
Log:
[AMBER-30] - applied patch from Antonio Sanso
Modified:
incubator/amber/trunk/oauth-2.0/oauth2-authzserver/src/main/java/org/apache/amber/oauth2/as/validator/CodeValidator.java
incubator/amber/trunk/oauth-2.0/oauth2-authzserver/src/test/java/org/apache/amber/oauth2/as/OAuthRequestTest.java
Modified: incubator/amber/trunk/oauth-2.0/oauth2-authzserver/src/main/java/org/apache/amber/oauth2/as/validator/CodeValidator.java
URL: http://svn.apache.org/viewvc/incubator/amber/trunk/oauth-2.0/oauth2-authzserver/src/main/java/org/apache/amber/oauth2/as/validator/CodeValidator.java?rev=1211146&r1=1211145&r2=1211146&view=diff
==============================================================================
--- incubator/amber/trunk/oauth-2.0/oauth2-authzserver/src/main/java/org/apache/amber/oauth2/as/validator/CodeValidator.java (original)
+++ incubator/amber/trunk/oauth-2.0/oauth2-authzserver/src/main/java/org/apache/amber/oauth2/as/validator/CodeValidator.java Tue Dec 6 21:07:59 2011
@@ -39,7 +39,6 @@ public class CodeValidator extends Abstr
public CodeValidator() {
requiredParams.add(OAuth.OAUTH_RESPONSE_TYPE);
requiredParams.add(OAuth.OAUTH_CLIENT_ID);
- requiredParams.add(OAuth.OAUTH_REDIRECT_URI);
}
@Override
Modified: incubator/amber/trunk/oauth-2.0/oauth2-authzserver/src/test/java/org/apache/amber/oauth2/as/OAuthRequestTest.java
URL: http://svn.apache.org/viewvc/incubator/amber/trunk/oauth-2.0/oauth2-authzserver/src/test/java/org/apache/amber/oauth2/as/OAuthRequestTest.java?rev=1211146&r1=1211145&r2=1211146&view=diff
==============================================================================
--- incubator/amber/trunk/oauth-2.0/oauth2-authzserver/src/test/java/org/apache/amber/oauth2/as/OAuthRequestTest.java (original)
+++ incubator/amber/trunk/oauth-2.0/oauth2-authzserver/src/test/java/org/apache/amber/oauth2/as/OAuthRequestTest.java Tue Dec 6 21:07:59 2011
@@ -128,26 +128,6 @@ public class OAuthRequestTest {
}
verify(request);
-
- reset(request);
-
- expect(request.getParameter(OAuth.OAUTH_RESPONSE_TYPE)).andStubReturn(ResponseType.CODE.toString());
- expect(request.getMethod()).andStubReturn(OAuth.HttpMethod.GET);
- expect(request.getContentType()).andStubReturn(OAuth.ContentType.URL_ENCODED);
-
- expect(request.getParameter(OAuth.OAUTH_CLIENT_ID)).andStubReturn("client_id");
- expect(request.getParameter(OAuth.OAUTH_REDIRECT_URI)).andStubReturn(null);
- replay(request);
-
- try {
- new OAuthAuthzRequest(request);
- fail("Exception expected");
- } catch (OAuthProblemException e) {
- Assert.assertEquals(OAuthError.TokenResponse.INVALID_REQUEST, e.getError());
- }
-
- verify(request);
-
}
@Test