Skip to content
GitHub Actions / Checkstyle completed Jan 8, 2025 in 0s

Checkstyle report

33 violation(s) found

Annotations

Check warning on line 40 in cdap-common/src/main/java/io/cdap/cdap/common/http/AuthenticationChannelHandler.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck

Extra separation in import group before 'java.io.IOException'

Check warning on line 40 in cdap-common/src/main/java/io/cdap/cdap/common/http/AuthenticationChannelHandler.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck

Wrong lexicographical order for 'java.io.IOException' import. Should be before 'org.slf4j.LoggerFactory'.

Check warning on line 41 in cdap-common/src/main/java/io/cdap/cdap/common/http/AuthenticationChannelHandler.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck

Wrong lexicographical order for 'java.util.Queue' import. Should be before 'org.slf4j.LoggerFactory'.

Check warning on line 42 in cdap-common/src/main/java/io/cdap/cdap/common/http/AuthenticationChannelHandler.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck

Wrong lexicographical order for 'java.util.stream.Collectors' import. Should be before 'org.slf4j.LoggerFactory'.

Check warning on line 43 in cdap-common/src/main/java/io/cdap/cdap/common/http/AuthenticationChannelHandler.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck

Wrong lexicographical order for 'javax.annotation.Nullable' import. Should be before 'org.slf4j.LoggerFactory'.

Check warning on line 66 in cdap-common/src/main/java/io/cdap/cdap/common/http/AuthenticationChannelHandler.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.javadoc.MissingJavadocMethodCheck

Missing a Javadoc comment.

Check warning on line 185 in cdap-common/src/main/java/io/cdap/cdap/common/http/AuthenticationChannelHandler.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.whitespace.ParenPadCheck

')' is preceded with whitespace.

Check warning on line 202 in cdap-common/src/main/java/io/cdap/cdap/common/http/AuthenticationChannelHandler.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck

'typecast' is not followed by whitespace.

Check warning on line 216 in cdap-common/src/main/java/io/cdap/cdap/common/http/AuthenticationChannelHandler.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.whitespace.OperatorWrapCheck

'+' should be on a new line.

Check warning on line 235 in cdap-common/src/main/java/io/cdap/cdap/common/http/AuthenticationChannelHandler.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.javadoc.NonEmptyAtclauseDescriptionCheck

At-clause should have a non-empty description.

Check warning on line 242 in cdap-common/src/main/java/io/cdap/cdap/common/http/AuthenticationChannelHandler.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck

WhitespaceAround: '{' is not preceded with whitespace.

Check warning on line 256 in cdap-common/src/main/java/io/cdap/cdap/common/http/AuthenticationChannelHandler.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck

WhitespaceAround: '{' is not preceded with whitespace.

Check warning on line 33 in cdap-common/src/main/java/io/cdap/cdap/common/auditlogging/AuditLogSetterHook.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.javadoc.SummaryJavadocCheck

First sentence of Javadoc is missing an ending period.

Check warning on line 51 in cdap-common/src/main/java/io/cdap/cdap/common/auditlogging/AuditLogSetterHook.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck

WhitespaceAround: '{' is not preceded with whitespace.

Check warning on line 41 in cdap-common/src/test/java/io/cdap/cdap/common/http/AuthenticationChannelHandlerTest.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck

Extra separation in import group before 'java.util.ArrayDeque'

Check warning on line 41 in cdap-common/src/test/java/io/cdap/cdap/common/http/AuthenticationChannelHandlerTest.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck

Wrong lexicographical order for 'java.util.ArrayDeque' import. Should be before 'org.mockito.Mockito'.

Check warning on line 42 in cdap-common/src/test/java/io/cdap/cdap/common/http/AuthenticationChannelHandlerTest.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck

Wrong lexicographical order for 'java.util.Queue' import. Should be before 'org.mockito.Mockito'.

Check warning on line 106 in cdap-common/src/test/java/io/cdap/cdap/common/http/AuthenticationChannelHandlerTest.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck

Distance between variable 'internalAuthEnabled' 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).

Check warning on line 107 in cdap-common/src/test/java/io/cdap/cdap/common/http/AuthenticationChannelHandlerTest.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck

Distance between variable 'auditLogWriterMock' 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).

Check warning on line 122 in cdap-common/src/test/java/io/cdap/cdap/common/http/AuthenticationChannelHandlerTest.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck

Distance between variable 'internalAuthEnabled' 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).

Check warning on line 123 in cdap-common/src/test/java/io/cdap/cdap/common/http/AuthenticationChannelHandlerTest.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck

Distance between variable 'auditLogWriterMock' 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).

Check warning on line 136 in cdap-common/src/test/java/io/cdap/cdap/common/http/AuthenticationChannelHandlerTest.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck

WhitespaceAround: '{' is not preceded with whitespace.

Check warning on line 145 in cdap-common/src/test/java/io/cdap/cdap/common/http/AuthenticationChannelHandlerTest.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.whitespace.EmptyLineSeparatorCheck

'METHOD_DEF' should be separated from previous line.

Check warning on line 26 in cdap-security-spi/src/main/java/io/cdap/cdap/security/spi/authentication/SecurityRequestContext.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck

Extra separation in import group before 'java.util.ArrayDeque'

Check warning on line 130 in cdap-security-spi/src/main/java/io/cdap/cdap/security/spi/authentication/SecurityRequestContext.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck

WhitespaceAround: '{' is not preceded with whitespace.