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

Stein Welberg commented on AMBER-64:
------------------------------------

I created a new patch. Both the QueryparameterApplier and the test have been 
updated. Now it's not possible anymore that the location uri contains both 
query parameters and fragment params at the same time. It also discards any 
params that are not allowed according to the spec.
                
> QueryParameterApplier needs to include the scope parameter in the fragment
> --------------------------------------------------------------------------
>
>                 Key: AMBER-64
>                 URL: https://issues.apache.org/jira/browse/AMBER-64
>             Project: Amber
>          Issue Type: Bug
>            Reporter: Stein Welberg
>            Assignee: Antonio Sanso
>         Attachments: AMBER-64_improved.patch
>
>
> According to the spec (see [0]) the scope parameter also needs to be included 
> in the url fragment if it is provided.
> Please find the patch attached to fix this attached to this issue.
> [0] http://tools.ietf.org/html/rfc6749#section-4.2.2

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to