Skip to content

Add routeId to filterContext#3856

Merged
MustafaSaber merged 2 commits intozalando:masterfrom
xavivars:feature/route-id
Feb 7, 2026
Merged

Add routeId to filterContext#3856
MustafaSaber merged 2 commits intozalando:masterfrom
xavivars:feature/route-id

Conversation

@xavivars
Copy link
Contributor

@xavivars xavivars commented Feb 4, 2026

This MR exposes the route ID to filterContext.

Over the past few years, we've seen many instances where it could have been handy to know the route that was triggered for a specific request: we have predicates that match based on A/B testing, or other input (headers/cookies), and there may be multiple routes that end up in the same origin.

Being able to track (and potentially log, or apply filters based on) the route ID would be extremely helpful for us.

Slack conversation: https://gophers.slack.com/archives/C82Q5JNH5/p1755799907005649

Filters can have now access to the specific route ID that was triggered for the request.
Signed-off-by: Xavi Ivars <xavi.ivars@gmail.com>
@zalando-robot
Copy link

Cannot start a pipeline due to:

No accountable user for this pipeline: no Zalando employee associated to this GitHub username

Click on pipeline status check Details link below for more information.

Signed-off-by: Xavi Ivars <xavi.ivars@gmail.com>
@zalando-robot
Copy link

Cannot start a pipeline due to:

No accountable user for this pipeline: no Zalando employee associated to this GitHub username

Click on pipeline status check Details link below for more information.

@szuecs szuecs added the minor no risk changes, for example new filters label Feb 7, 2026
@MustafaSaber MustafaSaber merged commit aad5f21 into zalando:master Feb 7, 2026
17 of 18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement minor no risk changes, for example new filters

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants