[keyserver] Remove payloads from `siweAuthResponder` `ServerError`s
ClosedPublic
Actions

Authored by atul on Dec 27 2022, 11:32 PM.

Details

Reviewers

tomek
ashoat

Commits

rCOMMc4844d9fde08: [keyserver] Remove payloads from `siweAuthResponder` `ServerError`s

Summary

Context:
https://phab.comm.dev/D6033#181748
https://phab.comm.dev/D6033#181828

Basically looks like these status payloads are superfluous and aren't used anywhere... so cutting them.

Depends on D6060

Test Plan

Didn't really test... pretty confident this is a benign change.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

atul created this revision.Dec 27 2022, 11:32 PM

atul published this revision for review.Dec 27 2022, 11:32 PM

Harbormaster completed remote builds in B14814: Diff 20205.Dec 27 2022, 11:45 PM

ashoat accepted this revision.Dec 28 2022, 6:11 AM

This revision is now accepted and ready to land.Dec 28 2022, 6:11 AM

rebase before landing

This revision was landed with ongoing or failed builds.Dec 28 2022, 3:20 PM

Closed by commit rCOMMc4844d9fde08: [keyserver] Remove payloads from `siweAuthResponder` `ServerError`s (authored by atul). · Explain Why

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMMc4844d9fde08: [keyserver] Remove payloads from `siweAuthResponder` `ServerError`s.

Harbormaster failed remote builds in B14858: Diff 20265!Dec 28 2022, 3:23 PM

Revision Contents
Changeset List

Path

Size

keyserver/

src/

responders/

user-responders.js

8 lines

Diff 20266

View Options

keyserver/src/responders/user-responders.js

[keyserver] Remove payloads from `siweAuthResponder` `ServerError`sClosedPublicActions