Page MenuHomePhabricator

[keyserver][native] remove cookie logic
ClosedPublic

Authored by varun on Mar 5 2024, 11:31 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, May 12, 11:31 PM
Unknown Object (File)
Sun, May 12, 11:31 PM
Unknown Object (File)
Sun, May 12, 11:31 PM
Unknown Object (File)
Sun, May 12, 11:09 PM
Unknown Object (File)
Apr 7 2024, 8:16 PM
Unknown Object (File)
Apr 7 2024, 8:15 PM
Unknown Object (File)
Apr 7 2024, 8:14 PM
Unknown Object (File)
Apr 7 2024, 8:13 PM
Subscribers

Details

Summary

the identity service is now completely stateless again so we can safely remove cookie logic from our gRPC clients. this code ensured that the same identity service node would handle start and finish RPCs, but that is no longer necessary since we're storing the in-progress workflows in DDB instead of in memory. this code was only added to the gRPC clients, not gRPC-web, so there is nothing to remove from web.

Depends on D11256

Test Plan

successfully logged in and registered from native and keyserver with staging identity

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

varun requested review of this revision.Mar 5 2024, 11:49 PM
This revision is now accepted and ready to land.Mar 6 2024, 6:35 AM

Should this stack include a diff to bring us back to having two parallel identity instances in production?

Should this stack include a diff to bring us back to having two parallel identity instances in production?

yeah it should

This revision was automatically updated to reflect the committed changes.