Page MenuHomePhabricator
Differential D8143

[keyserver] Validate search endpoint response
ClosedPublic
Actions

Authored by inka on Jun 7 2023, 6:33 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 18, 1:38 AM
Unknown Object (File)
Thu, Apr 18, 1:38 AM
Unknown Object (File)
Thu, Apr 18, 1:38 AM
Unknown Object (File)
Thu, Apr 18, 1:38 AM
Unknown Object (File)
Thu, Apr 18, 1:38 AM
Unknown Object (File)
Thu, Apr 18, 1:35 AM
Unknown Object (File)
Mar 5 2024, 9:28 PM
Unknown Object (File)
Feb 27 2024, 1:58 AM
View All 32 Files
Subscribers
ashoat
atul
tomek

Details

Reviewers
michal
Commits
rCOMMac297e5134ca: [keyserver] Validate search endpoint response
Summary

issue: https://linear.app/comm/issue/ENG-3939/validate-search-endpoint
Search endpoint was missing a validation on the response

Test Plan

Called the search endpoint from client, checked that no errors / warnings appear

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

inka created this revision.Jun 7 2023, 6:33 AM
Herald added subscribers: tomek, ashoat. · View Herald TranscriptJun 7 2023, 6:33 AM
inka added 1 blocking reviewer(s): michal.Jun 7 2023, 6:33 AM
Harbormaster completed remote builds in B20088: Diff 27523.Jun 7 2023, 6:50 AM
inka requested review of this revision.Jun 7 2023, 6:50 AM
michal accepted this revision.Jun 12 2023, 2:35 AM

The output validation looks good but you also need to modify the input logic.

keyserver/src/responders/message-responders.js
496–499 ↗(On Diff #27523)
This revision is now accepted and ready to land.Jun 12 2023, 2:35 AM
inka updated this revision to Diff 27646.Jun 12 2023, 7:16 AM

Modify input logic

Harbormaster failed remote builds in B20181: Diff 27646!Jun 12 2023, 7:18 AM
inka updated this revision to Diff 27651.Jun 12 2023, 7:29 AM

Rebase

Harbormaster failed remote builds in B20185: Diff 27651!Jun 12 2023, 7:43 AM
atul added a subscriber: atul.Jun 12 2023, 9:35 AM

(Restarted failing iOS build)

Harbormaster completed remote builds in B20185: Diff 27651.Jun 12 2023, 9:44 AM
Closed by commit rCOMMac297e5134ca: [keyserver] Validate search endpoint response (authored by • InkaSokolowska). · Explain WhyJun 12 2023, 10:58 PM
This revision was automatically updated to reflect the committed changes.
• InkaSokolowska added a commit: rCOMMac297e5134ca: [keyserver] Validate search endpoint response.

Revision Contents
Changeset List

PathSize
keyserver/
src/
responders/
22 lines

Diff 27675

View Options

keyserver/src/responders/message-responders.js

Loading...