This is an automated email from the ASF dual-hosted git repository. buhhunyx pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/cxf.git
The following commit(s) were added to refs/heads/master by this push: new fc8019f cxf-rt-rs-security-oauth2: remove unused checkstyle configuration fc8019f is described below commit fc8019fd4c3bdb579d177f98c2cafe4391746e74 Author: Alexey Markevich <buhhu...@gmail.com> AuthorDate: Tue Jul 21 11:40:53 2020 +0300 cxf-rt-rs-security-oauth2: remove unused checkstyle configuration --- .../cxf/rs/security/oauth2/client/JoseClientCodeStateManager.java | 3 +-- .../cxf/rs/security/oauth2/services/RedirectionBasedGrantService.java | 2 +- .../cxf/rs/security/oauth2/grants/jwt/AbstractJwtHandlerTest.java | 3 +-- 3 files changed, 3 insertions(+), 5 deletions(-) diff --git a/rt/rs/security/oauth-parent/oauth2/src/main/java/org/apache/cxf/rs/security/oauth2/client/JoseClientCodeStateManager.java b/rt/rs/security/oauth-parent/oauth2/src/main/java/org/apache/cxf/rs/security/oauth2/client/JoseClientCodeStateManager.java index 08613fc..b0fffa4 100644 --- a/rt/rs/security/oauth-parent/oauth2/src/main/java/org/apache/cxf/rs/security/oauth2/client/JoseClientCodeStateManager.java +++ b/rt/rs/security/oauth-parent/oauth2/src/main/java/org/apache/cxf/rs/security/oauth2/client/JoseClientCodeStateManager.java @@ -107,10 +107,9 @@ public class JoseClientCodeStateManager implements ClientCodeStateManager { throw new SecurityException(); } String json = jws.getUnsignedEncodedSequence(); - //CHECKSTYLE:OFF + Map<String, List<String>> map = CastUtils.cast((Map<?, ?>)jsonp.fromJson(json)); return (MultivaluedMap<String, String>)map; //NOPMD - //CHECKSTYLE:ON } public void setSignatureProvider(JwsSignatureProvider signatureProvider) { diff --git a/rt/rs/security/oauth-parent/oauth2/src/main/java/org/apache/cxf/rs/security/oauth2/services/RedirectionBasedGrantService.java b/rt/rs/security/oauth-parent/oauth2/src/main/java/org/apache/cxf/rs/security/oauth2/services/RedirectionBasedGrantService.java index 38ddcf7..cee3334 100644 --- a/rt/rs/security/oauth-parent/oauth2/src/main/java/org/apache/cxf/rs/security/oauth2/services/RedirectionBasedGrantService.java +++ b/rt/rs/security/oauth-parent/oauth2/src/main/java/org/apache/cxf/rs/security/oauth2/services/RedirectionBasedGrantService.java @@ -243,11 +243,11 @@ public abstract class RedirectionBasedGrantService extends AbstractOAuthService return finalResponse; } - //CHECKSTYLE:OFF public Set<String> getSupportedResponseTypes() { return supportedResponseTypes; } + protected boolean canAuthorizationBeSkipped(MultivaluedMap<String, String> params, Client client, UserSubject userSubject, diff --git a/rt/rs/security/oauth-parent/oauth2/src/test/java/org/apache/cxf/rs/security/oauth2/grants/jwt/AbstractJwtHandlerTest.java b/rt/rs/security/oauth-parent/oauth2/src/test/java/org/apache/cxf/rs/security/oauth2/grants/jwt/AbstractJwtHandlerTest.java index 143d972..77681c2 100644 --- a/rt/rs/security/oauth-parent/oauth2/src/test/java/org/apache/cxf/rs/security/oauth2/grants/jwt/AbstractJwtHandlerTest.java +++ b/rt/rs/security/oauth-parent/oauth2/src/test/java/org/apache/cxf/rs/security/oauth2/grants/jwt/AbstractJwtHandlerTest.java @@ -44,9 +44,8 @@ public class AbstractJwtHandlerTest { private static final byte[] SIGNATURE = "mySignature".getBytes(); @Rule - //CHECKSTYLE:OFF public EasyMockRule rule = new EasyMockRule(this); - //CHECKSTYLE:ON + private AbstractJwtHandler handler; @Mock private JwsSignatureVerifier signatureVerifier;