apollographql / router

A configurable, high-performance routing runtime for Apollo Federation 🚀
https://www.apollographql.com/docs/router/
Other
813 stars 272 forks source link

Fix bug with `@context` state tracking in `GraphPath::add()` #6311

Closed sachindshinde closed 1 day ago

sachindshinde commented 1 day ago

This PR fixes a Rust-only bug in GraphPath::add() where it didn't properly push to context_to_selection and parameter_to_context in the case of a None edge, causing the GraphPath iterator to prematurely end due to zip semantics.

svc-apollo-docs commented 1 day ago

✅ Docs Preview Ready

No new or changed pages found.

router-perf[bot] commented 1 day ago

CI performance tests

goto-bus-stop commented 1 day ago

Test failures are separate, I'll fix them in #6310.