Page MenuHomePhabricator
Differential D11610

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

Authored by atul on Apr 9 2024, 9:53 PM.
Tags
None
Referenced Files
F1779892: D11610.diff
Sat, May 18, 1:52 AM
Unknown Object (File)
Tue, May 7, 4:47 PM
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)
Apr 17 2024, 2:20 PM
View All 15 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.Apr 9 2024, 9:53 PM
Herald added a subscriber: ashoat. · View Herald TranscriptApr 9 2024, 9:53 PM
Harbormaster returned this revision to the author for changes because remote builds failed.Apr 9 2024, 9:55 PM
Harbormaster failed remote builds in B28112: Diff 38968!
atul added a child revision: D11611: [keyserver] Update `updateRelationshipsResponder` to handle `RelationshipRequest`.Apr 9 2024, 10:11 PM
atul requested review of this revision.Apr 10 2024, 8:09 AM
Harbormaster completed remote builds in B28112: Diff 38968.Apr 10 2024, 10:03 AM
atul updated this revision to Diff 39007.Apr 10 2024, 12:21 PM

rebasing

Harbormaster completed remote builds in B28139: Diff 39007.Apr 10 2024, 12:40 PM
tomek accepted this revision.Apr 11 2024, 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.Apr 11 2024, 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...