Modernize MixedAuthenticatorTest to use List.of() for empty authorities#3328
Conversation
Agent-Logs-Url: https://github.com/Waffle/waffle/sessions/55976625-31a0-46ed-bee0-d9ba60c784f6 Co-authored-by: hazendaz <975267+hazendaz@users.noreply.github.com>
Agent-Logs-Url: https://github.com/Waffle/waffle/sessions/55976625-31a0-46ed-bee0-d9ba60c784f6 Co-authored-by: hazendaz <975267+hazendaz@users.noreply.github.com>
|
@copilot Lets continue as more found after the initial ones you updated. See Error: D:\a\waffle\waffle\Source\JNA\waffle-spring-security5\src\test\java\waffle\spring\DelegatingNegotiateSecurityFilterTest.java:118: Prefer java.util.List.sort(Comparator.naturalOrder()) |
Agent-Logs-Url: https://github.com/Waffle/waffle/sessions/d7a92697-cd3b-434f-8691-cbf046a39019 Co-authored-by: hazendaz <975267+hazendaz@users.noreply.github.com>
Updated all 6 reported Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
|
@copilot Please also fix D:\a\waffle\waffle\Source\JNA\waffle-tomcat10\src\test\java\waffle\apache\MixedAuthenticatorTest.java:355: Prefer java.util.List.of(). Since this hit tomcat 10, its likely in the other tomcat modules too so maybe best to look for similar pattern throughout. |
Agent-Logs-Url: https://github.com/Waffle/waffle/sessions/672aa47a-e185-45d5-8891-7c699f1dc91c Co-authored-by: hazendaz <975267+hazendaz@users.noreply.github.com>
Applied the fix in both Tomcat 10 and Tomcat 11 Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
|
@copilot Here are next set of issues Error: Medium: To make log readable, log format () should contain non-sign character. [waffle.spring.DelegatingNegotiateSecurityFilter] At DelegatingNegotiateSecurityFilter.java:[line 140] SLF4J_SIGN_ONLY_FORMAT |
Agent-Logs-Url: https://github.com/Waffle/waffle/sessions/aa355aeb-1cc0-4672-8494-4cdfe4e67541 Co-authored-by: hazendaz <975267+hazendaz@users.noreply.github.com>
Applied the Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
|
@copilot Next one to modernize D:\a\waffle\waffle\Source\JNA\waffle-demo-jakarta\waffle-demo-spring-boot-filter3\src\main\java\waffle\spring\boot\demo\DemoController.java:32: Prefer java.lang.String.formatted(Object...) |
Agent-Logs-Url: https://github.com/Waffle/waffle/sessions/3b743252-2e7c-4072-9ed7-bbe3bdb2400c Co-authored-by: hazendaz <975267+hazendaz@users.noreply.github.com>
Applied the modernization in Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
|
@copilot Please also update these modernization issues 67 |
Agent-Logs-Url: https://github.com/Waffle/waffle/sessions/94a08263-afe8-4eb2-9e83-10faf5871464 Co-authored-by: hazendaz <975267+hazendaz@users.noreply.github.com>
Updated all 6 Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
|
@copilot Also fix Error: D:\a\waffle\waffle\Source\JNA\waffle-demo-jakarta\waffle-demo-spring-boot-filter4\src\main\java\waffle\spring\boot\demo\DemoController.java:32: Prefer java.lang.String.formatted(Object...) |
Agent-Logs-Url: https://github.com/Waffle/waffle/sessions/48d6eead-a29e-43f8-9ac6-26c3277c7c4e Co-authored-by: hazendaz <975267+hazendaz@users.noreply.github.com>
Fixed in Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
|
@copilot PLease fix this too D:\a\waffle\waffle\Source\JNA\waffle-shiro\src\test\java\waffle\shiro\GroupMappingWaffleRealmTest.java:49: Prefer java.util.Map.of(Object, Object) |
Agent-Logs-Url: https://github.com/Waffle/waffle/sessions/e1e6b307-2fc4-426c-99dd-05ef9a8447b2 Co-authored-by: hazendaz <975267+hazendaz@users.noreply.github.com>
Fixed in Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
|
@copilot Also fix D:\a\waffle\waffle\Source\JNA\waffle-shiro-jakarta\src\test\java\waffle\shiro\GroupMappingWaffleRealmTest.java:49: Prefer java.util.Map.of(Object, Object) |
Agent-Logs-Url: https://github.com/Waffle/waffle/sessions/20d0dbb5-d103-4da4-8cb9-752e2ee6d444 Co-authored-by: hazendaz <975267+hazendaz@users.noreply.github.com>
Fixed in Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
Modernizer flagged
MixedAuthenticatorTestfor legacy empty-list construction (Collections.emptyList()) in the Tomcat 9 test module. This updates the test to use the Java 9+ collection factory API expected by current rules while preserving behavior.Scope
waffle-tomcat9test code atMixedAuthenticatorTest(custom principal setup path).Code modernization
Collections.emptyList()withList.of()when constructingGenericPrincipal.java.util.Collectionstojava.util.List.Behavioral impact
GenericPrincipal.