[ 
https://issues.apache.org/jira/browse/AMBER-45?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13177191#comment-13177191
 ] 

Tommaso Teofili commented on AMBER-45:
--------------------------------------

making it package local seems ok to me.
Also note that the above patch makes the integration-tests fail due to :


[ERROR] Failed to execute goal 
org.apache.maven.plugins:maven-compiler-plugin:2.3.2:testCompile 
(default-testCompile) on project oauth2-integration-tests: Compilation failure
[ERROR] 
.../amber/trunk/oauth-2.0/oauth2-integration-tests/src/test/java/org/apache/amber/oauth2/integration/endpoints/AuthzEndpoint.java:[68,16]
 authorizationResponse(javax.servlet.http.HttpServletRequest,int) in 
org.apache.amber.oauth2.as.response.OAuthASResponse cannot be applied to (int)

Nothing serious, just I think the test should be updated.


                
> Authorization Response needs to be updated to latest spec
> ---------------------------------------------------------
>
>                 Key: AMBER-45
>                 URL: https://issues.apache.org/jira/browse/AMBER-45
>             Project: Amber
>          Issue Type: Bug
>          Components: OAuth 2.0 - Authorization Server
>            Reporter: Antonio Sanso
>         Attachments: AMBER-45-patch.txt
>
>
> As for [0]
> state
>          REQUIRED if the "state" parameter was present in the client
>          authorization request.  The exact value received from the
>          client.
> patch to follow
> [0] http://tools.ietf.org/html/draft-ietf-oauth-v2-22#section-4.1.2

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to