when we call the registration, login, and update password _finish RPCs, we need to use the load balancer cookie to ensure these requests are routed to the same identity service instance that received the _start RPCs.
Details
Details
- Reviewers
bartek - Commits
- rCOMM2030bd785167: [grpc clients] use cookie for finish requests
called the native_rust_library client methods and confirmed that registration/login/update password all worked on staging with multiple instances running
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
Comment Actions
I'm glad that you've figured it out! 🎉
I have one concern inline, feel free to re-request if I've been missing something
| shared/grpc_clients/src/error.rs | ||
|---|---|---|
| 34–36 ↗ | (On Diff #34080) | I find this more universal |
| shared/grpc_clients/src/identity/shared.rs | ||
| 63–65 ↗ | (On Diff #34080) | What about local dev environment? This cookie won't be available. |