Skip to content

PR #15786 #2020

Sign in to view logs

PR #15786

PR #15786 #2020

Triggered via dynamic January 2, 2025 04:23
@masiveskymasivesky
⁠ 00f60af
refs/pull/15786/head
Status Success
Total duration 1m 17s
Artifacts –

codeql

on: dynamic
Matrix: analyze
Fit to window
Zoom out
Zoom in

Annotations

56 warnings
Analyze (ruby)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Analyze (python)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Analyze (javascript)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: cdap-app-fabric/src/main/java/io/cdap/cdap/app/guice/AppFabricServiceRuntimeModule.java#L155
Wrong lexicographical order for 'io.cdap.cdap.common.encryption.guice.DataStorageAeadEncryptionModule' import. Should be before 'io.cdap.cdap.securestore.spi.SecretStore'.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: cdap-app-fabric/src/main/java/io/cdap/cdap/internal/credential/CredentialProfileStore.java#L24
Wrong lexicographical order for 'io.cdap.cdap.common.encryption.AeadCipher' import. Should be before 'io.cdap.cdap.proto.id.CredentialProfileId'.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: cdap-app-fabric/src/main/java/io/cdap/cdap/internal/credential/CredentialProfileStore.java#L25
Wrong lexicographical order for 'io.cdap.cdap.common.encryption.guice.DataStorageAeadEncryptionModule' import. Should be before 'io.cdap.cdap.proto.id.CredentialProfileId'.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: cdap-app-fabric/src/main/java/io/cdap/cdap/internal/credential/CredentialIdentityStore.java#L24
Wrong lexicographical order for 'io.cdap.cdap.common.encryption.AeadCipher' import. Should be before 'io.cdap.cdap.proto.id.CredentialProfileId'.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: cdap-app-fabric/src/main/java/io/cdap/cdap/internal/credential/CredentialIdentityStore.java#L25
Wrong lexicographical order for 'io.cdap.cdap.common.encryption.guice.DataStorageAeadEncryptionModule' import. Should be before 'io.cdap.cdap.proto.id.CredentialProfileId'.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: cdap-app-fabric/src/test/java/io/cdap/cdap/internal/credential/CredentialProviderTestBase.java#L39
Wrong lexicographical order for 'io.cdap.cdap.common.encryption.NoOpAeadCipher' import. Should be before 'io.cdap.cdap.security.authorization.AuthorizationEnforcementModule'.
com.puppycrawl.tools.checkstyle.checks.javadoc.MissingJavadocTypeCheck: cdap-common/src/main/java/io/cdap/cdap/common/encryption/guice/AeadCipherCryptorExtensionLoader.java#L30
Missing a Javadoc comment.
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-common/src/main/java/io/cdap/cdap/common/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.naming.MemberNameCheck: cdap-common/src/main/java/io/cdap/cdap/common/encryption/guice/AbstractAeadCipherProvider.java#L31
Member name 'NOOP_AEAD_CIPHER_NAME' must match pattern '^[a-z][a-zA-Z0-9]+$'.
com.puppycrawl.tools.checkstyle.checks.javadoc.MissingJavadocMethodCheck: cdap-common/src/main/java/io/cdap/cdap/common/encryption/guice/AbstractAeadCipherProvider.java#L37
Missing a Javadoc comment.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: cdap-common/src/test/java/io/cdap/cdap/common/encryption/DataStorageAeadEncryptionModuleTest.java#L24
Wrong lexicographical order for 'io.cdap.cdap.common.encryption.guice.DataStorageAeadEncryptionModule' import. Should be before 'io.cdap.cdap.common.guice.ConfigModule'.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: cdap-common/src/test/java/io/cdap/cdap/common/encryption/UserCredentialAeadEncryptionModuleTest.java#L24
Wrong lexicographical order for 'io.cdap.cdap.common.encryption.guice.UserCredentialAeadEncryptionModule' import. Should be before 'io.cdap.cdap.common.guice.ConfigModule'.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: cdap-gateway/src/main/java/io/cdap/cdap/gateway/router/RouterMain.java#L32
Wrong lexicographical order for 'io.cdap.cdap.common.encryption.guice.UserCredentialAeadEncryptionModule' import. Should be before 'io.cdap.cdap.common.runtime.DaemonMain'.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: cdap-gateway/src/main/java/io/cdap/cdap/gateway/router/NettyRouter.java#L38
Wrong lexicographical order for 'io.cdap.cdap.common.encryption.AeadCipher' import. Should be before 'io.cdap.cdap.security.auth.UserIdentityExtractor'.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: cdap-gateway/src/main/java/io/cdap/cdap/gateway/router/NettyRouter.java#L39
Wrong lexicographical order for 'io.cdap.cdap.common.encryption.guice.UserCredentialAeadEncryptionModule' import. Should be before 'io.cdap.cdap.security.auth.UserIdentityExtractor'.
com.puppycrawl.tools.checkstyle.checks.javadoc.MissingJavadocMethodCheck: cdap-gateway/src/main/java/io/cdap/cdap/gateway/router/NettyRouter.java#L106
Missing a Javadoc comment.
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.javadoc.SummaryJavadocCheck: cdap-gateway/src/main/java/io/cdap/cdap/gateway/router/NettyRouter.java#L282
First sentence of Javadoc is missing an ending period.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: cdap-gateway/src/main/java/io/cdap/cdap/gateway/router/handlers/AuthenticationHandler.java#L35
Wrong lexicographical order for 'io.cdap.cdap.common.encryption.AeadCipher' import. Should be before 'io.cdap.cdap.security.auth.UserIdentityPair'.
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.javadoc.MissingJavadocMethodCheck: cdap-gateway/src/main/java/io/cdap/cdap/gateway/router/handlers/AuthenticationHandler.java#L92
Missing a Javadoc comment.
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-gateway/src/main/java/io/cdap/cdap/gateway/router/handlers/AuthenticationHandler.java#L137
Abbreviation in name 'clientIP' 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.imports.CustomImportOrderCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/AuthServerAnnounceTest.java#L34
Wrong lexicographical order for 'io.cdap.cdap.common.encryption.NoOpAeadCipher' import. Should be before 'io.cdap.cdap.security.auth.UserIdentityExtractor'.
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.imports.CustomImportOrderCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/NettyRouterHttpsTest.java#L33
Wrong lexicographical order for 'io.cdap.cdap.common.encryption.NoOpAeadCipher' import. Should be before 'io.cdap.cdap.security.auth.UserIdentityExtractor'.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/NettyRouterHttpTest.java#L31
Wrong lexicographical order for 'io.cdap.cdap.common.encryption.NoOpAeadCipher' import. Should be before 'io.cdap.cdap.security.auth.UserIdentityExtractor'.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/NettyRouterTestBase.java#L41
Wrong lexicographical order for 'io.cdap.cdap.common.encryption.NoOpAeadCipher' import. Should be before 'io.cdap.cdap.security.auth.TokenValidator'.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/RouterResource.java#L29
Wrong lexicographical order for 'io.cdap.cdap.common.encryption.NoOpAeadCipher' import. Should be before 'io.cdap.cdap.security.auth.UserIdentityExtractor'.
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.imports.CustomImportOrderCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/AuditLogTest.java#L34
Wrong lexicographical order for 'io.cdap.cdap.common.encryption.NoOpAeadCipher' import. Should be before 'io.cdap.cdap.security.auth.TokenValidator'.
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.imports.CustomImportOrderCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/RoutingToDataSetsTest.java#L31
Wrong lexicographical order for 'io.cdap.cdap.common.encryption.NoOpAeadCipher' import. Should be before 'io.cdap.cdap.security.auth.UserIdentityExtractor'.
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.imports.CustomImportOrderCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/ConfigBasedRequestBlockingTest.java#L25
Wrong lexicographical order for 'io.cdap.cdap.common.encryption.NoOpAeadCipher' import. Should be before 'io.cdap.cdap.security.auth.TokenValidator'.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: cdap-master/src/main/java/io/cdap/cdap/master/environment/k8s/RouterServiceMain.java#L35
Wrong lexicographical order for 'io.cdap.cdap.common.encryption.guice.UserCredentialAeadEncryptionModule' import. Should be before 'io.cdap.cdap.proto.id.NamespaceId'.
com.puppycrawl.tools.checkstyle.checks.javadoc.SummaryJavadocCheck: cdap-master/src/main/java/io/cdap/cdap/master/environment/k8s/RouterServiceMain.java#L51
First sentence of Javadoc is missing an ending period.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: cdap-security/src/main/java/io/cdap/cdap/security/authorization/AuthorizationEnforcementModule.java#L29
Wrong lexicographical order for 'io.cdap.cdap.common.encryption.guice.UserCredentialAeadEncryptionModule' import. Should be before 'io.cdap.cdap.proto.security.Permission'.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: cdap-security/src/main/java/io/cdap/cdap/security/authorization/DefaultAccessEnforcer.java#L39
Wrong lexicographical order for 'io.cdap.cdap.common.encryption.AeadCipher' import. Should be before 'io.cdap.cdap.proto.security.Role'.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: cdap-security/src/main/java/io/cdap/cdap/security/authorization/DefaultAccessEnforcer.java#L40
Wrong lexicographical order for 'io.cdap.cdap.common.encryption.guice.UserCredentialAeadEncryptionModule' import. Should be before 'io.cdap.cdap.proto.security.Role'.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: cdap-security/src/test/java/io/cdap/cdap/security/authorization/DefaultAccessEnforcerTest.java#L50
Wrong lexicographical order for 'io.cdap.cdap.common.encryption.FakeAeadCipher' import. Should be before 'io.cdap.cdap.security.auth.context.AuthenticationTestContext'.