[keyserver] improve error messages in rust-node-addon
ClosedPublic
Actions

Authored by varun on Jun 22 2023, 11:45 PM.

Details

Reviewers

• jon
bartek

Commits

rCOMM8e11f2df18b3: [keyserver] improve error messages in rust-node-addon

Summary

add a more detailed message in case an error is caught.

Depends on D8298

Test Plan

ran keyserver but not identity service to get errors connecting to identity. saw better error logs

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

varun created this revision.Jun 22 2023, 11:45 PM

Herald added subscribers: tomek, ashoat. · View Herald TranscriptJun 22 2023, 11:45 PM

varun added a child revision: D8300: [keyserver] call identity service during account creation and deletion.Jun 22 2023, 11:46 PM

Harbormaster returned this revision to the author for changes because remote builds failed.Jun 23 2023, 12:07 AM

Harbormaster failed remote builds in B20445: Diff 28027!

varun requested review of this revision.Jun 23 2023, 1:22 PM

• jon accepted this revision.Jun 26 2023, 5:54 AM

This revision is now accepted and ready to land.Jun 26 2023, 5:54 AM

This revision was landed with ongoing or failed builds.Jun 26 2023, 6:54 PM

Closed by commit rCOMM8e11f2df18b3: [keyserver] improve error messages in rust-node-addon (authored by varun). · Explain Why

This revision was automatically updated to reflect the committed changes.

varun added a commit: rCOMM8e11f2df18b3: [keyserver] improve error messages in rust-node-addon.

Revision Contents
Changeset List

Path

Size

keyserver/

addons/

rust-node-addon/

src/

identity_client/

add_reserved_username.rs

2 lines

remove_reserved_username.rs

2 lines

Diff 28027

View Options

keyserver/addons/rust-node-addon/src/identity_client/add_reserved_username.rs