Page MenuHomePhabricator

[keyserver] Update `updateRelationshipsResponder` to handle `RelationshipRequest`
ClosedPublic

Authored by atul on Tue, Apr 9, 10:11 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 26, 6:07 PM
Unknown Object (File)
Fri, Apr 26, 5:24 PM
Unknown Object (File)
Wed, Apr 17, 4:42 PM
Unknown Object (File)
Wed, Apr 17, 4:20 PM
Unknown Object (File)
Tue, Apr 16, 2:38 PM
Unknown Object (File)
Tue, Apr 16, 12:32 PM
Unknown Object (File)
Tue, Apr 16, 10:46 AM
Unknown Object (File)
Mon, Apr 15, 11:12 PM
Subscribers

Details

Summary

Make necessary changes to updateRelationshipsResponder and updateRelationships to support RelationshipRequest which is union of previous TraditionalRelationshipRequest and FarcasterRelationshipRequest.

  • Using request.action instead of destructuring makes it clear to flow in request.action === relationshipActions.FARCASTER_MUTUAL branch that request.userIDsToFID is available
  • We get userIDs using userIDsToFID if farcaster request and pass to new Set(requestUserIDs) which is the only place request.userIDs is "directly" consumed
NOTE: Will update validator in next diff.

Depends on D11610

Test Plan

Send both traditional and farcaster requests, set breakpoint in updateRelationships, ensure that values are as expected at each step.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Tue, Apr 9, 10:13 PM
Harbormaster failed remote builds in B28113: Diff 38969!
Harbormaster failed remote builds in B28114: Diff 38970!
atul requested review of this revision.Wed, Apr 10, 9:54 AM
This revision is now accepted and ready to land.Thu, Apr 11, 8:24 AM
This revision was landed with ongoing or failed builds.Thu, Apr 18, 1:26 PM
This revision was automatically updated to reflect the committed changes.