Page MenuHomePhabricator
Differential D11610

[keyserver] Narrow `updateRelationshipInputValidator` to exclude `farcaster` action
ClosedPublic
Actions

Authored by atul on Tue, Apr 9, 9:53 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, May 1, 2:22 PM
Unknown Object (File)
Fri, Apr 26, 7:17 PM
Unknown Object (File)
Fri, Apr 26, 1:40 PM
Unknown Object (File)
Sun, Apr 21, 4:28 PM
Unknown Object (File)
Thu, Apr 18, 1:30 PM
Unknown Object (File)
Wed, Apr 17, 2:20 PM
Unknown Object (File)
Wed, Apr 17, 10:44 AM
Unknown Object (File)
Mon, Apr 15, 7:34 PM
View All 13 Files
Subscribers
ashoat

Details

Reviewers
ginsu
tomek
varun
will
Commits
rCOMM8eef17ad3a41: [keyserver] Narrow `updateRelationshipInputValidator` to exclude `farcaster`…
Summary

Modify updateRelationshipInputValidator so action is an enum of traditionalRelationshipActionsList vs relationshipActionsList.

Depends on D11609

Test Plan

Tried making request with farcaster action and it failed as expected.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

atul created this revision.Tue, Apr 9, 9:53 PM
Herald added a subscriber: ashoat. · View Herald TranscriptTue, Apr 9, 9:53 PM
Harbormaster returned this revision to the author for changes because remote builds failed.Tue, Apr 9, 9:55 PM
Harbormaster failed remote builds in B28112: Diff 38968!
atul added a child revision: D11611: [keyserver] Update `updateRelationshipsResponder` to handle `RelationshipRequest`.Tue, Apr 9, 10:11 PM
atul requested review of this revision.Wed, Apr 10, 8:09 AM
Harbormaster completed remote builds in B28112: Diff 38968.Wed, Apr 10, 10:03 AM
atul updated this revision to Diff 39007.Wed, Apr 10, 12:21 PM

rebasing

Harbormaster completed remote builds in B28139: Diff 39007.Wed, Apr 10, 12:40 PM
tomek accepted this revision.Thu, Apr 11, 2:24 AM
tomek added inline comments.
keyserver/src/responders/relationship-responders.js
17–20 ↗(On Diff #39007)

Does that mean it's not possible to update Farcaster relationship?

This revision is now accepted and ready to land.Thu, Apr 11, 2:24 AM
atul added inline comments.Thu, Apr 18, 1:18 PM
keyserver/src/responders/relationship-responders.js
17–20 ↗(On Diff #39007)

It will be via FarcasterRelationshipRequest, just not with TraditionalRelationshipRequest.

atul updated this revision to Diff 39250.Thu, Apr 18, 1:20 PM

rebase and land

This revision was landed with ongoing or failed builds.Thu, Apr 18, 1:26 PM
Closed by commit rCOMM8eef17ad3a41: [keyserver] Narrow `updateRelationshipInputValidator` to exclude `farcaster`… (authored by atul). · Explain Why
This revision was automatically updated to reflect the committed changes.
atul added a commit: rCOMM8eef17ad3a41: [keyserver] Narrow `updateRelationshipInputValidator` to exclude `farcaster`….
Harbormaster completed remote builds in B28309: Diff 39250.Thu, Apr 18, 1:56 PM

Revision Contents
Changeset List

PathSize
keyserver/
src/
responders/
7 lines
lib/
types/
2 lines

Diff 39256

View Options

keyserver/src/responders/relationship-responders.js

Loading...
View Options

lib/types/relationship-types.js

Loading...