Page MenuHomePhabricator
Differential D11612

[lib] Update `updateRelationshipInputValidator` to be union of `[traditional/farcaster]RelationshipRequestValidator`
ClosedPublic
Actions

Authored by atul on Tue, Apr 9, 10:19 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Apr 27, 9:01 PM
Unknown Object (File)
Sat, Apr 27, 12:19 AM
Unknown Object (File)
Fri, Apr 26, 7:50 PM
Unknown Object (File)
Fri, Apr 26, 11:16 AM
Unknown Object (File)
Sun, Apr 21, 10:28 AM
Unknown Object (File)
Tue, Apr 16, 10:09 PM
Unknown Object (File)
Tue, Apr 16, 4:00 PM
Unknown Object (File)
Tue, Apr 16, 9:35 AM
View All 13 Files
Subscribers
ashoat

Details

Reviewers
ginsu
tomek
varun
will
Commits
rCOMM811b86e47c66: [lib] Update `updateRelationshipInputValidator` to be union of `…
Summary

Update this validator so it supports requests of type TraditionalRelationshipRequest and FarcasterRelationshipRequest

Also renamed updateFarcasterRelationshipInputValidator to farcasterRelationshipRequestValidator for clarity and more consistent naming.

Depends on D11611

Test Plan

Send both TraditionalRelationshipRequest and FarcasterRelationshipRequest requests and ensure that they are validated by keyserver. Confirmed with log statements.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

atul created this revision.Tue, Apr 9, 10:19 PM
Herald added a subscriber: ashoat. · View Herald TranscriptTue, Apr 9, 10:19 PM
Harbormaster returned this revision to the author for changes because remote builds failed.Tue, Apr 9, 10:22 PM
Harbormaster failed remote builds in B28115: Diff 38971!
atul requested review of this revision.Wed, Apr 10, 8:10 AM
Harbormaster completed remote builds in B28115: Diff 38971.Wed, Apr 10, 10:05 AM
atul updated this revision to Diff 39009.Wed, Apr 10, 12:22 PM

rebase

atul added a child revision: D11624: [lib] Fix `farcasterRelationshipRequestValidator`.Wed, Apr 10, 12:47 PM
Harbormaster completed remote builds in B28141: Diff 39009.Wed, Apr 10, 12:50 PM
tomek accepted this revision.Thu, Apr 11, 8:26 AM
This revision is now accepted and ready to land.Thu, Apr 11, 8:26 AM
atul updated this revision to Diff 39252.Thu, Apr 18, 1:21 PM

rebase and alnd

This revision was landed with ongoing or failed builds.Thu, Apr 18, 1:26 PM
Closed by commit rCOMM811b86e47c66: [lib] Update `updateRelationshipInputValidator` to be union of `… (authored by atul). · Explain Why
This revision was automatically updated to reflect the committed changes.
atul added a commit: rCOMM811b86e47c66: [lib] Update `updateRelationshipInputValidator` to be union of `….
Harbormaster completed remote builds in B28311: Diff 39252.Thu, Apr 18, 2:07 PM

Revision Contents
Changeset List

PathSize
keyserver/
src/
responders/
11 lines
lib/
types/
2 lines
10 lines

Diff 39258

View Options

keyserver/src/responders/relationship-responders.js

Loading...
View Options

lib/types/relationship-types.js

Loading...
View Options

lib/types/relationship-types.test.js

Loading...