slachiewicz commented on PR #12692:
URL: https://github.com/apache/maven/pull/12692#issuecomment-5218436832
Thanks — all three points addressed in 783450bd7a.
The `ANY_USER` one turned out to be broader than it looked: it is 11 sites,
not one. Every Jetty 9 constraint in the suite declared explicit roles, and the
port flattened all of them, so each is now mapped back to `SPECIFIC_ROLE` with
the roles it had before — `{"user", "admin"}` for `HttpServer`, `"user"` or
`"deployer"` for the ITs. Worth noting MNG-553 expressed its role through the
two-arg `Constraint(__BASIC_AUTH, "user")` constructor rather than `setRoles`,
so it does not show up in a `setRoles` grep.
The 16 auth and handler ITs touched by these changes pass locally: 30 tests,
0 failures.
Also opened #12693 to backport this to `maven-4.0.x`.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
