diff --git a/core/src/main/java/org/apache/cxf/interceptor/LoggingInInterceptor.java b/core/src/main/java/org/apache/cxf/interceptor/LoggingInInterceptor.java index 48feaf2038a..5b1d6a36e84 100644 --- a/core/src/main/java/org/apache/cxf/interceptor/LoggingInInterceptor.java +++ b/core/src/main/java/org/apache/cxf/interceptor/LoggingInInterceptor.java @@ -177,10 +177,10 @@ protected void logReader(Message message, Reader reader, LoggingMessage buffer) if (writer.getTempFile() != null) { //large thing on disk... buffer.getMessage().append("\nMessage (saved to tmp file):\n"); - buffer.getMessage().append("Filename: " + writer.getTempFile().getAbsolutePath() + "\n"); + buffer.getMessage().append("Filename: ").append(writer.getTempFile().getAbsolutePath()).append('\n'); } if (writer.size() > limit && limit != -1) { - buffer.getMessage().append("(message truncated to " + limit + " bytes)\n"); + buffer.getMessage().append("(message truncated to ").append(limit).append(" bytes)\n"); } writer.writeCacheTo(buffer.getPayload(), limit); writer.close(); @@ -216,11 +216,11 @@ protected void logInputStream(Message message, InputStream is, LoggingMessage bu if (bos.getTempFile() != null) { //large thing on disk... buffer.getMessage().append("\nMessage (saved to tmp file):\n"); - buffer.getMessage().append("Filename: " + bos.getTempFile().getAbsolutePath() + "\n"); + buffer.getMessage().append("Filename: ").append(bos.getTempFile().getAbsolutePath()).append('\n'); } boolean truncated = false; if (bos.size() > limit && limit != -1) { - buffer.getMessage().append("(message truncated to " + limit + " bytes)\n"); + buffer.getMessage().append("(message truncated to ").append(limit).append(" bytes)\n"); truncated = true; } writePayload(buffer.getPayload(), bos, encoding, ct, truncated); diff --git a/core/src/main/java/org/apache/cxf/interceptor/LoggingOutInterceptor.java b/core/src/main/java/org/apache/cxf/interceptor/LoggingOutInterceptor.java index 0f9ef6e0af1..2d70b6589b7 100644 --- a/core/src/main/java/org/apache/cxf/interceptor/LoggingOutInterceptor.java +++ b/core/src/main/java/org/apache/cxf/interceptor/LoggingOutInterceptor.java @@ -236,14 +236,14 @@ public void onClose(CachedOutputStream cos) { if (cos.getTempFile() == null) { //buffer.append("Outbound Message:\n"); if (cos.size() >= lim) { - buffer.getMessage().append("(message truncated to " + lim + " bytes)\n"); + buffer.getMessage().append("(message truncated to ").append(lim).append(" bytes)\n"); truncated = true; } } else { buffer.getMessage().append("Outbound Message (saved to tmp file):\n"); - buffer.getMessage().append("Filename: " + cos.getTempFile().getAbsolutePath() + "\n"); + buffer.getMessage().append("Filename: ").append(cos.getTempFile().getAbsolutePath()).append('\n'); if (cos.size() >= lim) { - buffer.getMessage().append("(message truncated to " + lim + " bytes)\n"); + buffer.getMessage().append("(message truncated to ").append(lim).append(" bytes)\n"); truncated = true; } } diff --git a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/impl/UriBuilderImpl.java b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/impl/UriBuilderImpl.java index 3313247ebe5..31933b3e972 100644 --- a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/impl/UriBuilderImpl.java +++ b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/impl/UriBuilderImpl.java @@ -563,7 +563,7 @@ private UriBuilder doPath(String path, boolean checkSegments) { // such empty paths having matrix parameters... int schemeIndex = pathEncoded.indexOf("//"); if (schemeIndex != -1) { - int pathComponentStart = pathEncoded.indexOf("/", schemeIndex + 2); + int pathComponentStart = pathEncoded.indexOf('/', schemeIndex + 2); if (pathComponentStart == -1) { this.originalPathEmpty = true; pathComponentStart = pathEncoded.indexOf(';'); diff --git a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/utils/HttpUtils.java b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/utils/HttpUtils.java index 083871e7649..58bbf306fca 100644 --- a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/utils/HttpUtils.java +++ b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/utils/HttpUtils.java @@ -602,7 +602,7 @@ public static String getSetEncoding(MediaType mt, MultivaluedMap return defaultEncoding; } try { - "0".getBytes(enc); + "0".getBytes(enc); //NOPMD return enc; } catch (UnsupportedEncodingException ex) { String message = new org.apache.cxf.common.i18n.Message("UNSUPPORTED_ENCODING", diff --git a/rt/javascript/javascript-rt/src/main/java/org/apache/cxf/javascript/service/ServiceJavascriptBuilder.java b/rt/javascript/javascript-rt/src/main/java/org/apache/cxf/javascript/service/ServiceJavascriptBuilder.java index e2664dca615..9b46f825397 100755 --- a/rt/javascript/javascript-rt/src/main/java/org/apache/cxf/javascript/service/ServiceJavascriptBuilder.java +++ b/rt/javascript/javascript-rt/src/main/java/org/apache/cxf/javascript/service/ServiceJavascriptBuilder.java @@ -338,7 +338,7 @@ private void buildOperationFunction(StringBuilder parameterList) { code.append("//\n"); - code.append("function " + code.append("function " //NOPMD + opFunctionGlobalName + "(" + responseCallbackParams + ((parameterList.length() > 0 && !currentOperation.isOneWay()) diff --git a/rt/rs/description/src/main/java/org/apache/cxf/jaxrs/model/wadl/WadlGenerator.java b/rt/rs/description/src/main/java/org/apache/cxf/jaxrs/model/wadl/WadlGenerator.java index 897a2ffbb12..881e29dcc19 100644 --- a/rt/rs/description/src/main/java/org/apache/cxf/jaxrs/model/wadl/WadlGenerator.java +++ b/rt/rs/description/src/main/java/org/apache/cxf/jaxrs/model/wadl/WadlGenerator.java @@ -153,6 +153,7 @@ public class WadlGenerator implements ContainerRequestFilter { private static final String JAXB_DEFAULT_NAME = "##default"; private static final String CLASSPATH_PREFIX = "classpath:"; private static final String DEFAULT_NS_PREFIX = "prefix"; + private static final MediaType DEFAULT_REP_MEDIA_TYPE = MediaType.WILDCARD_TYPE; private static final Map> PARAMETER_TYPE_MAP; static { PARAMETER_TYPE_MAP = new EnumMap<>(ParameterType.class); @@ -195,7 +196,6 @@ public class WadlGenerator implements ContainerRequestFilter { private String applicationTitle; private String nsPrefix = DEFAULT_NS_PREFIX; private MediaType defaultWadlResponseMediaType = MediaType.APPLICATION_XML_TYPE; - private final MediaType defaultRepMediaType = MediaType.WILDCARD_TYPE; private String stylesheetReference; private boolean applyStylesheetLocally; private Bus bus; @@ -1075,7 +1075,7 @@ protected void handleRepresentation(StringBuilder sb, Set> jaxbTypes, if (MultivaluedMap.class.isAssignableFrom(type)) { types = Collections.singletonList(MediaType.APPLICATION_FORM_URLENCODED_TYPE); } else if (isWildcard(types)) { - types = Collections.singletonList(defaultRepMediaType); + types = Collections.singletonList(DEFAULT_REP_MEDIA_TYPE); } Method opMethod = getMethod(ori); diff --git a/rt/rs/extensions/search/src/main/java/org/apache/cxf/jaxrs/ext/search/SimpleSearchCondition.java b/rt/rs/extensions/search/src/main/java/org/apache/cxf/jaxrs/ext/search/SimpleSearchCondition.java index 19b6ba097a5..683473b4e3c 100644 --- a/rt/rs/extensions/search/src/main/java/org/apache/cxf/jaxrs/ext/search/SimpleSearchCondition.java +++ b/rt/rs/extensions/search/src/main/java/org/apache/cxf/jaxrs/ext/search/SimpleSearchCondition.java @@ -50,7 +50,7 @@ public class SimpleSearchCondition implements SearchCondition { SUPPORTED_TYPES.add(ConditionType.LESS_THAN); SUPPORTED_TYPES.add(ConditionType.LESS_OR_EQUALS); } - private final ConditionType joiningType = ConditionType.AND; + private static final ConditionType JOINING_TYPE = ConditionType.AND; private T condition; private List> scts; @@ -129,7 +129,7 @@ public T getCondition() { @Override public ConditionType getConditionType() { if (scts.size() > 1) { - return joiningType; + return JOINING_TYPE; } return scts.get(0).getStatement().getCondition(); } diff --git a/rt/rs/security/jose-parent/jose-jaxrs/src/main/java/org/apache/cxf/rs/security/jose/jaxrs/AbstractJweJsonWriterProvider.java b/rt/rs/security/jose-parent/jose-jaxrs/src/main/java/org/apache/cxf/rs/security/jose/jaxrs/AbstractJweJsonWriterProvider.java index ec6b16588e6..1b77075613f 100644 --- a/rt/rs/security/jose-parent/jose-jaxrs/src/main/java/org/apache/cxf/rs/security/jose/jaxrs/AbstractJweJsonWriterProvider.java +++ b/rt/rs/security/jose-parent/jose-jaxrs/src/main/java/org/apache/cxf/rs/security/jose/jaxrs/AbstractJweJsonWriterProvider.java @@ -24,7 +24,7 @@ import java.util.ArrayList; import java.util.Arrays; import java.util.Collections; -import java.util.HashSet; +import java.util.EnumSet; import java.util.LinkedList; import java.util.List; import java.util.Properties; @@ -109,7 +109,7 @@ protected List getInitializedEncryptionProviders(List keyAlgos = new HashSet<>(); + Set keyAlgos = EnumSet.noneOf(KeyAlgorithm.class); List keyProviders = new LinkedList<>(); for (int i = 0; i < propLocs.size(); i++) { diff --git a/rt/rs/security/xml/src/main/java/org/apache/cxf/rs/security/saml/AbstractSamlInHandler.java b/rt/rs/security/xml/src/main/java/org/apache/cxf/rs/security/saml/AbstractSamlInHandler.java index 9f29614b2ff..74144fc3e7e 100644 --- a/rt/rs/security/xml/src/main/java/org/apache/cxf/rs/security/saml/AbstractSamlInHandler.java +++ b/rt/rs/security/xml/src/main/java/org/apache/cxf/rs/security/saml/AbstractSamlInHandler.java @@ -337,7 +337,7 @@ private boolean compareCredentials( if (tlsCerts != null && tlsCerts.length > 0 && subjectCerts != null && subjectCerts.length > 0 && tlsCerts[0].equals(subjectCerts[0])) { return true; - } else if (tlsCerts != null && tlsCerts.length > 0 && subjectPublicKey != null + } else if (tlsCerts != null && tlsCerts.length > 0 && tlsCerts[0].getPublicKey().equals(subjectPublicKey)) { return true; } diff --git a/rt/security/src/main/java/org/apache/cxf/rt/security/crypto/CryptoUtils.java b/rt/security/src/main/java/org/apache/cxf/rt/security/crypto/CryptoUtils.java index a5adba572f3..26964017954 100644 --- a/rt/security/src/main/java/org/apache/cxf/rt/security/crypto/CryptoUtils.java +++ b/rt/security/src/main/java/org/apache/cxf/rt/security/crypto/CryptoUtils.java @@ -581,7 +581,7 @@ public static Cipher initCipher(Key secretKey, KeyProperties keyProps, int mode) if (algorithm.equals("AESWrap")) { int keySize = secretKey.getEncoded().length; algorithm = "AESWrap_" + keySize * 8; - secretKey = new SecretKeySpec(secretKey.getEncoded(), 0, keySize, "AES"); + secretKey = new SecretKeySpec(secretKey.getEncoded(), 0, keySize, "AES"); //NOPMD - false positive } Cipher c = Cipher.getInstance(algorithm); if (keyProps == null || keyProps.getAlgoSpec() == null && keyProps.getSecureRandom() == null) { diff --git a/services/sts/sts-core/src/test/java/org/apache/cxf/sts/token/provider/JWTProviderLifetimeTest.java b/services/sts/sts-core/src/test/java/org/apache/cxf/sts/token/provider/JWTProviderLifetimeTest.java index 7e297bdf7af..8339ee294c7 100644 --- a/services/sts/sts-core/src/test/java/org/apache/cxf/sts/token/provider/JWTProviderLifetimeTest.java +++ b/services/sts/sts-core/src/test/java/org/apache/cxf/sts/token/provider/JWTProviderLifetimeTest.java @@ -124,7 +124,7 @@ public void testJWTProviderLifetime() throws Exception { Instant now = Instant.now(); Long expiry = (Long)jwt.getClaim(JwtConstants.CLAIM_EXPIRY); - Instant.ofEpochSecond(expiry).isAfter(now); + assertTrue(Instant.ofEpochSecond(expiry).isAfter(now)); } /** diff --git a/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/JAXRSClientServerSpringBookTest.java b/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/JAXRSClientServerSpringBookTest.java index 4a609d41237..ff61f02e3f5 100644 --- a/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/JAXRSClientServerSpringBookTest.java +++ b/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/JAXRSClientServerSpringBookTest.java @@ -286,7 +286,7 @@ public void testGetWadlFromWadlLocation() throws Exception { String piData = ((ProcessingInstruction)n).getData(); int hRefStart = piData.indexOf("href=\""); if (hRefStart > 0) { - int hRefEnd = piData.indexOf("\"", hRefStart + 6); + int hRefEnd = piData.indexOf('"', hRefStart + 6); templateRef = piData.substring(hRefStart + 6, hRefEnd); } } diff --git a/systests/ldap/src/test/java/org/apache/cxf/systest/ldap/sts/LDAPClaimsTest.java b/systests/ldap/src/test/java/org/apache/cxf/systest/ldap/sts/LDAPClaimsTest.java index d259a0d700b..138afcd11dc 100644 --- a/systests/ldap/src/test/java/org/apache/cxf/systest/ldap/sts/LDAPClaimsTest.java +++ b/systests/ldap/src/test/java/org/apache/cxf/systest/ldap/sts/LDAPClaimsTest.java @@ -138,7 +138,7 @@ public void testRetrieveClaims() throws Exception { if (expectedClaims.contains(c.getClaimType())) { expectedClaims.remove(c.getClaimType()); } else { - Assert.assertTrue("Claim '" + c.getClaimType() + "' not requested", false); + Assert.fail("Claim '" + c.getClaimType() + "' not requested"); } } } @@ -171,7 +171,7 @@ public void testRetrieveClaimsUsingLDAPLookup() throws Exception { if (expectedClaims.contains(c.getClaimType())) { expectedClaims.remove(c.getClaimType()); } else { - Assert.assertTrue("Claim '" + c.getClaimType() + "' not requested", false); + Assert.fail("Claim '" + c.getClaimType() + "' not requested"); } } } @@ -210,7 +210,7 @@ public void testMultiUserBaseDNs() throws Exception { if (expectedClaims.contains(c.getClaimType())) { expectedClaims.remove(c.getClaimType()); } else { - Assert.assertTrue("Claim '" + c.getClaimType() + "' not requested", false); + Assert.fail("Claim '" + c.getClaimType() + "' not requested"); } } @@ -232,7 +232,7 @@ public void testMultiUserBaseDNs() throws Exception { if (expectedClaims.contains(c.getClaimType())) { expectedClaims.remove(c.getClaimType()); } else { - Assert.assertTrue("Claim '" + c.getClaimType() + "' not requested", false); + Assert.fail("Claim '" + c.getClaimType() + "' not requested"); } } } @@ -295,7 +295,7 @@ public void testRetrieveClaimsWithUnsupportedOptionalClaimType() throws Exceptio if (expectedClaims.contains(c.getClaimType())) { expectedClaims.remove(c.getClaimType()); } else { - Assert.assertTrue("Claim '" + c.getClaimType() + "' not requested", false); + Assert.fail("Claim '" + c.getClaimType() + "' not requested"); } } } diff --git a/systests/transport-jms/src/test/java/org/apache/cxf/jms/testsuite/util/JMSTestUtil.java b/systests/transport-jms/src/test/java/org/apache/cxf/jms/testsuite/util/JMSTestUtil.java index 27791ed2f81..46c3fd7fa3f 100644 --- a/systests/transport-jms/src/test/java/org/apache/cxf/jms/testsuite/util/JMSTestUtil.java +++ b/systests/transport-jms/src/test/java/org/apache/cxf/jms/testsuite/util/JMSTestUtil.java @@ -84,7 +84,7 @@ public static TestCaseType getTestCase(String testId) { private static void loadTestCases() throws Exception { JAXBContext context = JAXBContext.newInstance("org.apache.cxf.testsuite.testcase"); Unmarshaller unmarshaller = context.createUnmarshaller(); - JAXBElement e = (JAXBElement)unmarshaller.unmarshal(new JMSTestUtil().getClass() + JAXBElement e = (JAXBElement)unmarshaller.unmarshal(JMSTestUtil.class .getResource("/org/apache/cxf/jms/testsuite/util/testcases.xml")); testcases = (TestCasesType)e.getValue(); } diff --git a/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/IDLToWSDLProcessor.java b/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/IDLToWSDLProcessor.java index 538ed2f6397..9ec42ef7d40 100644 --- a/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/IDLToWSDLProcessor.java +++ b/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/IDLToWSDLProcessor.java @@ -436,7 +436,7 @@ private Map getServiceNames(Binding[] bindings, boolean isDefaul for (int j = 0; j < bindingTokens.length - 2; j++) { name.append(bindingTokens[j]).append('.'); } - name.append(bindingTokens[bindingTokens.length - 2] + "CORBAService"); + name.append(bindingTokens[bindingTokens.length - 2]).append("CORBAService"); serviceNames.put(ns, name.toString()); } else { serviceNames.put(ns, idl + "CORBAService"); diff --git a/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/ScopedNameVisitor.java b/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/ScopedNameVisitor.java index 80427772934..299ad4ac5bc 100644 --- a/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/ScopedNameVisitor.java +++ b/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/ScopedNameVisitor.java @@ -622,7 +622,7 @@ protected static void populateAliasSchemaType(CorbaType corbaType, Alias alias = (Alias) corbaType; //loop through alias base types, till you get a non-alias corba type CorbaType type = findCorbaType(typeMap, alias.getBasetype()); - while ((type != null) && type instanceof Alias) { + while (type instanceof Alias) { alias = (Alias) type; type = findCorbaType(typeMap, alias.getBasetype()); }