[keyserver] Validate notifications
ClosedPublic
Actions

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

Details

Reviewers

kamil
tomek
marcin

Commits

rCOMM7c7b43562c94: [keyserver] Validate notifications

Summary

Introduce validators and validate notifications.

Depends on D7805

Test Plan

Tested web and ios (emulator) notifications and checked if there were any errors.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

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

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

michal added a child revision: D7807: [keyserver] Validate link responders.May 15 2023, 3:55 AM

Harbormaster completed remote builds in B19332: Diff 26446.May 15 2023, 4:08 AM

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

Might be good for @marcin to take a look as well, as he is very familiar with this code

marcin accepted this revision.May 16 2023, 1:05 AM

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

Closed by commit rCOMM7c7b43562c94: [keyserver] Validate notifications (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: rCOMM7c7b43562c94: [keyserver] Validate notifications.

Revision Contents
Changeset List

Path

Size

keyserver/

src/

push/

send.js

64 lines

lib/

types/

notif-types.js

10 lines

Diff 26572

View Options

keyserver/src/push/send.js