oscerd commented on PR #27077:
URL: https://github.com/apache/camel/pull/27077#issuecomment-5906564173
Thanks @davsclaus — both addressed in 2b52471:
- **Ordering:** `Authorization` is now checked before the body, so a request
payload isn't taken as the token; new test verifies a valid Bearer header wins
over a body payload. Documented that validation replaces the body with the
`JwtSvid`.
- **401 handler:** corrected — a missing token throws
`IllegalArgumentException`, a rejected one
`io.spiffe.exception.JwtSvidException`; the handler catches both.
- Minor: `removeHeaders` switched to the canonical `{ pattern: ... }` form.
On the rebase: this and #27053 both touch the operations list and
`SpiffeProducerTest`, so I'll rebase whichever merges second. Re-requesting
review.
_Claude Code on behalf of oscerd_
--
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]
