[keyserver] Validate link responders
ClosedPublic
Actions

Authored by michal on May 15 2023, 3:55 AM.

Details

Reviewers

kamil
tomek

Commits

rCOMM72310720eb50: [keyserver] Validate link responders

Summary

Add validators and validate link responders, add tests.

Depends on D7806

Test Plan

yarn jest

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

michal created this revision.May 15 2023, 3:55 AM

Herald added subscribers: atul, ashoat. · View Herald TranscriptMay 15 2023, 3:55 AM

michal added a child revision: D7808: [keyserver] Convert upload responders.May 15 2023, 3:57 AM

Harbormaster completed remote builds in B19333: Diff 26447.May 15 2023, 4:12 AM

michal requested review of this revision.May 15 2023, 4:12 AM

tomek accepted this revision.May 16 2023, 3:18 AM

This revision is now accepted and ready to land.May 16 2023, 3:18 AM

Closed by commit rCOMM72310720eb50: [keyserver] Validate link responders (authored by Michal Gniadek <michal.gniadek@swmansion.com>). · Explain WhyMay 17 2023, 1:29 AM

This revision was automatically updated to reflect the committed changes.

Michal Gniadek <michal.gniadek@swmansion.com> added a commit: rCOMM72310720eb50: [keyserver] Validate link responders.

@michal are you planning to write validators for fetchPrimaryInviteLinksResponder? I guess it should be done during the rebase.

Sorry, I should have noticed that during rebase, I will put up a diff with the validators.

Revision Contents
Changeset List

Path

Size

keyserver/

src/

responders/

link-responders.js

39 lines

responder-validators.test.js

15 lines

Diff 26573

View Options

keyserver/src/responders/link-responders.js