Page MenuHomePhabricator
Differential D10813

[keyserver] Update auth request validator
ClosedPublic
Actions

Authored by tomek on Jan 25 2024, 6:39 AM.
Tags
None
Referenced Files
F3388812: D10813.id36131.diff
Fri, Nov 29, 4:07 PM
F3388421: D10813.diff
Fri, Nov 29, 2:14 PM
Unknown Object (File)
Fri, Nov 8, 1:16 AM
Unknown Object (File)
Tue, Nov 5, 11:27 PM
Unknown Object (File)
Tue, Nov 5, 9:39 PM
Unknown Object (File)
Tue, Nov 5, 7:34 PM
Unknown Object (File)
Oct 22 2024, 4:41 PM
Unknown Object (File)
Oct 22 2024, 4:41 PM
View All 49 Files
Subscribers
ashoat

Details

Reviewers
varun
inka
michal
Commits
rCOMM0d12f0220b5e: [keyserver] Update auth request validator
Summary

The validator wasn't matching the type.

Depends on D10786

Test Plan

Call an endpoint and check if the validator passes.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

tomek created this revision.Jan 25 2024, 6:39 AM
Herald added a subscriber: ashoat. · View Herald TranscriptJan 25 2024, 6:39 AM
tomek added a child revision: D10814: [lib] Call keyserver auth.Jan 25 2024, 6:40 AM
Harbormaster completed remote builds in B26154: Diff 36131.Jan 25 2024, 6:58 AM
tomek requested review of this revision.Jan 25 2024, 6:58 AM
varun accepted this revision.Jan 25 2024, 7:10 AM

ah, sorry about that... i must have missed this while rebasing the stack starting with D10490 to add source to the request type.

thanks for fixing 🙏

This revision is now accepted and ready to land.Jan 25 2024, 7:10 AM
Closed by commit rCOMM0d12f0220b5e: [keyserver] Update auth request validator (authored by tomek). · Explain WhyJan 29 2024, 3:44 AM
This revision was automatically updated to reflect the committed changes.
tomek added a commit: rCOMM0d12f0220b5e: [keyserver] Update auth request validator.

Revision Contents
Changeset List

PathSize
keyserver/
src/
responders/
1 line

Diff 36250

View Options

keyserver/src/responders/user-responders.js

Loading...