allthingssecurity commented on PR #26799: URL: https://github.com/apache/camel/pull/26799#issuecomment-5828171224
@gnodet @davsclaus heads-up: the early return added to `openCircuit` here also affects two existing callers that call it on an already open circuit. - When the half open handler is not ready, `calculateState` relied on `openCircuit` to schedule the next check, so the circuit now stays open for good. - With `keepOpen`, `onStart` relied on it to suspend the consumer of a restarted route. I filed CAMEL-25014 and sent #26874. It keeps the idempotent `openCircuit` from this PR and adds a `reopenCircuit` for those two paths. Since this PR has `port/camel-4.22.x`, the fix probably wants the same port. _Claude Code on behalf of allthingssecurity_ -- 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]
