You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by co...@apache.org on 2015/08/06 16:46:25 UTC

[1/2] cxf git commit: Fixing some issues thrown up by analysis.apache.org

Repository: cxf
Updated Branches:
  refs/heads/3.0.x-fixes dd0340f1d -> 2d81cd8f0


Fixing some issues thrown up by analysis.apache.org


Project: http://git-wip-us.apache.org/repos/asf/cxf/repo
Commit: http://git-wip-us.apache.org/repos/asf/cxf/commit/a727d21f
Tree: http://git-wip-us.apache.org/repos/asf/cxf/tree/a727d21f
Diff: http://git-wip-us.apache.org/repos/asf/cxf/diff/a727d21f

Branch: refs/heads/3.0.x-fixes
Commit: a727d21fa73bd8d5753782eb20f01034e7e4ea48
Parents: dd0340f
Author: Colm O hEigeartaigh <co...@apache.org>
Authored: Thu Aug 6 15:43:55 2015 +0100
Committer: Colm O hEigeartaigh <co...@apache.org>
Committed: Thu Aug 6 15:44:47 2015 +0100

----------------------------------------------------------------------
 .../apache/cxf/rs/security/jose/jwk/JsonWebKeys.java    | 12 ++++++++----
 .../org/apache/cxf/rs/security/jose/jwk/JwkUtils.java   |  2 +-
 2 files changed, 9 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cxf/blob/a727d21f/rt/rs/security/jose/src/main/java/org/apache/cxf/rs/security/jose/jwk/JsonWebKeys.java
----------------------------------------------------------------------
diff --git a/rt/rs/security/jose/src/main/java/org/apache/cxf/rs/security/jose/jwk/JsonWebKeys.java b/rt/rs/security/jose/src/main/java/org/apache/cxf/rs/security/jose/jwk/JsonWebKeys.java
index 1ca3ca8..55a19f6 100644
--- a/rt/rs/security/jose/src/main/java/org/apache/cxf/rs/security/jose/jwk/JsonWebKeys.java
+++ b/rt/rs/security/jose/src/main/java/org/apache/cxf/rs/security/jose/jwk/JsonWebKeys.java
@@ -112,15 +112,19 @@ public class JsonWebKeys extends JsonMapObject {
         return map;
     }
     public List<JsonWebKey> getKeys(String keyType) {
-        return getKeyTypeMap().get(keyType);
+        KeyType kt = KeyType.getKeyType(keyType);
+        if (kt == null) {
+            return null;
+        }
+        return getKeyTypeMap().get(kt);
     }
     public List<JsonWebKey> getRsaKeys() {
-        return getKeyTypeMap().get(JsonWebKey.KEY_TYPE_RSA);
+        return getKeyTypeMap().get(KeyType.RSA);
     }
     public List<JsonWebKey> getEllipticKeys() {
-        return getKeyTypeMap().get(JsonWebKey.KEY_TYPE_ELLIPTIC);
+        return getKeyTypeMap().get(KeyType.EC);
     }
     public List<JsonWebKey> getSecretKeys() {
-        return getKeyTypeMap().get(JsonWebKey.KEY_TYPE_OCTET);
+        return getKeyTypeMap().get(KeyType.OCTET);
     }
 }

http://git-wip-us.apache.org/repos/asf/cxf/blob/a727d21f/rt/rs/security/jose/src/main/java/org/apache/cxf/rs/security/jose/jwk/JwkUtils.java
----------------------------------------------------------------------
diff --git a/rt/rs/security/jose/src/main/java/org/apache/cxf/rs/security/jose/jwk/JwkUtils.java b/rt/rs/security/jose/src/main/java/org/apache/cxf/rs/security/jose/jwk/JwkUtils.java
index e1f161c..f996178 100644
--- a/rt/rs/security/jose/src/main/java/org/apache/cxf/rs/security/jose/jwk/JwkUtils.java
+++ b/rt/rs/security/jose/src/main/java/org/apache/cxf/rs/security/jose/jwk/JwkUtils.java
@@ -512,7 +512,7 @@ public final class JwkUtils {
     }
     public static void setPublicKeyInfo(JsonWebKey jwk, JoseHeaders headers, String algo,
                                         boolean reportPublicKey, boolean reportPublicKeyId) {
-        if (reportPublicKey && JsonWebKey.KEY_TYPE_RSA.equals(jwk.getKeyType())) {
+        if (reportPublicKey && KeyType.RSA.equals(jwk.getKeyType())) {
             List<String> chain = CastUtils.cast((List<?>)jwk.getProperty("x5c"));
             //TODO: if needed the chain can be reported as part of a 'jwk' property
             if (chain != null) {


[2/2] cxf git commit: Checkstyle fix

Posted by co...@apache.org.
Checkstyle fix


Project: http://git-wip-us.apache.org/repos/asf/cxf/repo
Commit: http://git-wip-us.apache.org/repos/asf/cxf/commit/2d81cd8f
Tree: http://git-wip-us.apache.org/repos/asf/cxf/tree/2d81cd8f
Diff: http://git-wip-us.apache.org/repos/asf/cxf/diff/2d81cd8f

Branch: refs/heads/3.0.x-fixes
Commit: 2d81cd8f04a8c52daa142b37fd0e042ec6a1fbab
Parents: a727d21
Author: Colm O hEigeartaigh <co...@apache.org>
Authored: Thu Aug 6 15:46:16 2015 +0100
Committer: Colm O hEigeartaigh <co...@apache.org>
Committed: Thu Aug 6 15:46:16 2015 +0100

----------------------------------------------------------------------
 .../cxf/rs/security/jose/jwk/JwkUtilsTest.java  | 22 ++++++++++----------
 1 file changed, 11 insertions(+), 11 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cxf/blob/2d81cd8f/rt/rs/security/jose/src/test/java/org/apache/cxf/rs/security/jose/jwk/JwkUtilsTest.java
----------------------------------------------------------------------
diff --git a/rt/rs/security/jose/src/test/java/org/apache/cxf/rs/security/jose/jwk/JwkUtilsTest.java b/rt/rs/security/jose/src/test/java/org/apache/cxf/rs/security/jose/jwk/JwkUtilsTest.java
index 34d3183..c18d2a1 100644
--- a/rt/rs/security/jose/src/test/java/org/apache/cxf/rs/security/jose/jwk/JwkUtilsTest.java
+++ b/rt/rs/security/jose/src/test/java/org/apache/cxf/rs/security/jose/jwk/JwkUtilsTest.java
@@ -23,17 +23,17 @@ import org.junit.Test;
 
 public class JwkUtilsTest extends Assert {
     private static final String RSA_KEY = "{"
-      + "\"kty\": \"RSA\","
-      + "\"n\": \"0vx7agoebGcQSuuPiLJXZptN9nndrQmbXEps2aiAFbWhM78LhWx4cbbfAAt"
-      +      "VT86zwu1RK7aPFFxuhDR1L6tSoc_BJECPebWKRXjBZCiFV4n3oknjhMstn6"
-      +      "4tZ_2W-5JsGY4Hc5n9yBXArwl93lqt7_RN5w6Cf0h4QyQ5v-65YGjQR0_FD"
-      +      "W2QvzqY368QQMicAtaSqzs8KJZgnYb9c7d0zgdAZHzu6qMQvRL5hajrn1n9"
-      +      "1CbOpbISD08qNLyrdkt-bFTWhAI4vMQFh6WeZu0fM4lFd2NcRwr3XPksINH"
-      +      "aQ-G_xBniIqbw0Ls1jF44-csFCur-kEgU8awapJzKnqDKgw\","
-      + "\"e\": \"AQAB\","
-      + "\"alg\": \"RS256\","
-      + "\"kid\": \"2011-04-29\""
-      + "}\"";
+        + "\"kty\": \"RSA\","
+        + "\"n\": \"0vx7agoebGcQSuuPiLJXZptN9nndrQmbXEps2aiAFbWhM78LhWx4cbbfAAt"
+        +      "VT86zwu1RK7aPFFxuhDR1L6tSoc_BJECPebWKRXjBZCiFV4n3oknjhMstn6"
+        +      "4tZ_2W-5JsGY4Hc5n9yBXArwl93lqt7_RN5w6Cf0h4QyQ5v-65YGjQR0_FD"
+        +      "W2QvzqY368QQMicAtaSqzs8KJZgnYb9c7d0zgdAZHzu6qMQvRL5hajrn1n9"
+        +      "1CbOpbISD08qNLyrdkt-bFTWhAI4vMQFh6WeZu0fM4lFd2NcRwr3XPksINH"
+        +      "aQ-G_xBniIqbw0Ls1jF44-csFCur-kEgU8awapJzKnqDKgw\","
+        + "\"e\": \"AQAB\","
+        + "\"alg\": \"RS256\","
+        + "\"kid\": \"2011-04-29\""
+        + "}\"";
     @Test
     public void testRsaKeyThumbprint() throws Exception {
         String thumbprint = JwkUtils.getThumbprint(RSA_KEY);