-
Notifications
You must be signed in to change notification settings - Fork 368
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Introduce extension point for building requests to authorize EP #2127
Conversation
...ntity.oauth.par/src/main/java/org/wso2/carbon/identity/oauth/par/core/ParRequestBuilder.java
Outdated
Show resolved
Hide resolved
...ndpoint/src/main/java/org/wso2/carbon/identity/oauth/endpoint/authz/OAuth2AuthzEndpoint.java
Outdated
Show resolved
Hide resolved
...oauth.par/src/main/java/org/wso2/carbon/identity/oauth/par/model/OAuthParRequestWrapper.java
Outdated
Show resolved
Hide resolved
...th.endpoint/src/main/java/org/wso2/carbon/identity/oauth/endpoint/par/OAuth2ParEndpoint.java
Show resolved
Hide resolved
...th.endpoint/src/main/java/org/wso2/carbon/identity/oauth/endpoint/par/OAuth2ParEndpoint.java
Outdated
Show resolved
Hide resolved
....oauth.endpoint/src/main/java/org/wso2/carbon/identity/oauth/endpoint/util/EndpointUtil.java
Outdated
Show resolved
Hide resolved
....oauth.endpoint/src/main/java/org/wso2/carbon/identity/oauth/endpoint/util/EndpointUtil.java
Outdated
Show resolved
Hide resolved
...o2.carbon.identity.oauth/src/main/java/org/wso2/carbon/identity/oauth2/util/RequestUtil.java
Outdated
Show resolved
Hide resolved
041758b
to
9bf56fb
Compare
9bf56fb
to
81fa5f4
Compare
....oauth.endpoint/src/main/java/org/wso2/carbon/identity/oauth/endpoint/util/EndpointUtil.java
Show resolved
Hide resolved
....oauth.endpoint/src/main/java/org/wso2/carbon/identity/oauth/endpoint/util/EndpointUtil.java
Outdated
Show resolved
Hide resolved
...o2.carbon.identity.oauth/src/main/java/org/wso2/carbon/identity/oauth2/util/RequestUtil.java
Outdated
Show resolved
Hide resolved
...o2.carbon.identity.oauth/src/main/java/org/wso2/carbon/identity/oauth2/util/RequestUtil.java
Outdated
Show resolved
Hide resolved
...th.endpoint/src/main/java/org/wso2/carbon/identity/oauth/endpoint/par/OAuth2ParEndpoint.java
Outdated
Show resolved
Hide resolved
...oauth.par/src/main/java/org/wso2/carbon/identity/oauth/par/model/OAuthParRequestWrapper.java
Show resolved
Hide resolved
....oauth.endpoint/src/main/java/org/wso2/carbon/identity/oauth/endpoint/util/EndpointUtil.java
Outdated
Show resolved
Hide resolved
@@ -332,7 +332,7 @@ | |||
<limit implementation="org.jacoco.report.check.Limit"> | |||
<counter>COMPLEXITY</counter> | |||
<value>COVEREDRATIO</value> | |||
<minimum>0.53</minimum> | |||
<minimum>0.52</minimum> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is the reason to reduce the code cover ratio?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unit tests have not been added covering the PAR implementation. Will be addressed in a new PR.
...ntity.oauth.par/src/main/java/org/wso2/carbon/identity/oauth/par/core/ParRequestBuilder.java
Outdated
Show resolved
Hide resolved
Have proper parent Exception for ParAuthFailureException, check IdentityException |
...ntity.oauth.par/src/main/java/org/wso2/carbon/identity/oauth/par/core/ParRequestBuilder.java
Outdated
Show resolved
Hide resolved
4dc01d6
to
aa33241
Compare
...carbon.identity.oauth.par/src/main/java/org/wso2/carbon/identity/oauth/par/util/ParUtil.java
Outdated
Show resolved
Hide resolved
...bon.identity.oauth/src/main/java/org/wso2/carbon/identity/oauth2/AbstractRequestBuilder.java
Outdated
Show resolved
Hide resolved
...ntity.oauth.par/src/main/java/org/wso2/carbon/identity/oauth/par/core/ParRequestBuilder.java
Outdated
Show resolved
Hide resolved
...uth/src/main/java/org/wso2/carbon/identity/oauth2/internal/OAuth2ServiceComponentHolder.java
Outdated
Show resolved
Hide resolved
...uth/src/main/java/org/wso2/carbon/identity/oauth2/internal/OAuth2ServiceComponentHolder.java
Outdated
Show resolved
Hide resolved
...o2.carbon.identity.oauth/src/main/java/org/wso2/carbon/identity/oauth2/util/RequestUtil.java
Show resolved
Hide resolved
a56aae3
to
447a085
Compare
...tity.oauth.par/src/main/java/org/wso2/carbon/identity/oauth/par/core/ParAuthServiceImpl.java
Show resolved
Hide resolved
...o2.carbon.identity.oauth/src/main/java/org/wso2/carbon/identity/oauth2/util/RequestUtil.java
Outdated
Show resolved
Hide resolved
9b12504
to
a10086d
Compare
....oauth.par/src/main/java/org/wso2/carbon/identity/oauth/par/core/OAuthParRequestWrapper.java
Outdated
Show resolved
Hide resolved
...ity.oauth/src/main/java/org/wso2/carbon/identity/oauth2/internal/OAuth2ServiceComponent.java
Outdated
Show resolved
Hide resolved
...tity.oauth.par/src/main/java/org/wso2/carbon/identity/oauth/par/core/ParAuthServiceImpl.java
Outdated
Show resolved
Hide resolved
...ntity.oauth.par/src/main/java/org/wso2/carbon/identity/oauth/par/core/ParRequestBuilder.java
Outdated
Show resolved
Hide resolved
...ntity.oauth.par/src/main/java/org/wso2/carbon/identity/oauth/par/core/ParRequestBuilder.java
Outdated
Show resolved
Hide resolved
....oauth.par/src/main/java/org/wso2/carbon/identity/oauth/par/core/OAuthParRequestWrapper.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved based on that a PR with unit tests will be sent soon for this.
Unit tests added with: #2134 |
Proposed changes in this pull request
This PR introduces:
Related Issues