Skip to content

[CDAP-20872] Add AeadCipher SPI for Encryption in CDAP #13579

[CDAP-20872] Add AeadCipher SPI for Encryption in CDAP

[CDAP-20872] Add AeadCipher SPI for Encryption in CDAP #13579

Triggered via pull request November 16, 2023 04:15
Status Success
Total duration 10s
Artifacts

trigger.yml

on: pull_request
trigger
0s
trigger
Fit to window
Zoom out
Zoom in

Annotations

53 warnings
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-data-fabric/src/test/java/io/cdap/cdap/data2/dataset2/lib/table/leveldb/LevelDBMetricsTableTest.java#L47
Abbreviation in name 'LevelDBMetricsTableTest' must contain no more than '1' consecutive capital letters.
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-data-fabric/src/test/java/io/cdap/cdap/data2/dataset2/lib/table/leveldb/LevelDBTableServiceTest.java#L57
Abbreviation in name 'LevelDBTableServiceTest' must contain no more than '1' consecutive capital letters.
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: cdap-data-fabric/src/test/java/io/cdap/cdap/data2/dataset2/lib/table/leveldb/LevelDBTableServiceTest.java#L84
Distance between variable 'table2' declaration and its first usage is 4, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: cdap-data-fabric/src/test/java/io/cdap/cdap/data2/dataset2/lib/table/leveldb/LevelDBTableServiceTest.java#L97
Distance between variable 'table1Size' declaration and its first usage is 4, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: cdap-data-fabric/src/test/java/io/cdap/cdap/data2/dataset2/lib/table/leveldb/LevelDBTableServiceTest.java#L99
Distance between variable 'table2Size' declaration and its first usage is 5, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: cdap-data-fabric/src/test/java/io/cdap/cdap/data2/dataset2/lib/table/leveldb/LevelDBTableServiceTest.java#L140
Distance between variable 'uncompressedDiskSizeBytes' declaration and its first usage is 5, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: cdap-data-fabric/src/test/java/io/cdap/cdap/data2/dataset2/lib/table/leveldb/LevelDBTableServiceTest.java#L167
Distance between variable 'emptyTableDiskSize' declaration and its first usage is 4, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: cdap-data-fabric/src/test/java/io/cdap/cdap/data2/dataset2/lib/table/leveldb/LevelDBTableCoreTest.java#L250
Distance between variable 'valRead' declaration and its first usage is 10, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-gateway/src/main/java/io/cdap/cdap/gateway/router/handlers/AuthenticationHandler.java#L87
Abbreviation in name 'authServerURLs' must contain no more than '1' consecutive capital letters.
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-gateway/src/main/java/io/cdap/cdap/gateway/router/handlers/AuthenticationHandler.java#L194
Abbreviation in name 'getAuthenticationURLs' must contain no more than '1' consecutive capital letters.
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-gateway/src/main/java/io/cdap/cdap/gateway/router/handlers/AuthenticationHandler.java#L301
Abbreviation in name 'getConfiguredAuthServerURLs' must contain no more than '1' consecutive capital letters.
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-gateway/src/main/java/io/cdap/cdap/gateway/router/NettyRouter.java#L216
Abbreviation in name 'finalSSLHandlerFactory' must contain no more than '1' consecutive capital letters.
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/NettyRouterHttpTest.java#L78
Distance between variable 'sConfiguration' declaration and its first usage is 4, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/NettyRouterHttpTest.java#L83
Distance between variable 'discoveryServiceClient' declaration and its first usage is 4, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/NettyRouterHttpTest.java#L85
Distance between variable 'userIdentityExtractor' declaration and its first usage is 4, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/RouterResource.java#L65
Distance between variable 'discoveryServiceClient' declaration and its first usage is 4, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/RouterResource.java#L68
Distance between variable 'extractor' declaration and its first usage is 4, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/RouterResource.java#L69
Distance between variable 'sConf' declaration and its first usage is 4, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/AuditLogTest.java#L90
Distance between variable 'sConf' declaration and its first usage is 4, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/AuditLogTest.java#L165
Abbreviation in name 'createURLConnection' must contain no more than '1' consecutive capital letters.
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/NettyRouterTestBase.java#L138
Abbreviation in name 'getHTTPClient' must contain no more than '1' consecutive capital letters.
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/NettyRouterTestBase.java#L142
Abbreviation in name 'resolveURI' must contain no more than '1' consecutive capital letters.
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/NettyRouterTestBase.java#L592
Abbreviation in name 'openURL' must contain no more than '1' consecutive capital letters.
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/AuthServerAnnounceTest.java#L69
Abbreviation in name 'testEmptyAnnounceAddressURLsConfig' must contain no more than '1' consecutive capital letters.
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/AuthServerAnnounceTest.java#L81
Abbreviation in name 'testAnnounceURLsConfig' must contain no more than '1' consecutive capital letters.
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/AuthServerAnnounceTest.java#L96
Abbreviation in name 'getAuthURI' must contain no more than '1' consecutive capital letters.
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/AuthServerAnnounceTest.java#L107
Abbreviation in name 'resolveURI' must contain no more than '1' consecutive capital letters.
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/AuthServerAnnounceTest.java#L128
Distance between variable 'sConfiguration' declaration and its first usage is 6, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/AuthServerAnnounceTest.java#L133
Distance between variable 'discoveryServiceClient' declaration and its first usage is 6, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/AuthServerAnnounceTest.java#L136
Distance between variable 'userIdentityExtractor' declaration and its first usage is 6, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/RoutingToDataSetsTest.java#L72
Distance between variable 'discoveryServiceClient' declaration and its first usage is 4, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/RoutingToDataSetsTest.java#L74
Distance between variable 'userIdentityExtractor' declaration and its first usage is 4, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/RoutingToDataSetsTest.java#L76
Distance between variable 'sConf' declaration and its first usage is 4, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-security/src/main/java/io/cdap/cdap/security/encryption/guice/AbstractAeadCipherProvider.java#L31
Abbreviation in name 'NOOP_AEAD_CIPHER_NAME' must contain no more than '1' consecutive capital letters.
com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck: cdap-security/src/main/java/io/cdap/cdap/security/encryption/guice/LazyDelegateAeadCipher.java#L48
WhitespaceAround: 'synchronized' is not followed by whitespace. Empty blocks may only be represented as {} when not part of a multi-block statement (4.1.3)
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-security/src/main/java/io/cdap/cdap/security/authorization/DefaultAccessEnforcer.java#L285
Abbreviation in name 'isAccessingSystemNSAsMasterUser' must contain no more than '1' consecutive capital letters.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: cdap-security/src/test/java/io/cdap/cdap/security/encryption/guice/LazyDelegateAeadCipherTest.java#L26
Wrong lexicographical order for 'io.cdap.cdap.security.spi.encryption.AeadCipherContext' import. Should be before 'io.cdap.cdap.security.spi.encryption.AeadCipherCryptor'.
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: cdap-security/src/test/java/io/cdap/cdap/security/authorization/DefaultAccessEnforcerTest.java#L223
Distance between variable 'authEnforcementService' declaration and its first usage is 11, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: cdap-security/src/test/java/io/cdap/cdap/security/authorization/DefaultAccessEnforcerTest.java#L232
Distance between variable 'namespaces' declaration and its first usage is 11, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).