You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by bu...@apache.org on 2020/07/21 08:41:47 UTC
[cxf] branch master updated: cxf-rt-rs-security-oauth2: remove
unused checkstyle configuration
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 <bu...@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;