Page MenuHomePhabricator

[keyserver] Start validating responders output
ClosedPublic

Authored by michal on May 4 2023, 7:51 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Mar 17, 10:56 PM
Unknown Object (File)
Feb 19 2024, 3:12 PM
Unknown Object (File)
Feb 19 2024, 10:06 AM
Unknown Object (File)
Feb 19 2024, 6:25 AM
Unknown Object (File)
Jan 30 2024, 11:24 PM
Unknown Object (File)
Dec 31 2023, 3:16 AM
Unknown Object (File)
Dec 16 2023, 11:00 PM
Unknown Object (File)
Dec 16 2023, 7:46 PM
Subscribers

Details

Summary

Use the validators and the validateOutput function introduced in the previous diff in the api responders. The id schema conversion is behind the flag for now, but the outputs are still validated.

Depends on D7711

Test Plan

Test web and ios emulator if all outputs are validated correctly and no server error is thrown.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

michal retitled this revision from [keyserver] Convert responders output to [keyserver] Start validating responders output.May 4 2023, 7:53 AM
michal edited the summary of this revision. (Show Details)
keyserver/src/responders/user-responders.js
703–711 ↗(On Diff #26070)

This is done in this diff, because updateUserAvatarResponder was modified while my stack was up and I would have to reorder and modify diffs for to add it to the existing diffs. The changes are small so I thought that I will just add them here.

michal requested review of this revision.May 4 2023, 8:08 AM
This revision is now accepted and ready to land.May 5 2023, 12:36 AM