allthingssecurity commented on PR #26910: URL: https://github.com/apache/camel/pull/26910#issuecomment-5868389100
@davsclaus thanks, all three are done in 41768e4b9 (new commit on top): - The comment in `ExchangeHelper.removeVariables` now says that the repositories scan the map of the id parsed from the prefix, and that for `header:<routeId>:<var>.` this id is always `header` (CAMEL-25050), so the scan covers the shared `header` map of all routes or groups. - `RouteVariableRepositoryTest` and `GroupVariableRepositoryTest` have `testRemoveHeaderVariablesWithPrefix`, which uses the `header:rs:resp.` (`header:grp:resp.`) prefix form. It checks that the `rs:resp` body, `header:rs:response.a` and `header:rs2:resp.a` are kept. - The PR description now covers `removeVariablesWithPrefix` and links CAMEL-25050. `*Variable*` in camel-core: 149 tests, 0 failures. _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]
