Built by Maven

PMD Results

The following document contains the results of PMD 7.14.0.

Violations By Priority

Priority 1

Wren Security Commons Audit - CSV Event Handler - org/forgerock/audit/handlers/csv/CsvAuditEventHandler.java

Rule Violation Line
DoubleCheckedLocking Double checked locking is not thread safe in Java. 303

Wren Security Commons Utility Classes - Utility Library - org/forgerock/util/LazyList.java

Rule Violation Line
DoubleCheckedLocking Double checked locking is not thread safe in Java. 61

Wren Security Commons Utility Classes - Utility Library - org/forgerock/util/LazyMap.java

Rule Violation Line
DoubleCheckedLocking Double checked locking is not thread safe in Java. 62

Priority 2

Wren Security Commons Authn. - JASPI (JSR-196) JWT Session Module - org/forgerock/jaspi/modules/session/jwt/JwtSessionModule.java

Rule Violation Line
AvoidBranchingStatementAsLastInLoop Avoid using a branching statement as the last in a loop. 106

Wren Security Commons Authn. - JASPI (JSR-196) OpenAM Session Module - org/forgerock/jaspi/modules/session/openam/OpenAMSessionModule.java

Rule Violation Line
AvoidBranchingStatementAsLastInLoop Avoid using a branching statement as the last in a loop. 424

Priority 3

Wren Security Commons Audit - Framework Core - org/forgerock/audit/AuditServiceImpl.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 16–543

Wren Security Commons Audit - Framework Core - org/forgerock/audit/events/AccessAuditEventBuilder.java

Rule Violation Line
UnusedPrivateField Avoid unused private fields such as 'logger'. 121

Wren Security Commons Audit - Framework Core - org/forgerock/audit/events/EventTopicsMetaDataBuilder.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'final' on resource specification 'configStream': resource specifications are implicitly final 147

Wren Security Commons Audit - Framework Core - org/forgerock/audit/events/handlers/writers/RotatableWriter.java

Rule Violation Line
CollapsibleIfStatements This if statement could be combined with its parent 151–153
CollapsibleIfStatements This if statement could be combined with its parent 225–227

Wren Security Commons Audit - Framework Core - org/forgerock/audit/events/handlers/writers/TextWriter.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'public' on class 'Stream': the class is declared in an interface type 59

Wren Security Commons Audit - Framework Core - org/forgerock/audit/filter/FilterChainBuilder.java

Rule Violation Line
EmptyCatchBlock Avoid empty catch blocks 70–72

Wren Security Commons Audit - Framework Core - org/forgerock/audit/rotation/FixedTimeRotationPolicy.java

Rule Violation Line
UnusedPrivateField Avoid unused private fields such as 'logger'. 31

Wren Security Commons Audit - Framework Core - org/forgerock/audit/secure/KeyStoreHandler.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'getStore': the method is declared in an interface type 31
UnnecessaryModifier Unnecessary modifier 'public' on method 'setStore': the method is declared in an interface type 41
UnnecessaryModifier Unnecessary modifier 'public' on method 'getPassword': the method is declared in an interface type 48
UnnecessaryModifier Unnecessary modifier 'public' on method 'getLocation': the method is declared in an interface type 55
UnnecessaryModifier Unnecessary modifier 'public' on method 'getType': the method is declared in an interface type 62
UnnecessaryModifier Unnecessary modifier 'public' on method 'store': the method is declared in an interface type 70

Wren Security Commons Audit - Framework Core - org/forgerock/audit/util/DateUtil.java

Rule Violation Line
EmptyCatchBlock Avoid empty catch blocks 186–188

Wren Security Commons Audit - Framework Core - org/forgerock/audit/util/JsonValueUtils.java

Rule Violation Line
UnnecessaryReturn Unnecessary return statement 303

Wren Security Commons Audit - CSV Event Handler - org/forgerock/audit/handlers/csv/CsvAuditEventHandler.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 16–609

Wren Security Commons Audit - CSV Event Handler - org/forgerock/audit/handlers/csv/CsvSecureMapReader.java

Rule Violation Line
UnusedPrivateField Avoid unused private fields such as 'logger'. 34

Wren Security Commons Audit - CSV Event Handler - org/forgerock/audit/handlers/csv/SecureCsvWriter.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 16–435
SimplifiedTernary This conditional expression can be simplified with || or && 302

Wren Security Commons Audit - CSV Event Handler - org/forgerock/audit/handlers/csv/StandardCsvWriter.java

Rule Violation Line
UnusedPrivateField Avoid unused private fields such as 'logger'. 45
SimplifiedTernary This conditional expression can be simplified with || or && 116

Wren Security Commons Audit - Elasticsearch Event Handler - org/forgerock/audit/handlers/elasticsearch/ElasticsearchAuditEventHandler.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 16–528

Wren Security Commons Audit - Elasticsearch Event Handler - org/forgerock/audit/handlers/elasticsearch/ElasticsearchQueryFilterVisitor.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 16–280

Wren Security Commons Audit - JDBC Event Handler - org/forgerock/audit/handlers/jdbc/BufferedJdbcAuditEventExecutor.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'final' on resource specification 'preparedStatement': resource specifications are implicitly final 192

Wren Security Commons Audit - JDBC Event Handler - org/forgerock/audit/handlers/jdbc/JdbcAuditEventExecutorImpl.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'final' on resource specification 'preparedStatement': resource specifications are implicitly final 57

Wren Security Commons Audit - JDBC Event Handler - org/forgerock/audit/handlers/jdbc/StringSqlQueryFilterVisitor.java

Rule Violation Line
UselessQualifiedThis Useless qualified this usage in the same class. 68

Wren Security Commons Audit - JMS Event Handler - org/forgerock/audit/handlers/jms/JmsAuditEventHandler.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17–304
EmptyCatchBlock Avoid empty catch blocks 257–259

Wren Security Commons Audit - JSON Event Handler - org/forgerock/audit/handlers/json/JsonAuditEventHandler.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17–226
UnnecessaryModifier Unnecessary modifier 'final' on resource specification 'reader': resource specifications are implicitly final 136–137
UnnecessaryModifier Unnecessary modifier 'final' on resource specification 'reader': resource specifications are implicitly final 163–164

Wren Security Commons Audit - JSON Event Handler - org/forgerock/audit/handlers/json/JsonFileWriter.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 18–534

Wren Security Commons Audit - Splunk Event Handler - org/forgerock/audit/handlers/splunk/SplunkAuditEventHandler.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 16–249

Wren Security Commons Audit - Syslog Event Handler - org/forgerock/audit/handlers/syslog/SyslogFormatter.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17–326

Wren Security Commons Audit - JSON Support Library - org/forgerock/audit/json/AuditJsonConfig.java

Rule Violation Line
UnusedPrivateField Avoid unused private fields such as 'EVENTS_FIELD'. 57

Wren Security Commons Audit - Servlet Example - org/forgerock/audit/AuditHttpApplication.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'final' on resource specification 'eventHandlersConfig': resource specifications are implicitly final 70

Wren Security Commons Authn. - JASPI (JSR-196) Functional Tests - org/forgerock/caf/authn/test/configuration/AuditResource.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17–130

Wren Security Commons Authn. - JASPI (JSR-196) IWA Module - org/forgerock/jaspi/modules/iwa/wdsso/WDSSO.java

Rule Violation Line
CollapsibleIfStatements This if statement could be combined with its parent 370–373

Wren Security Commons Authn. - JASPI (JSR-196) JWT Session Module - org/forgerock/jaspi/modules/session/jwt/AbstractJwtSessionModule.java

Rule Violation Line
SimplifiedTernary This conditional expression can be simplified with || or && 188
SimplifiedTernary This conditional expression can be simplified with || or && 190
SimplifiedTernary This conditional expression can be simplified with || or && 192

Wren Security Commons Authn. - JASPI (JSR-196) JWT Session Module - org/forgerock/jaspi/modules/session/jwt/JwtSessionModule.java

Rule Violation Line
SimplifiedTernary This conditional expression can be simplified with || or && 175–177

Wren Security Commons Authn. - JASPI (JSR-196) OpenAM Session Module - org/forgerock/jaspi/modules/session/openam/OpenAMSessionModule.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17–462
UnusedPrivateField Avoid unused private fields such as 'MIME_TYPE_APPLICATION_JSON'. 82
UnusedPrivateField Avoid unused private fields such as 'CHARACTER_ENCODING'. 83

Wren Security Commons Authn. - JASPI (JSR-196) OIDC Module - org/forgerock/jaspi/modules/openid/OpenIdConnectModule.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17–272

Wren Security Commons Authn. - JASPI (JSR-196) OIDC Module - org/forgerock/jaspi/modules/openid/helpers/SimpleHTTPClient.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'final' on resource specification 'reader': resource specifications are implicitly final 91

Wren Security Commons Authn. - JASPI (JSR-196) OIDC Module - org/forgerock/jaspi/modules/openid/resolvers/OpenIdResolver.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifiers 'public static final' on field 'KEY_ALIAS_KEY': the field is declared in an interface type 41
UnnecessaryModifier Unnecessary modifiers 'public static final' on field 'ISSUER_KEY': the field is declared in an interface type 46
UnnecessaryModifier Unnecessary modifiers 'public static final' on field 'CLIENT_SECRET_KEY': the field is declared in an interface type 51
UnnecessaryModifier Unnecessary modifiers 'public static final' on field 'JWK': the field is declared in an interface type 56
UnnecessaryModifier Unnecessary modifiers 'public static final' on field 'WELL_KNOWN_CONFIGURATION': the field is declared in an interface type 61
UnnecessaryModifier Unnecessary modifiers 'public static final' on field 'KEYSTORE_LOCATION_KEY': the field is declared in an interface type 66
UnnecessaryModifier Unnecessary modifiers 'public static final' on field 'KEYSTORE_TYPE_KEY': the field is declared in an interface type 71
UnnecessaryModifier Unnecessary modifiers 'public static final' on field 'KEYSTORE_PASS_KEY': the field is declared in an interface type 76
UnnecessaryModifier Unnecessary modifier 'public' on method 'validateIdentity': the method is declared in an interface type 84
UnnecessaryModifier Unnecessary modifier 'public' on method 'getIssuer': the method is declared in an interface type 91

Wren Security Commons Authn. - JASPI (JSR-196) OIDC Module - org/forgerock/jaspi/modules/openid/resolvers/service/OpenIdResolverService.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'getResolverForIssuer': the method is declared in an interface type 53
UnnecessaryModifier Unnecessary modifier 'public' on method 'configureResolverWithKey': the method is declared in an interface type 66
UnnecessaryModifier Unnecessary modifier 'public' on method 'configureResolverWithSecret': the method is declared in an interface type 78
UnnecessaryModifier Unnecessary modifier 'public' on method 'configureResolverWithJWK': the method is declared in an interface type 90
UnnecessaryModifier Unnecessary modifier 'public' on method 'configureResolverWithWellKnownOpenIdConfiguration': the method is declared in an interface type 101

Wren Security Commons Authn. - JASPI (JSR-196) OIDC Module - org/forgerock/jaspi/modules/openid/resolvers/service/OpenIdResolverServiceConfigurator.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'configureService': the method is declared in an interface type 40

Wren Security Commons Authn. - JASPI (JSR-196) OIDC Module - org/forgerock/jaspi/modules/openid/resolvers/service/OpenIdResolverServiceImpl.java

Rule Violation Line
UnusedPrivateField Avoid unused private fields such as 'readTimeout'. 50
UnusedPrivateField Avoid unused private fields such as 'connTimeout'. 51

Wren Security Commons Authn. - JASPI (JSR-196) Runtime - org/forgerock/caf/authentication/framework/AggregateAuthContext.java

Rule Violation Line
UnusedPrivateField Avoid unused private fields such as 'logger'. 56

Wren Security Commons Authn. - JASPI (JSR-196) Runtime - org/forgerock/caf/authentication/framework/AuthenticationFramework.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 18–275

Wren Security Commons Authn. - JASPI (JSR-196) Runtime - org/forgerock/caf/authentication/framework/SessionAuthContext.java

Rule Violation Line
UnusedPrivateField Avoid unused private fields such as 'logger'. 42

Wren Security Commons Authz. - OAuth2 Filter Module - org/forgerock/authz/modules/oauth2/OAuth2CrestAuthorizationModule.java

Rule Violation Line
UnusedFormalParameter Avoid unused method parameters such as 'securityContext'. 212

Wren Security Commons Bloom Filters - Core - org/forgerock/bloomfilter/BatchingBloomFilter.java

Rule Violation Line
CollapsibleIfStatements This if statement could be combined with its parent 80–94

Wren Security Commons Bloom Filters - Core - org/forgerock/bloomfilter/BloomFilters.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'static' on enum 'NeverExpires': nested enums are implicitly static 49

Wren Security Commons Bloom Filters - Monitoring - org/forgerock/bloomfilter/monitoring/BloomFilterMonitor.java

Rule Violation Line
CollapsibleIfStatements This if statement could be combined with its parent 300–305

Wren Security Guice Core - org/forgerock/guice/core/InjectorConfiguration.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'private' on constructor 'InjectorConfiguration()': enum constructors are implicitly private 62

Wren Security Guice Core - org/forgerock/guice/core/InjectorHolder.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'private' on constructor 'InjectorHolder()': enum constructors are implicitly private 53

Wren Security Guice Core - org/forgerock/guice/core/internal/commons/lang/reflect/MemberUtils.java

Rule Violation Line
EmptyCatchBlock Avoid empty catch blocks 52–54
EmptyCatchBlock Avoid empty catch blocks 85–87

Wren Security Commons HTTP - CHF Bindings Test Utilities - org/forgerock/http/bindings/BindingTest.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 16–391
UnnecessaryModifier Unnecessary modifier 'final' on resource specification 'handler': resource specifications are implicitly final 171
UnnecessaryModifier Unnecessary modifier 'final' on resource specification 'handler': resource specifications are implicitly final 191
UnnecessaryModifier Unnecessary modifier 'final' on resource specification 'handler': resource specifications are implicitly final 218
UnnecessaryModifier Unnecessary modifier 'final' on resource specification 'handler': resource specifications are implicitly final 243
UnnecessaryModifier Unnecessary modifier 'final' on resource specification 'httpClientHandler': resource specifications are implicitly final 287

Wren Security Commons HTTP - Apache Async HttpClient integration - org/forgerock/http/apache/async/AsyncHttpClientProvider.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17–194

Wren Security Commons HTTP - Apache HttpClient integration - org/forgerock/http/apache/sync/SyncHttpClientProvider.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 19–126

Wren Security Commons HTTP - Client Test Utilities - org/forgerock/http/handler/HttpClientHandlerTest.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17–163

Wren Security Commons HTTP - Core Library - org/forgerock/http/header/ContentLengthHeader.java

Rule Violation Line
EmptyCatchBlock Avoid empty catch blocks 59–61

Wren Security Commons HTTP - Core Library - org/forgerock/http/header/HeaderUtil.java

Rule Violation Line
EmptyControlStatement Empty if statement 218–222

Wren Security Commons HTTP - Core Library - org/forgerock/http/io/BranchingStreamWrapper.java

Rule Violation Line
EmptyCatchBlock Avoid empty catch blocks 199–201

Wren Security Commons HTTP - Core Library - org/forgerock/http/io/FileBuffer.java

Rule Violation Line
EmptyCatchBlock Avoid empty catch blocks 122–124

Wren Security Commons HTTP - Core Library - org/forgerock/http/protocol/Cookie.java

Rule Violation Line
SimplifiedTernary This conditional expression can be simplified with || or && 172
SimplifiedTernary This conditional expression can be simplified with || or && 221

Wren Security Commons HTTP - Core Library - org/forgerock/http/protocol/Entity.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17–488

Wren Security Commons HTTP - Core Library - org/forgerock/http/protocol/Request.java

Rule Violation Line
EmptyCatchBlock Avoid empty catch blocks 84–86

Wren Security Commons HTTP - Core Library - org/forgerock/http/protocol/Response.java

Rule Violation Line
UnnecessarySemicolon Unnecessary semicolon 148

Wren Security Commons HTTP - Core Library - org/forgerock/http/routing/RouteMatchers.java

Rule Violation Line
EmptyCatchBlock Avoid empty catch blocks 286–288

Wren Security Commons HTTP - Core Library - org/forgerock/http/routing/UriRouteMatcher.java

Rule Violation Line
EmptyControlStatement Empty else statement 217–219

Wren Security Commons HTTP - Core Library - org/forgerock/http/spi/Loader.java

Rule Violation Line
UnnecessarySemicolon Unnecessary semicolon 41

Wren Security Commons HTTP - Core Library - org/forgerock/http/util/Uris.java

Rule Violation Line
JumbledIncrementer Avoid modifying an outer loop incrementer in an inner loop for update expression 411–428

Wren Security Commons HTTP - Core Library - org/forgerock/services/routing/AbstractRouter.java

Rule Violation Line
CollapsibleIfStatements This if statement could be combined with its parent 269–271

Wren Security Commons HTTP - OpenAPI Descriptor Example Module - org/forgerock/http/example/DescribedOauth2Endpoint.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17–233

Wren Security Commons HTTP - Grizzly Integration Library - org/forgerock/http/grizzly/HandlerAdapter.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17–221

Wren Security Commons HTTP - OAuth 2.0 Resource Server Support - org/forgerock/http/oauth2/resolver/OpenAmAccessTokenResolver.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17–226

Wren Security Commons HTTP - Servlet integration library - org/forgerock/http/servlet/HttpFrameworkServlet.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17–374

Wren Security I18N Maven Plugin - org/forgerock/i18n/maven/AbstractGenerateMessagesMojo.java

Rule Violation Line
EmptyControlStatement Empty if statement 223–225
CollapsibleIfStatements This if statement could be combined with its parent 648–652
EmptyCatchBlock Avoid empty catch blocks 748–750
EmptyCatchBlock Avoid empty catch blocks 756–758

Wren Security I18N Maven Plugin - org/forgerock/i18n/maven/CleanMessagesMojo.java

Rule Violation Line
CollapsibleIfStatements This if statement could be combined with its parent 358–361

Wren Security Commons JSON Crypto - Core Library - org/forgerock/json/crypto/JsonCrypto.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17–159

Wren Security Commons JSON Schema - Draft-04 Validator - org/forgerock/json/schema/validator/helpers/FormatHelper.java

Rule Violation Line
UnusedFormalParameter Avoid unused constructor parameters such as 'format'. 40

Wren Security Commons JSON Schema - Draft-04 Validator - org/forgerock/json/schema/validator/helpers/MaximumHelper.java

Rule Violation Line
CollapsibleIfStatements This if statement could be combined with its parent 84–86

Wren Security Commons JSON Schema - Draft-04 Validator - org/forgerock/json/schema/validator/helpers/MinimumHelper.java

Rule Violation Line
CollapsibleIfStatements This if statement could be combined with its parent 85–87

Wren Security Commons JSON Schema - Draft-04 Validator - org/forgerock/json/schema/validator/validators/BooleanTypeValidator.java

Rule Violation Line
UnnecessaryReturn Unnecessary return statement 58

Wren Security Commons JSON Schema - Draft-04 Validator - org/forgerock/json/schema/validator/validators/IntegerTypeValidator.java

Rule Violation Line
CollapsibleIfStatements This if statement could be combined with its parent 116–118

Wren Security Commons JSON Schema - Draft-04 Validator - org/forgerock/json/schema/validator/validators/NumberTypeValidator.java

Rule Violation Line
CollapsibleIfStatements This if statement could be combined with its parent 117–119

Wren Security Commons JSON Schema - Draft-04 Validator - org/forgerock/json/schema/validator/validators/ObjectTypeValidator.java

Rule Violation Line
EmptyCatchBlock Avoid empty catch blocks 173–176
EmptyControlStatement Empty if statement 195–203
CollapsibleIfStatements This if statement could be combined with its parent 229–240

Wren Security Commons JSON Schema - Draft-04 Validator - org/forgerock/json/schema/validator/validators/SimpleValidator.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'validate': the method is declared in an interface type 41

Wren Security Commons JSON Schema - Draft-04 Validator - org/forgerock/json/schema/validator/validators/StringTypeValidator.java

Rule Violation Line
EmptyCatchBlock Avoid empty catch blocks 110–112
CollapsibleIfStatements This if statement could be combined with its parent 133–135

Wren Security Commons JSON Schema - Draft-04 Validator - org/forgerock/json/schema/validator/validators/UnionTypeValidator.java

Rule Violation Line
EmptyCatchBlock Avoid empty catch blocks 83–85

Wren Security Commons JWT - org/forgerock/json/jose/jwe/JweAlgorithmType.java

Rule Violation Line
UselessOverridingMethod Overriding method merely calls super 40

Wren Security Commons JWT - org/forgerock/json/jose/jwe/handlers/encryption/AbstractRSAESPkcs1V15AesCbcHmacEncryptionHandler.java

Rule Violation Line
UnusedFormalParameter Avoid unused constructor parameters such as 'signingManager'. 46

Wren Security Commons JWT - org/forgerock/json/jose/jwe/handlers/encryption/RSAEncryptionHandler.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17–164

Wren Security Commons JWT - org/forgerock/json/jose/jwk/KeyType.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'private' on constructor 'KeyType(String)': enum constructors are implicitly private 49

Wren Security Commons JWT - org/forgerock/json/jose/jwk/KeyUse.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'private' on constructor 'KeyUse()': enum constructors are implicitly private 38

Wren Security Commons JWT - org/forgerock/json/jose/jws/JwsAlgorithm.java

Rule Violation Line
UselessOverridingMethod Overriding method merely calls super 152

Wren Security Commons JWT - org/forgerock/json/jose/jws/JwtSecureHeader.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17–409

Wren Security Commons JWT - org/forgerock/json/jose/utils/DerUtils.java

Rule Violation Line
CollapsibleIfStatements This if statement could be combined with its parent 57–59

Wren Security Commons - API Descriptor - org/forgerock/api/CrestApiProducer.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17–158

Wren Security Commons - API Descriptor - org/forgerock/api/commons/CommonsApi.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'private' on constructor 'Errors(int, String)': enum constructors are implicitly private 176

Wren Security Commons - API Descriptor - org/forgerock/api/jackson/CrestPropertyDetailsSchemaFactoryWrapper.java

Rule Violation Line
UselessOverridingMethod Overriding method merely calls super 137

Wren Security Commons - API Descriptor - org/forgerock/api/jackson/CrestStringSchema.java

Rule Violation Line
UselessOperationOnImmutable The result of an operation on an immutable object is ignored 139
UselessOperationOnImmutable The result of an operation on an immutable object is ignored 149
UnnecessaryReturn Unnecessary return statement 187

Wren Security Commons - API Descriptor - org/forgerock/api/markup/ApiDocGenerator.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 18–1130

Wren Security Commons - API Descriptor - org/forgerock/api/markup/asciidoc/AsciiDoc.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17–623
UnusedLocalVariable Avoid unused local variables such as 'symbol'. 283

Wren Security Commons - API Descriptor - org/forgerock/api/models/Action.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'final' on method 'action': the method is already in a final class 117

Wren Security Commons - API Descriptor - org/forgerock/api/models/ApiDescription.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 16–340

Wren Security Commons - API Descriptor - org/forgerock/api/models/Create.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'final' on method 'create': the method is already in a final class 77

Wren Security Commons - API Descriptor - org/forgerock/api/models/Delete.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'final' on method 'delete': the method is already in a final class 41

Wren Security Commons - API Descriptor - org/forgerock/api/models/Patch.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'final' on method 'patch': the method is already in a final class 85

Wren Security Commons - API Descriptor - org/forgerock/api/models/Read.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'final' on method 'read': the method is already in a final class 41

Wren Security Commons - API Descriptor - org/forgerock/api/models/Update.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'final' on method 'update': the method is already in a final class 41

Wren Security Commons - API Descriptor - org/forgerock/api/transform/OpenApiTransformer.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17–1624

Wren Security Commons - API Descriptor - org/forgerock/api/util/ValidationUtil.java

Rule Violation Line
SimplifiedTernary This conditional expression can be simplified with || or && 91

Wren Security Commons REST - JSON Resource Examples - org/forgerock/json/resource/examples/DynamicRealmDemo.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17–212

Wren Security Commons REST - JSON Resource Examples - org/forgerock/json/resource/examples/ReadModifyWriteDemo.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'final' on resource specification 'server': resource specifications are implicitly final 47
UnnecessaryModifier Unnecessary modifier 'final' on resource specification 'connection': resource specifications are implicitly final 48

Wren Security Commons REST - JSON Resource Examples - org/forgerock/json/resource/examples/VersionedResourcesDemo.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17–203

Wren Security Commons REST - JSON Resource Examples - org/forgerock/json/resource/http/examples/CrestHttpApplication.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17–181

Wren Security Commons REST - JSON Resource HTTP Binding - org/forgerock/json/resource/http/CrestAdapter.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17–478

Wren Security Commons REST - JSON Resource HTTP Binding - org/forgerock/json/resource/http/HttpAdapter.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17–900
EmptyControlStatement Empty if statement 352–356
EmptyControlStatement Empty if statement 520–524
EmptyControlStatement Empty if statement 590–594
CollapsibleIfStatements This if statement could be combined with its parent 656–658

Wren Security Commons REST - JSON Resource HTTP Binding - org/forgerock/json/resource/http/HttpUtils.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17–951
EmptyCatchBlock Avoid empty catch blocks 586–589

Wren Security Commons REST - JSON Resource HTTP Binding - org/forgerock/json/resource/http/RequestRunner.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17–557
UnnecessaryModifier Unnecessary modifier 'final' on method 'handleError': the method is already in a final class 129
UnnecessaryModifier Unnecessary modifier 'final' on method 'handleResult': the method is already in a final class 136
UnnecessaryModifier Unnecessary modifier 'final' on method 'visitActionRequest': the method is already in a final class 147
UnnecessaryModifier Unnecessary modifier 'final' on method 'visitCreateRequest': the method is already in a final class 183
UnnecessaryModifier Unnecessary modifier 'final' on method 'visitDeleteRequest': the method is already in a final class 232
UnnecessaryModifier Unnecessary modifier 'final' on method 'visitPatchRequest': the method is already in a final class 247
UnnecessaryModifier Unnecessary modifier 'final' on method 'visitQueryRequest': the method is already in a final class 262
UnnecessaryModifier Unnecessary modifier 'final' on method 'visitReadRequest': the method is already in a final class 344
UnnecessaryModifier Unnecessary modifier 'final' on method 'visitUpdateRequest': the method is already in a final class 359
UnusedFormalParameter Avoid unused method parameters such as 'e'. 374

Wren Security Commons REST - JSON Resource Core Library - org/forgerock/json/resource/DescribableResourceHandler.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'final' on method 'api': the method is already in a final class 63

Wren Security Commons REST - JSON Resource Core Library - org/forgerock/json/resource/MemoryBackend.java

Rule Violation Line
EmptyControlStatement Empty else statement 454–456

Wren Security Commons REST - JSON Resource Core Library - org/forgerock/json/resource/Requests.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'final' on method 'addSortKey': the method is already in a final class 460

Wren Security Commons REST - JSON Resource Core Library - org/forgerock/json/resource/Resources.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17–556

Wren Security Commons REST - JSON Resource Core Library - org/forgerock/json/resource/Router.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17–453

Forgerock Commons - Security - org/forgerock/security/keystore/KeystoreManagerException.java

Rule Violation Line
UnusedFormalParameter Avoid unused constructor parameters such as 'cause'. 48

Wren Security Commons Self Service - Core - org/forgerock/selfservice/core/AnonymousProcessService.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17–313

Wren Security Commons Self Service - Core - org/forgerock/selfservice/core/ProcessContextImpl.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17–238

Wren Security Commons Self Service - Core - org/forgerock/selfservice/core/UserUpdateService.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 16–169

Wren Security Commons Self Service - Core - org/forgerock/selfservice/core/crypto/FieldStorageScheme.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'hashField': the method is declared in an interface type 31
UnnecessaryModifier Unnecessary modifier 'public' on method 'fieldMatches': the method is declared in an interface type 41

Wren Security Commons Self Service - Core - org/forgerock/selfservice/core/util/RequirementsBuilder.java

Rule Violation Line
CollapsibleIfStatements This if statement could be combined with its parent 229–231

Wren Security Commons Self Service - Examples - org/forgerock/selfservice/example/ExampleEmailService.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 16–160

Wren Security Commons Self Service - Stages - org/forgerock/selfservice/stages/email/VerifyEmailAccountStage.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17–195

Wren Security Commons Self Service - Stages - org/forgerock/selfservice/stages/kba/SecurityAnswerDefinitionStage.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17–146

Wren Security Commons Self Service - Stages - org/forgerock/selfservice/stages/user/EmailUsernameStage.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 16–116

Wren Security Commons Utility Classes - Utility Library - org/forgerock/json/JsonValue.java

Rule Violation Line
EmptyCatchBlock Avoid empty catch blocks 1019–1021

Wren Security Commons Utility Classes - Utility Library - org/forgerock/json/RangeSet.java

Rule Violation Line
EmptyCatchBlock Avoid empty catch blocks 104–106

Wren Security Commons Utility Classes - Utility Library - org/forgerock/util/SimpleHTTPClient.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'final' on resource specification 'reader': resource specifications are implicitly final 96

Wren Security Commons Utility Classes - Utility Library - org/forgerock/util/i18n/LocalizableString.java

Rule Violation Line
EmptyCatchBlock Avoid empty catch blocks 98–100

Wren Security Commons Utility Classes - Utility Library - org/forgerock/util/query/QueryFilter.java

Rule Violation Line
OverrideBothEqualsAndHashcode Ensure you override both equals() and hashCode() 221

Wren Security Commons Utility Classes - Utility Library - org/forgerock/util/query/QueryFilterParser.java

Rule Violation Line
EmptyControlStatement Empty for statement 382–384

Wren Security Commons Utility Classes - Utility Library - org/forgerock/util/thread/listener/ShutdownListener.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'shutdown': the method is declared in an interface type 28

Wren Security Commons Utility Classes - Utility Library - org/forgerock/util/thread/listener/ShutdownPriority.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'private' on constructor 'ShutdownPriority(int)': enum constructors are implicitly private 50

Wren Security Commons Utility Classes - Utility Library - org/forgerock/util/time/Duration.java

Rule Violation Line
TooManyStaticImports Too many static imports may lead to messy code 17–415

Wren Security Commons Utility Classes - Utility Library - org/forgerock/util/xml/XMLUtils.java

Rule Violation Line
EmptyCatchBlock Avoid empty catch blocks 91
EmptyCatchBlock Avoid empty catch blocks 122

Priority 4

Wren Security Commons Audit - Framework Core - org/forgerock/audit/events/handlers/writers/RotatableWriter.java

Rule Violation Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'TimeUnit': 'MILLISECONDS' is already in scope because it is imported in this file 374
UnnecessaryFullyQualifiedName Unnecessary qualifier 'TimeUnit': 'MILLISECONDS' is already in scope because it is imported in this file 375
UnnecessaryFullyQualifiedName Unnecessary qualifier 'TimeUnit': 'MILLISECONDS' is already in scope because it is imported in this file 376

Wren Security Commons Audit - Framework Core - org/forgerock/audit/secure/KeyStoreSecureStorage.java

Rule Violation Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'KeyStoreSecureStorage': 'SIGNATURE_ALGORITHM' is already in scope because it is declared in an enclosing type 95
UnnecessaryFullyQualifiedName Unnecessary qualifier 'KeyStoreSecureStorage': 'SIGNATURE_ALGORITHM' is already in scope because it is declared in an enclosing type 104
UnnecessaryFullyQualifiedName Unnecessary qualifier 'KeyStoreSecureStorage': 'ENTRY_CURRENT_KEY' is already in scope because it is declared in an enclosing type 127
UnnecessaryFullyQualifiedName Unnecessary qualifier 'KeyStoreSecureStorage': 'ENTRY_INITIAL_KEY' is already in scope because it is declared in an enclosing type 132
UnnecessaryFullyQualifiedName Unnecessary qualifier 'KeyStoreSecureStorage': 'ENTRY_CURRENT_SIGNATURE' is already in scope because it is declared in an enclosing type 137
UnnecessaryFullyQualifiedName Unnecessary qualifier 'KeyStoreSecureStorage': 'ENTRY_CURRENT_KEY' is already in scope because it is declared in an enclosing type 148
UnnecessaryFullyQualifiedName Unnecessary qualifier 'KeyStoreSecureStorage': 'ENTRY_INITIAL_KEY' is already in scope because it is declared in an enclosing type 154

Wren Security Commons Audit - CSV Event Handler - org/forgerock/audit/handlers/csv/CsvAuditEventHandler.java

Rule Violation Line
UselessParentheses Useless parentheses. 537

Wren Security Commons Audit - JSON Event Handler - org/forgerock/audit/handlers/json/JsonFileWriter.java

Rule Violation Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'Math': 'max' is already in scope 101

Wren Security Commons Authn. - JASPI (JSR-196) IWA Module - org/forgerock/jaspi/modules/iwa/wdsso/Base64.java

Rule Violation Line
UselessParentheses Useless parentheses. 108
UselessParentheses Useless parentheses. 247
UselessParentheses Useless parentheses. 295
UselessParentheses Useless parentheses. 433
UselessParentheses Useless parentheses. 594

Wren Security Commons Authn. - JASPI (JSR-196) IWA Module - org/forgerock/jaspi/modules/iwa/wdsso/DerValue.java

Rule Violation Line
UselessParentheses Useless parentheses. 119

Wren Security Commons Authn. - JASPI (JSR-196) IWA Module - org/forgerock/jaspi/modules/iwa/wdsso/WDSSO.java

Rule Violation Line
UselessParentheses Useless parentheses. 368
UselessParentheses Useless parentheses. 370

Wren Security Commons Authn. - JASPI (JSR-196) OIDC Module - org/forgerock/jaspi/modules/openid/OpenIdConnectModule.java

Rule Violation Line
UnnecessaryImport Unused import 'jakarta.security.auth.message.AuthException' 29
UnnecessaryFullyQualifiedName Unnecessary qualifier 'OpenIdConnectModule': 'HEADER_KEY' is already in scope because it is declared in an enclosing type 140
UnnecessaryFullyQualifiedName Unnecessary qualifier 'OpenIdConnectModule': 'READ_TIMEOUT_KEY' is already in scope because it is declared in an enclosing type 143
UnnecessaryFullyQualifiedName Unnecessary qualifier 'OpenIdConnectModule': 'CONNECTION_TIMEOUT_KEY' is already in scope because it is declared in an enclosing type 144
UnnecessaryFullyQualifiedName Unnecessary qualifier 'OpenIdConnectModule': 'RESOLVERS_KEY' is already in scope because it is declared in an enclosing type 162

Wren Security Commons Authn. - JASPI (JSR-196) OIDC Module - org/forgerock/jaspi/modules/openid/resolvers/BaseOpenIdResolver.java

Rule Violation Line
UselessParentheses Useless parentheses. 104

Wren Security Commons Authn. - JASPI (JSR-196) OIDC Module - org/forgerock/jaspi/modules/openid/resolvers/service/OpenIdResolverServiceConfiguratorImpl.java

Rule Violation Line
UselessParentheses Useless parentheses. 116
UselessParentheses Useless parentheses. 117
UselessParentheses Useless parentheses. 118

Wren Security Commons Authn. - JASPI (JSR-196) Runtime - org/forgerock/caf/authentication/framework/AuthenticationFramework.java

Rule Violation Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'AuthenticationFramework': 'ATTRIBUTE_AUTH_CONTEXT' is already in scope because it is declared in an enclosing type 139
UnnecessaryFullyQualifiedName Unnecessary qualifier 'AuthenticationFramework': 'ATTRIBUTE_AUTH_CONTEXT' is already in scope because it is declared in an enclosing type 168
UnnecessaryFullyQualifiedName Unnecessary qualifier 'AuthenticationFramework': 'ATTRIBUTE_AUTH_PRINCIPAL' is already in scope because it is declared in an enclosing type 172
UnnecessaryFullyQualifiedName Unnecessary qualifier 'AuthenticationFramework': 'ATTRIBUTE_AUTH_CONTEXT' is already in scope because it is declared in an enclosing type 173
UnnecessaryFullyQualifiedName Unnecessary qualifier 'AuthenticationFramework': 'ATTRIBUTE_REQUEST_ID' is already in scope because it is declared in an enclosing type 188

Wren Security Commons Bloom Filters - Core - org/forgerock/bloomfilter/AtomicBloomFilter.java

Rule Violation Line
UselessParentheses Useless parentheses. 115

Wren Security Commons Bloom Filters - Core - org/forgerock/bloomfilter/BloomFilterChain.java

Rule Violation Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'Math': 'max' is already in scope 146

Wren Security Commons Bloom Filters - Core - org/forgerock/bloomfilter/BloomFilterStatistics.java

Rule Violation Line
UselessParentheses Useless parentheses. 106
UselessParentheses Useless parentheses. 138

Wren Security Commons Bloom Filters - Monitoring - org/forgerock/bloomfilter/monitoring/BloomFilterMonitor.java

Rule Violation Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'TimeUnit': 'SECONDS' is already in scope because it is imported in this file 187

Wren Security Guice Core - org/forgerock/guice/core/internal/commons/lang/reflect/ConstructorUtils.java

Rule Violation Line
UselessParentheses Useless parentheses. 79

Wren Security Commons HTTP - JMH benchmarks - org/forgerock/http/serialization/JacksonJsonSerialization.java

Rule Violation Line
UnnecessaryImport Unused import 'org.openjdk.jmh.annotations.Fork' 33

Wren Security Commons HTTP - Apache Async HttpClient integration - org/forgerock/http/apache/async/CloseableBufferFactory.java

Rule Violation Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'java.nio': 'ByteBuffer' is already in scope because it is imported in this file 58

Wren Security Commons HTTP - Core Library - org/forgerock/http/io/BranchingStreamWrapper.java

Rule Violation Line
UselessParentheses Useless parentheses. 88

Wren Security Commons HTTP - Core Library - org/forgerock/http/io/ByteArrayBranchingStream.java

Rule Violation Line
UselessParentheses Useless parentheses. 61

Wren Security Commons HTTP - Core Library - org/forgerock/http/swagger/SwaggerUtils.java

Rule Violation Line
UnnecessaryImport Unused import 'java.util.Map' 21
UnnecessaryImport Unused import 'io.swagger.v3.oas.models.PathItem' 25
UnnecessaryImport Unused import 'io.swagger.v3.oas.models.media.Schema' 27
UnnecessaryImport Unused import 'io.swagger.v3.oas.models.parameters.Parameter' 28
UnnecessaryImport Unused import 'io.swagger.v3.oas.models.responses.ApiResponse' 29
UnnecessaryImport Unused import 'io.swagger.v3.oas.models.security.SecurityScheme' 30

Wren Security Commons HTTP - Core Library - org/forgerock/http/util/Json.java

Rule Violation Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'Json': 'PREFERRED_LOCALES_ATTRIBUTE' is already in scope because it is declared in an enclosing type 299

Wren Security Commons HTTP - Core Library - org/forgerock/http/util/Paths.java

Rule Violation Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'Paths': 'urlDecode' is already in scope 85

Wren Security I18N Core - org/forgerock/i18n/LocalizableMessage.java

Rule Violation Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'LocalizableMessage': 'raw' is already in scope 61

Wren Security I18N Maven Plugin - org/forgerock/i18n/maven/AbstractGenerateMessagesMojo.java

Rule Violation Line
UselessParentheses Useless parentheses. 396

Wren Security I18N Maven Plugin - org/forgerock/i18n/maven/CleanMessagesMojo.java

Rule Violation Line
UselessParentheses Useless parentheses. 347

Wren Security I18N SLF4J Support - org/forgerock/i18n/slf4j/LocalizedLogger.java

Rule Violation Line
UselessParentheses Useless parentheses. 4795

Wren Security Commons JSON Crypto - Core Library - org/forgerock/json/crypto/simple/SimpleDecryptor.java

Rule Violation Line
UselessParentheses Useless parentheses. 116

Wren Security Commons JSON Crypto - Core Library - org/forgerock/json/crypto/simple/SimpleEncryptor.java

Rule Violation Line
UselessParentheses Useless parentheses. 167

Wren Security Commons JSON Schema - Draft-04 Validator CLI - org/forgerock/json/schema/Main.java

Rule Violation Line
UselessParentheses Useless parentheses. 169
UselessParentheses Useless parentheses. 169

Wren Security Commons JSON Schema - Draft-04 Validator - org/forgerock/json/schema/validator/validators/ArrayTypeValidator.java

Rule Violation Line
UselessParentheses Useless parentheses. 160
UselessParentheses Useless parentheses. 163

Wren Security Commons JSON Schema - Draft-04 Validator - org/forgerock/json/schema/validator/validators/ObjectTypeValidator.java

Rule Violation Line
UselessParentheses Useless parentheses. 153

Wren Security Commons JWT - org/forgerock/json/jose/jwe/EncryptionMethod.java

Rule Violation Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'EncryptionMethod': 'values' is already in scope 154

Wren Security Commons JWT - org/forgerock/json/jose/jwe/JweAlgorithm.java

Rule Violation Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'JweAlgorithm': 'values' is already in scope 90

Wren Security Commons JWT - org/forgerock/json/jose/jws/JwsAlgorithm.java

Rule Violation Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'JwsAlgorithm': 'values' is already in scope 138

Wren Security Commons JWT - org/forgerock/json/jose/utils/DerUtils.java

Rule Violation Line
UselessParentheses Useless parentheses. 61

Wren Security Commons - API Descriptor - org/forgerock/api/jackson/CrestObjectSchema.java

Rule Violation Line
UnnecessaryImport Unused import 'com.fasterxml.jackson.module.jsonSchema.jakarta.types.ObjectSchema.AdditionalProperties' 26
UnnecessaryImport Unused import 'com.fasterxml.jackson.module.jsonSchema.jakarta.types.ObjectSchema.NoAdditionalProperties' 27
UnnecessaryImport Unused import 'com.fasterxml.jackson.module.jsonSchema.jakarta.types.ObjectSchema.SchemaAdditionalProperties' 28

Wren Security Commons REST - JSON Resource HTTP Binding - org/forgerock/json/resource/http/HttpUtils.java

Rule Violation Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'HttpUtils': 'METHOD_POST' is already in scope because it is declared in an enclosing type 637
UnnecessaryFullyQualifiedName Unnecessary qualifier 'HttpUtils': 'HEADER_X_HTTP_METHOD_OVERRIDE' is already in scope because it is declared in an enclosing type 638
UnnecessaryFullyQualifiedName Unnecessary qualifier 'HttpUtils': 'HEADER_X_HTTP_METHOD_OVERRIDE' is already in scope because it is declared in an enclosing type 639

Wren Security Commons REST - JSON Resource HTTP Binding - org/forgerock/json/resource/http/RequestRunner.java

Rule Violation Line
UselessParentheses Useless parentheses. 124–126

Wren Security Commons REST - JSON Resource Core Library - org/forgerock/json/resource/AdviceWarning.java

Rule Violation Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'AdviceWarning': 'newBuilder' is already in scope 110
UnnecessaryFullyQualifiedName Unnecessary qualifier 'AdviceWarning': 'newBuilder' is already in scope 126

Wren Security Commons Self Service - Core - org/forgerock/selfservice/core/config/ClassNameFallbackPropertyTypeResolver.java

Rule Violation Line
UnnecessaryImport Unused import 'com.fasterxml.jackson.databind.ObjectMapper' 23
UnnecessaryImport Unused import 'com.fasterxml.jackson.databind.json.JsonMapper' 25

Wren Security Commons Self Service - Core - org/forgerock/selfservice/core/util/Answers.java

Rule Violation Line
UselessParentheses Useless parentheses. 71

Wren Security Commons Utility Classes - Utility Library - org/forgerock/json/JsonPatch.java

Rule Violation Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'JsonPatch': 'VALUE_PTR' is already in scope because it is declared in an enclosing type 68
UnnecessaryFullyQualifiedName Unnecessary qualifier 'JsonPatch': 'VALUE_PTR' is already in scope because it is declared in an enclosing type 69

Wren Security Commons Utility Classes - Utility Library - org/forgerock/json/JsonValue.java

Rule Violation Line
UselessParentheses Useless parentheses. 193
UselessParentheses Useless parentheses. 379
UselessParentheses Useless parentheses. 379
UselessParentheses Useless parentheses. 392
UselessParentheses Useless parentheses. 405
UselessParentheses Useless parentheses. 540
UselessParentheses Useless parentheses. 553
UselessParentheses Useless parentheses. 553
UselessParentheses Useless parentheses. 639
UselessParentheses Useless parentheses. 639
UselessParentheses Useless parentheses. 651
UselessParentheses Useless parentheses. 651
UselessParentheses Useless parentheses. 746
UselessParentheses Useless parentheses. 859
UselessParentheses Useless parentheses. 888
UselessParentheses Useless parentheses. 897
UselessParentheses Useless parentheses. 906
UselessParentheses Useless parentheses. 915
UselessParentheses Useless parentheses. 933
UselessParentheses Useless parentheses. 942

Wren Security Commons Utility Classes - Utility Library - org/forgerock/util/Pair.java

Rule Violation Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'Pair': 'of' is already in scope 49

Wren Security Commons Utility Classes - Utility Library - org/forgerock/util/RangeSet.java

Rule Violation Line
UselessParentheses Useless parentheses. 90
UselessParentheses Useless parentheses. 91
UselessParentheses Useless parentheses. 93
UselessParentheses Useless parentheses. 101

Wren Security Commons Utility Classes - Utility Library - org/forgerock/util/query/QueryFilter.java

Rule Violation Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'QueryFilter': 'equalTo' is already in scope 556
UnnecessaryFullyQualifiedName Unnecessary qualifier 'QueryFilter': 'greaterThan' is already in scope 558
UnnecessaryFullyQualifiedName Unnecessary qualifier 'QueryFilter': 'greaterThanOrEqualTo' is already in scope 560
UnnecessaryFullyQualifiedName Unnecessary qualifier 'QueryFilter': 'lessThan' is already in scope 562
UnnecessaryFullyQualifiedName Unnecessary qualifier 'QueryFilter': 'lessThanOrEqualTo' is already in scope 564
UnnecessaryFullyQualifiedName Unnecessary qualifier 'QueryFilter': 'contains' is already in scope 566
UnnecessaryFullyQualifiedName Unnecessary qualifier 'QueryFilter': 'startsWith' is already in scope 568

Wren Security Commons Utility Classes - Utility Library - org/forgerock/util/time/Duration.java

Rule Violation Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'TimeUnit': 'DAYS' is already in scope because it is imported in this file 276

Files

Wren Security Commons Audit - Framework Core - org/forgerock/audit/AuditServiceImpl.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 16–543

Wren Security Commons Audit - Framework Core - org/forgerock/audit/events/AccessAuditEventBuilder.java

Rule Violation Priority Line
UnusedPrivateField Avoid unused private fields such as 'logger'. 3 121

Wren Security Commons Audit - Framework Core - org/forgerock/audit/events/EventTopicsMetaDataBuilder.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'final' on resource specification 'configStream': resource specifications are implicitly final 3 147

Wren Security Commons Audit - Framework Core - org/forgerock/audit/events/handlers/writers/RotatableWriter.java

Rule Violation Priority Line
CollapsibleIfStatements This if statement could be combined with its parent 3 151–153
CollapsibleIfStatements This if statement could be combined with its parent 3 225–227
UnnecessaryFullyQualifiedName Unnecessary qualifier 'TimeUnit': 'MILLISECONDS' is already in scope because it is imported in this file 4 374
UnnecessaryFullyQualifiedName Unnecessary qualifier 'TimeUnit': 'MILLISECONDS' is already in scope because it is imported in this file 4 375
UnnecessaryFullyQualifiedName Unnecessary qualifier 'TimeUnit': 'MILLISECONDS' is already in scope because it is imported in this file 4 376

Wren Security Commons Audit - Framework Core - org/forgerock/audit/events/handlers/writers/TextWriter.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'public' on class 'Stream': the class is declared in an interface type 3 59

Wren Security Commons Audit - Framework Core - org/forgerock/audit/filter/FilterChainBuilder.java

Rule Violation Priority Line
EmptyCatchBlock Avoid empty catch blocks 3 70–72

Wren Security Commons Audit - Framework Core - org/forgerock/audit/rotation/FixedTimeRotationPolicy.java

Rule Violation Priority Line
UnusedPrivateField Avoid unused private fields such as 'logger'. 3 31

Wren Security Commons Audit - Framework Core - org/forgerock/audit/secure/KeyStoreHandler.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'getStore': the method is declared in an interface type 3 31
UnnecessaryModifier Unnecessary modifier 'public' on method 'setStore': the method is declared in an interface type 3 41
UnnecessaryModifier Unnecessary modifier 'public' on method 'getPassword': the method is declared in an interface type 3 48
UnnecessaryModifier Unnecessary modifier 'public' on method 'getLocation': the method is declared in an interface type 3 55
UnnecessaryModifier Unnecessary modifier 'public' on method 'getType': the method is declared in an interface type 3 62
UnnecessaryModifier Unnecessary modifier 'public' on method 'store': the method is declared in an interface type 3 70

Wren Security Commons Audit - Framework Core - org/forgerock/audit/secure/KeyStoreSecureStorage.java

Rule Violation Priority Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'KeyStoreSecureStorage': 'SIGNATURE_ALGORITHM' is already in scope because it is declared in an enclosing type 4 95
UnnecessaryFullyQualifiedName Unnecessary qualifier 'KeyStoreSecureStorage': 'SIGNATURE_ALGORITHM' is already in scope because it is declared in an enclosing type 4 104
UnnecessaryFullyQualifiedName Unnecessary qualifier 'KeyStoreSecureStorage': 'ENTRY_CURRENT_KEY' is already in scope because it is declared in an enclosing type 4 127
UnnecessaryFullyQualifiedName Unnecessary qualifier 'KeyStoreSecureStorage': 'ENTRY_INITIAL_KEY' is already in scope because it is declared in an enclosing type 4 132
UnnecessaryFullyQualifiedName Unnecessary qualifier 'KeyStoreSecureStorage': 'ENTRY_CURRENT_SIGNATURE' is already in scope because it is declared in an enclosing type 4 137
UnnecessaryFullyQualifiedName Unnecessary qualifier 'KeyStoreSecureStorage': 'ENTRY_CURRENT_KEY' is already in scope because it is declared in an enclosing type 4 148
UnnecessaryFullyQualifiedName Unnecessary qualifier 'KeyStoreSecureStorage': 'ENTRY_INITIAL_KEY' is already in scope because it is declared in an enclosing type 4 154

Wren Security Commons Audit - Framework Core - org/forgerock/audit/util/DateUtil.java

Rule Violation Priority Line
EmptyCatchBlock Avoid empty catch blocks 3 186–188

Wren Security Commons Audit - Framework Core - org/forgerock/audit/util/JsonValueUtils.java

Rule Violation Priority Line
UnnecessaryReturn Unnecessary return statement 3 303

Wren Security Commons Audit - CSV Event Handler - org/forgerock/audit/handlers/csv/CsvAuditEventHandler.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 16–609
DoubleCheckedLocking Double checked locking is not thread safe in Java. 1 303
UselessParentheses Useless parentheses. 4 537

Wren Security Commons Audit - CSV Event Handler - org/forgerock/audit/handlers/csv/CsvSecureMapReader.java

Rule Violation Priority Line
UnusedPrivateField Avoid unused private fields such as 'logger'. 3 34

Wren Security Commons Audit - CSV Event Handler - org/forgerock/audit/handlers/csv/SecureCsvWriter.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 16–435
SimplifiedTernary This conditional expression can be simplified with || or && 3 302

Wren Security Commons Audit - CSV Event Handler - org/forgerock/audit/handlers/csv/StandardCsvWriter.java

Rule Violation Priority Line
UnusedPrivateField Avoid unused private fields such as 'logger'. 3 45
SimplifiedTernary This conditional expression can be simplified with || or && 3 116

Wren Security Commons Audit - Elasticsearch Event Handler - org/forgerock/audit/handlers/elasticsearch/ElasticsearchAuditEventHandler.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 16–528

Wren Security Commons Audit - Elasticsearch Event Handler - org/forgerock/audit/handlers/elasticsearch/ElasticsearchQueryFilterVisitor.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 16–280

Wren Security Commons Audit - JDBC Event Handler - org/forgerock/audit/handlers/jdbc/BufferedJdbcAuditEventExecutor.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'final' on resource specification 'preparedStatement': resource specifications are implicitly final 3 192

Wren Security Commons Audit - JDBC Event Handler - org/forgerock/audit/handlers/jdbc/JdbcAuditEventExecutorImpl.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'final' on resource specification 'preparedStatement': resource specifications are implicitly final 3 57

Wren Security Commons Audit - JDBC Event Handler - org/forgerock/audit/handlers/jdbc/StringSqlQueryFilterVisitor.java

Rule Violation Priority Line
UselessQualifiedThis Useless qualified this usage in the same class. 3 68

Wren Security Commons Audit - JMS Event Handler - org/forgerock/audit/handlers/jms/JmsAuditEventHandler.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17–304
EmptyCatchBlock Avoid empty catch blocks 3 257–259

Wren Security Commons Audit - JSON Event Handler - org/forgerock/audit/handlers/json/JsonAuditEventHandler.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17–226
UnnecessaryModifier Unnecessary modifier 'final' on resource specification 'reader': resource specifications are implicitly final 3 136–137
UnnecessaryModifier Unnecessary modifier 'final' on resource specification 'reader': resource specifications are implicitly final 3 163–164

Wren Security Commons Audit - JSON Event Handler - org/forgerock/audit/handlers/json/JsonFileWriter.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 18–534
UnnecessaryFullyQualifiedName Unnecessary qualifier 'Math': 'max' is already in scope 4 101

Wren Security Commons Audit - Splunk Event Handler - org/forgerock/audit/handlers/splunk/SplunkAuditEventHandler.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 16–249

Wren Security Commons Audit - Syslog Event Handler - org/forgerock/audit/handlers/syslog/SyslogFormatter.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17–326

Wren Security Commons Audit - JSON Support Library - org/forgerock/audit/json/AuditJsonConfig.java

Rule Violation Priority Line
UnusedPrivateField Avoid unused private fields such as 'EVENTS_FIELD'. 3 57

Wren Security Commons Audit - Servlet Example - org/forgerock/audit/AuditHttpApplication.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'final' on resource specification 'eventHandlersConfig': resource specifications are implicitly final 3 70

Wren Security Commons Authn. - JASPI (JSR-196) Functional Tests - org/forgerock/caf/authn/test/configuration/AuditResource.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17–130

Wren Security Commons Authn. - JASPI (JSR-196) IWA Module - org/forgerock/jaspi/modules/iwa/wdsso/Base64.java

Rule Violation Priority Line
UselessParentheses Useless parentheses. 4 108
UselessParentheses Useless parentheses. 4 247
UselessParentheses Useless parentheses. 4 295
UselessParentheses Useless parentheses. 4 433
UselessParentheses Useless parentheses. 4 594

Wren Security Commons Authn. - JASPI (JSR-196) IWA Module - org/forgerock/jaspi/modules/iwa/wdsso/DerValue.java

Rule Violation Priority Line
UselessParentheses Useless parentheses. 4 119

Wren Security Commons Authn. - JASPI (JSR-196) IWA Module - org/forgerock/jaspi/modules/iwa/wdsso/WDSSO.java

Rule Violation Priority Line
UselessParentheses Useless parentheses. 4 368
CollapsibleIfStatements This if statement could be combined with its parent 3 370–373
UselessParentheses Useless parentheses. 4 370

Wren Security Commons Authn. - JASPI (JSR-196) JWT Session Module - org/forgerock/jaspi/modules/session/jwt/AbstractJwtSessionModule.java

Rule Violation Priority Line
SimplifiedTernary This conditional expression can be simplified with || or && 3 188
SimplifiedTernary This conditional expression can be simplified with || or && 3 190
SimplifiedTernary This conditional expression can be simplified with || or && 3 192

Wren Security Commons Authn. - JASPI (JSR-196) JWT Session Module - org/forgerock/jaspi/modules/session/jwt/JwtSessionModule.java

Rule Violation Priority Line
AvoidBranchingStatementAsLastInLoop Avoid using a branching statement as the last in a loop. 2 106
SimplifiedTernary This conditional expression can be simplified with || or && 3 175–177

Wren Security Commons Authn. - JASPI (JSR-196) OpenAM Session Module - org/forgerock/jaspi/modules/session/openam/OpenAMSessionModule.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17–462
UnusedPrivateField Avoid unused private fields such as 'MIME_TYPE_APPLICATION_JSON'. 3 82
UnusedPrivateField Avoid unused private fields such as 'CHARACTER_ENCODING'. 3 83
AvoidBranchingStatementAsLastInLoop Avoid using a branching statement as the last in a loop. 2 424

Wren Security Commons Authn. - JASPI (JSR-196) OIDC Module - org/forgerock/jaspi/modules/openid/OpenIdConnectModule.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17–272
UnnecessaryImport Unused import 'jakarta.security.auth.message.AuthException' 4 29
UnnecessaryFullyQualifiedName Unnecessary qualifier 'OpenIdConnectModule': 'HEADER_KEY' is already in scope because it is declared in an enclosing type 4 140
UnnecessaryFullyQualifiedName Unnecessary qualifier 'OpenIdConnectModule': 'READ_TIMEOUT_KEY' is already in scope because it is declared in an enclosing type 4 143
UnnecessaryFullyQualifiedName Unnecessary qualifier 'OpenIdConnectModule': 'CONNECTION_TIMEOUT_KEY' is already in scope because it is declared in an enclosing type 4 144
UnnecessaryFullyQualifiedName Unnecessary qualifier 'OpenIdConnectModule': 'RESOLVERS_KEY' is already in scope because it is declared in an enclosing type 4 162

Wren Security Commons Authn. - JASPI (JSR-196) OIDC Module - org/forgerock/jaspi/modules/openid/helpers/SimpleHTTPClient.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'final' on resource specification 'reader': resource specifications are implicitly final 3 91

Wren Security Commons Authn. - JASPI (JSR-196) OIDC Module - org/forgerock/jaspi/modules/openid/resolvers/BaseOpenIdResolver.java

Rule Violation Priority Line
UselessParentheses Useless parentheses. 4 104

Wren Security Commons Authn. - JASPI (JSR-196) OIDC Module - org/forgerock/jaspi/modules/openid/resolvers/OpenIdResolver.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifiers 'public static final' on field 'KEY_ALIAS_KEY': the field is declared in an interface type 3 41
UnnecessaryModifier Unnecessary modifiers 'public static final' on field 'ISSUER_KEY': the field is declared in an interface type 3 46
UnnecessaryModifier Unnecessary modifiers 'public static final' on field 'CLIENT_SECRET_KEY': the field is declared in an interface type 3 51
UnnecessaryModifier Unnecessary modifiers 'public static final' on field 'JWK': the field is declared in an interface type 3 56
UnnecessaryModifier Unnecessary modifiers 'public static final' on field 'WELL_KNOWN_CONFIGURATION': the field is declared in an interface type 3 61
UnnecessaryModifier Unnecessary modifiers 'public static final' on field 'KEYSTORE_LOCATION_KEY': the field is declared in an interface type 3 66
UnnecessaryModifier Unnecessary modifiers 'public static final' on field 'KEYSTORE_TYPE_KEY': the field is declared in an interface type 3 71
UnnecessaryModifier Unnecessary modifiers 'public static final' on field 'KEYSTORE_PASS_KEY': the field is declared in an interface type 3 76
UnnecessaryModifier Unnecessary modifier 'public' on method 'validateIdentity': the method is declared in an interface type 3 84
UnnecessaryModifier Unnecessary modifier 'public' on method 'getIssuer': the method is declared in an interface type 3 91

Wren Security Commons Authn. - JASPI (JSR-196) OIDC Module - org/forgerock/jaspi/modules/openid/resolvers/service/OpenIdResolverService.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'getResolverForIssuer': the method is declared in an interface type 3 53
UnnecessaryModifier Unnecessary modifier 'public' on method 'configureResolverWithKey': the method is declared in an interface type 3 66
UnnecessaryModifier Unnecessary modifier 'public' on method 'configureResolverWithSecret': the method is declared in an interface type 3 78
UnnecessaryModifier Unnecessary modifier 'public' on method 'configureResolverWithJWK': the method is declared in an interface type 3 90
UnnecessaryModifier Unnecessary modifier 'public' on method 'configureResolverWithWellKnownOpenIdConfiguration': the method is declared in an interface type 3 101

Wren Security Commons Authn. - JASPI (JSR-196) OIDC Module - org/forgerock/jaspi/modules/openid/resolvers/service/OpenIdResolverServiceConfigurator.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'configureService': the method is declared in an interface type 3 40

Wren Security Commons Authn. - JASPI (JSR-196) OIDC Module - org/forgerock/jaspi/modules/openid/resolvers/service/OpenIdResolverServiceConfiguratorImpl.java

Rule Violation Priority Line
UselessParentheses Useless parentheses. 4 116
UselessParentheses Useless parentheses. 4 117
UselessParentheses Useless parentheses. 4 118

Wren Security Commons Authn. - JASPI (JSR-196) OIDC Module - org/forgerock/jaspi/modules/openid/resolvers/service/OpenIdResolverServiceImpl.java

Rule Violation Priority Line
UnusedPrivateField Avoid unused private fields such as 'readTimeout'. 3 50
UnusedPrivateField Avoid unused private fields such as 'connTimeout'. 3 51

Wren Security Commons Authn. - JASPI (JSR-196) Runtime - org/forgerock/caf/authentication/framework/AggregateAuthContext.java

Rule Violation Priority Line
UnusedPrivateField Avoid unused private fields such as 'logger'. 3 56

Wren Security Commons Authn. - JASPI (JSR-196) Runtime - org/forgerock/caf/authentication/framework/AuthenticationFramework.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 18–275
UnnecessaryFullyQualifiedName Unnecessary qualifier 'AuthenticationFramework': 'ATTRIBUTE_AUTH_CONTEXT' is already in scope because it is declared in an enclosing type 4 139
UnnecessaryFullyQualifiedName Unnecessary qualifier 'AuthenticationFramework': 'ATTRIBUTE_AUTH_CONTEXT' is already in scope because it is declared in an enclosing type 4 168
UnnecessaryFullyQualifiedName Unnecessary qualifier 'AuthenticationFramework': 'ATTRIBUTE_AUTH_PRINCIPAL' is already in scope because it is declared in an enclosing type 4 172
UnnecessaryFullyQualifiedName Unnecessary qualifier 'AuthenticationFramework': 'ATTRIBUTE_AUTH_CONTEXT' is already in scope because it is declared in an enclosing type 4 173
UnnecessaryFullyQualifiedName Unnecessary qualifier 'AuthenticationFramework': 'ATTRIBUTE_REQUEST_ID' is already in scope because it is declared in an enclosing type 4 188

Wren Security Commons Authn. - JASPI (JSR-196) Runtime - org/forgerock/caf/authentication/framework/SessionAuthContext.java

Rule Violation Priority Line
UnusedPrivateField Avoid unused private fields such as 'logger'. 3 42

Wren Security Commons Authz. - OAuth2 Filter Module - org/forgerock/authz/modules/oauth2/OAuth2CrestAuthorizationModule.java

Rule Violation Priority Line
UnusedFormalParameter Avoid unused method parameters such as 'securityContext'. 3 212

Wren Security Commons Bloom Filters - Core - org/forgerock/bloomfilter/AtomicBloomFilter.java

Rule Violation Priority Line
UselessParentheses Useless parentheses. 4 115

Wren Security Commons Bloom Filters - Core - org/forgerock/bloomfilter/BatchingBloomFilter.java

Rule Violation Priority Line
CollapsibleIfStatements This if statement could be combined with its parent 3 80–94

Wren Security Commons Bloom Filters - Core - org/forgerock/bloomfilter/BloomFilterChain.java

Rule Violation Priority Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'Math': 'max' is already in scope 4 146

Wren Security Commons Bloom Filters - Core - org/forgerock/bloomfilter/BloomFilterStatistics.java

Rule Violation Priority Line
UselessParentheses Useless parentheses. 4 106
UselessParentheses Useless parentheses. 4 138

Wren Security Commons Bloom Filters - Core - org/forgerock/bloomfilter/BloomFilters.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'static' on enum 'NeverExpires': nested enums are implicitly static 3 49

Wren Security Commons Bloom Filters - Monitoring - org/forgerock/bloomfilter/monitoring/BloomFilterMonitor.java

Rule Violation Priority Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'TimeUnit': 'SECONDS' is already in scope because it is imported in this file 4 187
CollapsibleIfStatements This if statement could be combined with its parent 3 300–305

Wren Security Guice Core - org/forgerock/guice/core/InjectorConfiguration.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'private' on constructor 'InjectorConfiguration()': enum constructors are implicitly private 3 62

Wren Security Guice Core - org/forgerock/guice/core/InjectorHolder.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'private' on constructor 'InjectorHolder()': enum constructors are implicitly private 3 53

Wren Security Guice Core - org/forgerock/guice/core/internal/commons/lang/reflect/ConstructorUtils.java

Rule Violation Priority Line
UselessParentheses Useless parentheses. 4 79

Wren Security Guice Core - org/forgerock/guice/core/internal/commons/lang/reflect/MemberUtils.java

Rule Violation Priority Line
EmptyCatchBlock Avoid empty catch blocks 3 52–54
EmptyCatchBlock Avoid empty catch blocks 3 85–87

Wren Security Commons HTTP - CHF Bindings Test Utilities - org/forgerock/http/bindings/BindingTest.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 16–391
UnnecessaryModifier Unnecessary modifier 'final' on resource specification 'handler': resource specifications are implicitly final 3 171
UnnecessaryModifier Unnecessary modifier 'final' on resource specification 'handler': resource specifications are implicitly final 3 191
UnnecessaryModifier Unnecessary modifier 'final' on resource specification 'handler': resource specifications are implicitly final 3 218
UnnecessaryModifier Unnecessary modifier 'final' on resource specification 'handler': resource specifications are implicitly final 3 243
UnnecessaryModifier Unnecessary modifier 'final' on resource specification 'httpClientHandler': resource specifications are implicitly final 3 287

Wren Security Commons HTTP - JMH benchmarks - org/forgerock/http/serialization/JacksonJsonSerialization.java

Rule Violation Priority Line
UnnecessaryImport Unused import 'org.openjdk.jmh.annotations.Fork' 4 33

Wren Security Commons HTTP - Apache Async HttpClient integration - org/forgerock/http/apache/async/AsyncHttpClientProvider.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17–194

Wren Security Commons HTTP - Apache Async HttpClient integration - org/forgerock/http/apache/async/CloseableBufferFactory.java

Rule Violation Priority Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'java.nio': 'ByteBuffer' is already in scope because it is imported in this file 4 58

Wren Security Commons HTTP - Apache HttpClient integration - org/forgerock/http/apache/sync/SyncHttpClientProvider.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 19–126

Wren Security Commons HTTP - Client Test Utilities - org/forgerock/http/handler/HttpClientHandlerTest.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17–163

Wren Security Commons HTTP - Core Library - org/forgerock/http/header/ContentLengthHeader.java

Rule Violation Priority Line
EmptyCatchBlock Avoid empty catch blocks 3 59–61

Wren Security Commons HTTP - Core Library - org/forgerock/http/header/HeaderUtil.java

Rule Violation Priority Line
EmptyControlStatement Empty if statement 3 218–222

Wren Security Commons HTTP - Core Library - org/forgerock/http/io/BranchingStreamWrapper.java

Rule Violation Priority Line
UselessParentheses Useless parentheses. 4 88
EmptyCatchBlock Avoid empty catch blocks 3 199–201

Wren Security Commons HTTP - Core Library - org/forgerock/http/io/ByteArrayBranchingStream.java

Rule Violation Priority Line
UselessParentheses Useless parentheses. 4 61

Wren Security Commons HTTP - Core Library - org/forgerock/http/io/FileBuffer.java

Rule Violation Priority Line
EmptyCatchBlock Avoid empty catch blocks 3 122–124

Wren Security Commons HTTP - Core Library - org/forgerock/http/protocol/Cookie.java

Rule Violation Priority Line
SimplifiedTernary This conditional expression can be simplified with || or && 3 172
SimplifiedTernary This conditional expression can be simplified with || or && 3 221

Wren Security Commons HTTP - Core Library - org/forgerock/http/protocol/Entity.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17–488

Wren Security Commons HTTP - Core Library - org/forgerock/http/protocol/Request.java

Rule Violation Priority Line
EmptyCatchBlock Avoid empty catch blocks 3 84–86

Wren Security Commons HTTP - Core Library - org/forgerock/http/protocol/Response.java

Rule Violation Priority Line
UnnecessarySemicolon Unnecessary semicolon 3 148

Wren Security Commons HTTP - Core Library - org/forgerock/http/routing/RouteMatchers.java

Rule Violation Priority Line
EmptyCatchBlock Avoid empty catch blocks 3 286–288

Wren Security Commons HTTP - Core Library - org/forgerock/http/routing/UriRouteMatcher.java

Rule Violation Priority Line
EmptyControlStatement Empty else statement 3 217–219

Wren Security Commons HTTP - Core Library - org/forgerock/http/spi/Loader.java

Rule Violation Priority Line
UnnecessarySemicolon Unnecessary semicolon 3 41

Wren Security Commons HTTP - Core Library - org/forgerock/http/swagger/SwaggerUtils.java

Rule Violation Priority Line
UnnecessaryImport Unused import 'java.util.Map' 4 21
UnnecessaryImport Unused import 'io.swagger.v3.oas.models.PathItem' 4 25
UnnecessaryImport Unused import 'io.swagger.v3.oas.models.media.Schema' 4 27
UnnecessaryImport Unused import 'io.swagger.v3.oas.models.parameters.Parameter' 4 28
UnnecessaryImport Unused import 'io.swagger.v3.oas.models.responses.ApiResponse' 4 29
UnnecessaryImport Unused import 'io.swagger.v3.oas.models.security.SecurityScheme' 4 30

Wren Security Commons HTTP - Core Library - org/forgerock/http/util/Json.java

Rule Violation Priority Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'Json': 'PREFERRED_LOCALES_ATTRIBUTE' is already in scope because it is declared in an enclosing type 4 299

Wren Security Commons HTTP - Core Library - org/forgerock/http/util/Paths.java

Rule Violation Priority Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'Paths': 'urlDecode' is already in scope 4 85

Wren Security Commons HTTP - Core Library - org/forgerock/http/util/Uris.java

Rule Violation Priority Line
JumbledIncrementer Avoid modifying an outer loop incrementer in an inner loop for update expression 3 411–428

Wren Security Commons HTTP - Core Library - org/forgerock/services/routing/AbstractRouter.java

Rule Violation Priority Line
CollapsibleIfStatements This if statement could be combined with its parent 3 269–271

Wren Security Commons HTTP - OpenAPI Descriptor Example Module - org/forgerock/http/example/DescribedOauth2Endpoint.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17–233

Wren Security Commons HTTP - Grizzly Integration Library - org/forgerock/http/grizzly/HandlerAdapter.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17–221

Wren Security Commons HTTP - OAuth 2.0 Resource Server Support - org/forgerock/http/oauth2/resolver/OpenAmAccessTokenResolver.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17–226

Wren Security Commons HTTP - Servlet integration library - org/forgerock/http/servlet/HttpFrameworkServlet.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17–374

Wren Security I18N Core - org/forgerock/i18n/LocalizableMessage.java

Rule Violation Priority Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'LocalizableMessage': 'raw' is already in scope 4 61

Wren Security I18N Maven Plugin - org/forgerock/i18n/maven/AbstractGenerateMessagesMojo.java

Rule Violation Priority Line
EmptyControlStatement Empty if statement 3 223–225
UselessParentheses Useless parentheses. 4 396
CollapsibleIfStatements This if statement could be combined with its parent 3 648–652
EmptyCatchBlock Avoid empty catch blocks 3 748–750
EmptyCatchBlock Avoid empty catch blocks 3 756–758

Wren Security I18N Maven Plugin - org/forgerock/i18n/maven/CleanMessagesMojo.java

Rule Violation Priority Line
UselessParentheses Useless parentheses. 4 347
CollapsibleIfStatements This if statement could be combined with its parent 3 358–361

Wren Security I18N SLF4J Support - org/forgerock/i18n/slf4j/LocalizedLogger.java

Rule Violation Priority Line
UselessParentheses Useless parentheses. 4 4795

Wren Security Commons JSON Crypto - Core Library - org/forgerock/json/crypto/JsonCrypto.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17–159

Wren Security Commons JSON Crypto - Core Library - org/forgerock/json/crypto/simple/SimpleDecryptor.java

Rule Violation Priority Line
UselessParentheses Useless parentheses. 4 116

Wren Security Commons JSON Crypto - Core Library - org/forgerock/json/crypto/simple/SimpleEncryptor.java

Rule Violation Priority Line
UselessParentheses Useless parentheses. 4 167

Wren Security Commons JSON Schema - Draft-04 Validator CLI - org/forgerock/json/schema/Main.java

Rule Violation Priority Line
UselessParentheses Useless parentheses. 4 169
UselessParentheses Useless parentheses. 4 169

Wren Security Commons JSON Schema - Draft-04 Validator - org/forgerock/json/schema/validator/helpers/FormatHelper.java

Rule Violation Priority Line
UnusedFormalParameter Avoid unused constructor parameters such as 'format'. 3 40

Wren Security Commons JSON Schema - Draft-04 Validator - org/forgerock/json/schema/validator/helpers/MaximumHelper.java

Rule Violation Priority Line
CollapsibleIfStatements This if statement could be combined with its parent 3 84–86

Wren Security Commons JSON Schema - Draft-04 Validator - org/forgerock/json/schema/validator/helpers/MinimumHelper.java

Rule Violation Priority Line
CollapsibleIfStatements This if statement could be combined with its parent 3 85–87

Wren Security Commons JSON Schema - Draft-04 Validator - org/forgerock/json/schema/validator/validators/ArrayTypeValidator.java

Rule Violation Priority Line
UselessParentheses Useless parentheses. 4 160
UselessParentheses Useless parentheses. 4 163

Wren Security Commons JSON Schema - Draft-04 Validator - org/forgerock/json/schema/validator/validators/BooleanTypeValidator.java

Rule Violation Priority Line
UnnecessaryReturn Unnecessary return statement 3 58

Wren Security Commons JSON Schema - Draft-04 Validator - org/forgerock/json/schema/validator/validators/IntegerTypeValidator.java

Rule Violation Priority Line
CollapsibleIfStatements This if statement could be combined with its parent 3 116–118

Wren Security Commons JSON Schema - Draft-04 Validator - org/forgerock/json/schema/validator/validators/NumberTypeValidator.java

Rule Violation Priority Line
CollapsibleIfStatements This if statement could be combined with its parent 3 117–119

Wren Security Commons JSON Schema - Draft-04 Validator - org/forgerock/json/schema/validator/validators/ObjectTypeValidator.java

Rule Violation Priority Line
UselessParentheses Useless parentheses. 4 153
EmptyCatchBlock Avoid empty catch blocks 3 173–176
EmptyControlStatement Empty if statement 3 195–203
CollapsibleIfStatements This if statement could be combined with its parent 3 229–240

Wren Security Commons JSON Schema - Draft-04 Validator - org/forgerock/json/schema/validator/validators/SimpleValidator.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'validate': the method is declared in an interface type 3 41

Wren Security Commons JSON Schema - Draft-04 Validator - org/forgerock/json/schema/validator/validators/StringTypeValidator.java

Rule Violation Priority Line
EmptyCatchBlock Avoid empty catch blocks 3 110–112
CollapsibleIfStatements This if statement could be combined with its parent 3 133–135

Wren Security Commons JSON Schema - Draft-04 Validator - org/forgerock/json/schema/validator/validators/UnionTypeValidator.java

Rule Violation Priority Line
EmptyCatchBlock Avoid empty catch blocks 3 83–85

Wren Security Commons JWT - org/forgerock/json/jose/jwe/EncryptionMethod.java

Rule Violation Priority Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'EncryptionMethod': 'values' is already in scope 4 154

Wren Security Commons JWT - org/forgerock/json/jose/jwe/JweAlgorithm.java

Rule Violation Priority Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'JweAlgorithm': 'values' is already in scope 4 90

Wren Security Commons JWT - org/forgerock/json/jose/jwe/JweAlgorithmType.java

Rule Violation Priority Line
UselessOverridingMethod Overriding method merely calls super 3 40

Wren Security Commons JWT - org/forgerock/json/jose/jwe/handlers/encryption/AbstractRSAESPkcs1V15AesCbcHmacEncryptionHandler.java

Rule Violation Priority Line
UnusedFormalParameter Avoid unused constructor parameters such as 'signingManager'. 3 46

Wren Security Commons JWT - org/forgerock/json/jose/jwe/handlers/encryption/RSAEncryptionHandler.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17–164

Wren Security Commons JWT - org/forgerock/json/jose/jwk/KeyType.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'private' on constructor 'KeyType(String)': enum constructors are implicitly private 3 49

Wren Security Commons JWT - org/forgerock/json/jose/jwk/KeyUse.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'private' on constructor 'KeyUse()': enum constructors are implicitly private 3 38

Wren Security Commons JWT - org/forgerock/json/jose/jws/JwsAlgorithm.java

Rule Violation Priority Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'JwsAlgorithm': 'values' is already in scope 4 138
UselessOverridingMethod Overriding method merely calls super 3 152

Wren Security Commons JWT - org/forgerock/json/jose/jws/JwtSecureHeader.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17–409

Wren Security Commons JWT - org/forgerock/json/jose/utils/DerUtils.java

Rule Violation Priority Line
CollapsibleIfStatements This if statement could be combined with its parent 3 57–59
UselessParentheses Useless parentheses. 4 61

Wren Security Commons - API Descriptor - org/forgerock/api/CrestApiProducer.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17–158

Wren Security Commons - API Descriptor - org/forgerock/api/commons/CommonsApi.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'private' on constructor 'Errors(int, String)': enum constructors are implicitly private 3 176

Wren Security Commons - API Descriptor - org/forgerock/api/jackson/CrestObjectSchema.java

Rule Violation Priority Line
UnnecessaryImport Unused import 'com.fasterxml.jackson.module.jsonSchema.jakarta.types.ObjectSchema.AdditionalProperties' 4 26
UnnecessaryImport Unused import 'com.fasterxml.jackson.module.jsonSchema.jakarta.types.ObjectSchema.NoAdditionalProperties' 4 27
UnnecessaryImport Unused import 'com.fasterxml.jackson.module.jsonSchema.jakarta.types.ObjectSchema.SchemaAdditionalProperties' 4 28

Wren Security Commons - API Descriptor - org/forgerock/api/jackson/CrestPropertyDetailsSchemaFactoryWrapper.java

Rule Violation Priority Line
UselessOverridingMethod Overriding method merely calls super 3 137

Wren Security Commons - API Descriptor - org/forgerock/api/jackson/CrestStringSchema.java

Rule Violation Priority Line
UselessOperationOnImmutable The result of an operation on an immutable object is ignored 3 139
UselessOperationOnImmutable The result of an operation on an immutable object is ignored 3 149
UnnecessaryReturn Unnecessary return statement 3 187

Wren Security Commons - API Descriptor - org/forgerock/api/markup/ApiDocGenerator.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 18–1130

Wren Security Commons - API Descriptor - org/forgerock/api/markup/asciidoc/AsciiDoc.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17–623
UnusedLocalVariable Avoid unused local variables such as 'symbol'. 3 283

Wren Security Commons - API Descriptor - org/forgerock/api/models/Action.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'final' on method 'action': the method is already in a final class 3 117

Wren Security Commons - API Descriptor - org/forgerock/api/models/ApiDescription.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 16–340

Wren Security Commons - API Descriptor - org/forgerock/api/models/Create.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'final' on method 'create': the method is already in a final class 3 77

Wren Security Commons - API Descriptor - org/forgerock/api/models/Delete.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'final' on method 'delete': the method is already in a final class 3 41

Wren Security Commons - API Descriptor - org/forgerock/api/models/Patch.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'final' on method 'patch': the method is already in a final class 3 85

Wren Security Commons - API Descriptor - org/forgerock/api/models/Read.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'final' on method 'read': the method is already in a final class 3 41

Wren Security Commons - API Descriptor - org/forgerock/api/models/Update.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'final' on method 'update': the method is already in a final class 3 41

Wren Security Commons - API Descriptor - org/forgerock/api/transform/OpenApiTransformer.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17–1624

Wren Security Commons - API Descriptor - org/forgerock/api/util/ValidationUtil.java

Rule Violation Priority Line
SimplifiedTernary This conditional expression can be simplified with || or && 3 91

Wren Security Commons REST - JSON Resource Examples - org/forgerock/json/resource/examples/DynamicRealmDemo.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17–212

Wren Security Commons REST - JSON Resource Examples - org/forgerock/json/resource/examples/ReadModifyWriteDemo.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'final' on resource specification 'server': resource specifications are implicitly final 3 47
UnnecessaryModifier Unnecessary modifier 'final' on resource specification 'connection': resource specifications are implicitly final 3 48

Wren Security Commons REST - JSON Resource Examples - org/forgerock/json/resource/examples/VersionedResourcesDemo.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17–203

Wren Security Commons REST - JSON Resource Examples - org/forgerock/json/resource/http/examples/CrestHttpApplication.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17–181

Wren Security Commons REST - JSON Resource HTTP Binding - org/forgerock/json/resource/http/CrestAdapter.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17–478

Wren Security Commons REST - JSON Resource HTTP Binding - org/forgerock/json/resource/http/HttpAdapter.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17–900
EmptyControlStatement Empty if statement 3 352–356
EmptyControlStatement Empty if statement 3 520–524
EmptyControlStatement Empty if statement 3 590–594
CollapsibleIfStatements This if statement could be combined with its parent 3 656–658

Wren Security Commons REST - JSON Resource HTTP Binding - org/forgerock/json/resource/http/HttpUtils.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17–951
EmptyCatchBlock Avoid empty catch blocks 3 586–589
UnnecessaryFullyQualifiedName Unnecessary qualifier 'HttpUtils': 'METHOD_POST' is already in scope because it is declared in an enclosing type 4 637
UnnecessaryFullyQualifiedName Unnecessary qualifier 'HttpUtils': 'HEADER_X_HTTP_METHOD_OVERRIDE' is already in scope because it is declared in an enclosing type 4 638
UnnecessaryFullyQualifiedName Unnecessary qualifier 'HttpUtils': 'HEADER_X_HTTP_METHOD_OVERRIDE' is already in scope because it is declared in an enclosing type 4 639

Wren Security Commons REST - JSON Resource HTTP Binding - org/forgerock/json/resource/http/RequestRunner.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17–557
UselessParentheses Useless parentheses. 4 124–126
UnnecessaryModifier Unnecessary modifier 'final' on method 'handleError': the method is already in a final class 3 129
UnnecessaryModifier Unnecessary modifier 'final' on method 'handleResult': the method is already in a final class 3 136
UnnecessaryModifier Unnecessary modifier 'final' on method 'visitActionRequest': the method is already in a final class 3 147
UnnecessaryModifier Unnecessary modifier 'final' on method 'visitCreateRequest': the method is already in a final class 3 183
UnnecessaryModifier Unnecessary modifier 'final' on method 'visitDeleteRequest': the method is already in a final class 3 232
UnnecessaryModifier Unnecessary modifier 'final' on method 'visitPatchRequest': the method is already in a final class 3 247
UnnecessaryModifier Unnecessary modifier 'final' on method 'visitQueryRequest': the method is already in a final class 3 262
UnnecessaryModifier Unnecessary modifier 'final' on method 'visitReadRequest': the method is already in a final class 3 344
UnnecessaryModifier Unnecessary modifier 'final' on method 'visitUpdateRequest': the method is already in a final class 3 359
UnusedFormalParameter Avoid unused method parameters such as 'e'. 3 374

Wren Security Commons REST - JSON Resource Core Library - org/forgerock/json/resource/AdviceWarning.java

Rule Violation Priority Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'AdviceWarning': 'newBuilder' is already in scope 4 110
UnnecessaryFullyQualifiedName Unnecessary qualifier 'AdviceWarning': 'newBuilder' is already in scope 4 126

Wren Security Commons REST - JSON Resource Core Library - org/forgerock/json/resource/DescribableResourceHandler.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'final' on method 'api': the method is already in a final class 3 63

Wren Security Commons REST - JSON Resource Core Library - org/forgerock/json/resource/MemoryBackend.java

Rule Violation Priority Line
EmptyControlStatement Empty else statement 3 454–456

Wren Security Commons REST - JSON Resource Core Library - org/forgerock/json/resource/Requests.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'final' on method 'addSortKey': the method is already in a final class 3 460

Wren Security Commons REST - JSON Resource Core Library - org/forgerock/json/resource/Resources.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17–556

Wren Security Commons REST - JSON Resource Core Library - org/forgerock/json/resource/Router.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17–453

Forgerock Commons - Security - org/forgerock/security/keystore/KeystoreManagerException.java

Rule Violation Priority Line
UnusedFormalParameter Avoid unused constructor parameters such as 'cause'. 3 48

Wren Security Commons Self Service - Core - org/forgerock/selfservice/core/AnonymousProcessService.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17–313

Wren Security Commons Self Service - Core - org/forgerock/selfservice/core/ProcessContextImpl.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17–238

Wren Security Commons Self Service - Core - org/forgerock/selfservice/core/UserUpdateService.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 16–169

Wren Security Commons Self Service - Core - org/forgerock/selfservice/core/config/ClassNameFallbackPropertyTypeResolver.java

Rule Violation Priority Line
UnnecessaryImport Unused import 'com.fasterxml.jackson.databind.ObjectMapper' 4 23
UnnecessaryImport Unused import 'com.fasterxml.jackson.databind.json.JsonMapper' 4 25

Wren Security Commons Self Service - Core - org/forgerock/selfservice/core/crypto/FieldStorageScheme.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'hashField': the method is declared in an interface type 3 31
UnnecessaryModifier Unnecessary modifier 'public' on method 'fieldMatches': the method is declared in an interface type 3 41

Wren Security Commons Self Service - Core - org/forgerock/selfservice/core/util/Answers.java

Rule Violation Priority Line
UselessParentheses Useless parentheses. 4 71

Wren Security Commons Self Service - Core - org/forgerock/selfservice/core/util/RequirementsBuilder.java

Rule Violation Priority Line
CollapsibleIfStatements This if statement could be combined with its parent 3 229–231

Wren Security Commons Self Service - Examples - org/forgerock/selfservice/example/ExampleEmailService.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 16–160

Wren Security Commons Self Service - Stages - org/forgerock/selfservice/stages/email/VerifyEmailAccountStage.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17–195

Wren Security Commons Self Service - Stages - org/forgerock/selfservice/stages/kba/SecurityAnswerDefinitionStage.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17–146

Wren Security Commons Self Service - Stages - org/forgerock/selfservice/stages/user/EmailUsernameStage.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 16–116

Wren Security Commons Utility Classes - Utility Library - org/forgerock/json/JsonPatch.java

Rule Violation Priority Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'JsonPatch': 'VALUE_PTR' is already in scope because it is declared in an enclosing type 4 68
UnnecessaryFullyQualifiedName Unnecessary qualifier 'JsonPatch': 'VALUE_PTR' is already in scope because it is declared in an enclosing type 4 69

Wren Security Commons Utility Classes - Utility Library - org/forgerock/json/JsonValue.java

Rule Violation Priority Line
UselessParentheses Useless parentheses. 4 193
UselessParentheses Useless parentheses. 4 379
UselessParentheses Useless parentheses. 4 379
UselessParentheses Useless parentheses. 4 392
UselessParentheses Useless parentheses. 4 405
UselessParentheses Useless parentheses. 4 540
UselessParentheses Useless parentheses. 4 553
UselessParentheses Useless parentheses. 4 553
UselessParentheses Useless parentheses. 4 639
UselessParentheses Useless parentheses. 4 639
UselessParentheses Useless parentheses. 4 651
UselessParentheses Useless parentheses. 4 651
UselessParentheses Useless parentheses. 4 746
UselessParentheses Useless parentheses. 4 859
UselessParentheses Useless parentheses. 4 888
UselessParentheses Useless parentheses. 4 897
UselessParentheses Useless parentheses. 4 906
UselessParentheses Useless parentheses. 4 915
UselessParentheses Useless parentheses. 4 933
UselessParentheses Useless parentheses. 4 942
EmptyCatchBlock Avoid empty catch blocks 3 1019–1021

Wren Security Commons Utility Classes - Utility Library - org/forgerock/json/RangeSet.java

Rule Violation Priority Line
EmptyCatchBlock Avoid empty catch blocks 3 104–106

Wren Security Commons Utility Classes - Utility Library - org/forgerock/util/LazyList.java

Rule Violation Priority Line
DoubleCheckedLocking Double checked locking is not thread safe in Java. 1 61

Wren Security Commons Utility Classes - Utility Library - org/forgerock/util/LazyMap.java

Rule Violation Priority Line
DoubleCheckedLocking Double checked locking is not thread safe in Java. 1 62

Wren Security Commons Utility Classes - Utility Library - org/forgerock/util/Pair.java

Rule Violation Priority Line
UnnecessaryFullyQualifiedName Unnecessary qualifier 'Pair': 'of' is already in scope 4 49

Wren Security Commons Utility Classes - Utility Library - org/forgerock/util/RangeSet.java

Rule Violation Priority Line
UselessParentheses Useless parentheses. 4 90
UselessParentheses Useless parentheses. 4 91
UselessParentheses Useless parentheses. 4 93
UselessParentheses Useless parentheses. 4 101

Wren Security Commons Utility Classes - Utility Library - org/forgerock/util/SimpleHTTPClient.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'final' on resource specification 'reader': resource specifications are implicitly final 3 96

Wren Security Commons Utility Classes - Utility Library - org/forgerock/util/i18n/LocalizableString.java

Rule Violation Priority Line
EmptyCatchBlock Avoid empty catch blocks 3 98–100

Wren Security Commons Utility Classes - Utility Library - org/forgerock/util/query/QueryFilter.java

Rule Violation Priority Line
OverrideBothEqualsAndHashcode Ensure you override both equals() and hashCode() 3 221
UnnecessaryFullyQualifiedName Unnecessary qualifier 'QueryFilter': 'equalTo' is already in scope 4 556
UnnecessaryFullyQualifiedName Unnecessary qualifier 'QueryFilter': 'greaterThan' is already in scope 4 558
UnnecessaryFullyQualifiedName Unnecessary qualifier 'QueryFilter': 'greaterThanOrEqualTo' is already in scope 4 560
UnnecessaryFullyQualifiedName Unnecessary qualifier 'QueryFilter': 'lessThan' is already in scope 4 562
UnnecessaryFullyQualifiedName Unnecessary qualifier 'QueryFilter': 'lessThanOrEqualTo' is already in scope 4 564
UnnecessaryFullyQualifiedName Unnecessary qualifier 'QueryFilter': 'contains' is already in scope 4 566
UnnecessaryFullyQualifiedName Unnecessary qualifier 'QueryFilter': 'startsWith' is already in scope 4 568

Wren Security Commons Utility Classes - Utility Library - org/forgerock/util/query/QueryFilterParser.java

Rule Violation Priority Line
EmptyControlStatement Empty for statement 3 382–384

Wren Security Commons Utility Classes - Utility Library - org/forgerock/util/thread/listener/ShutdownListener.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'shutdown': the method is declared in an interface type 3 28

Wren Security Commons Utility Classes - Utility Library - org/forgerock/util/thread/listener/ShutdownPriority.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'private' on constructor 'ShutdownPriority(int)': enum constructors are implicitly private 3 50

Wren Security Commons Utility Classes - Utility Library - org/forgerock/util/time/Duration.java

Rule Violation Priority Line
TooManyStaticImports Too many static imports may lead to messy code 3 17–415
UnnecessaryFullyQualifiedName Unnecessary qualifier 'TimeUnit': 'DAYS' is already in scope because it is imported in this file 4 276

Wren Security Commons Utility Classes - Utility Library - org/forgerock/util/xml/XMLUtils.java

Rule Violation Priority Line
EmptyCatchBlock Avoid empty catch blocks 3 91
EmptyCatchBlock Avoid empty catch blocks 3 122