oscerd opened a new pull request, #29307:
URL: https://github.com/apache/camel/pull/29307

   Backport of the `ConjurClientImpl` part (item 1) of #29209 
([CAMEL-26256](https://issues.apache.org/jira/browse/CAMEL-26256)) to 
camel-4.18.x.
   
   On a 401, `ConjurClientImpl.retrieveSecret` and `createSecret` logged in 
again and called themselves, so with only an `authToken` the same token was 
sent again until a `StackOverflowError`. The client now retries once after 
logging in again, and only when it has an `apiKey` or a `password`; otherwise 
the 401 fails at once. `authToken` is now `volatile`, so a token renewed by one 
call is seen by the others.
   
   The two files are taken unchanged from the commit on main: 
`ConjurClientImpl` and the new `ConjurClientReauthenticationTest`. 
`ConjurClientImpl` on camel-4.18.x was identical to the one on main before the 
fix. The rest of #29209 stays on main: its start-time credential check changes 
when a misconfigured endpoint fails (at start instead of on its first message), 
which is not for a patch release, and its docs and upgrade-guide text describe 
both items.
   
   Tests: `ConjurClientReauthenticationTest` (6 tests) passes on camel-4.18.x, 
with the module built against the branch.
   
   This is a part of #29209 rather than the whole PR, so it asks for a review.
   
   _Claude Code on behalf of Andrea Cosentino (oscerd)_
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)
   


-- 
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]

Reply via email to