!admin debug get-signing-keys returns unhelpful error when run with no arguments #1458
Labels
No labels
Blocked
Bug
Cherry-picking
Database
Dependencies
Dependencies/Renovate
Difficulty
Easy
Difficulty
Hard
Difficulty
Medium
Documentation
Enhancement
Good first issue
Help wanted
Inherited
Matrix/Administration
Matrix/Appservices
Matrix/Auth
Matrix/Client
Matrix/Core
Matrix/E2EE
Matrix/Federation
Matrix/Hydra
Matrix/MSC
Matrix/Media
Matrix/T&S
Meta
Meta/CI
Meta/Packaging
Priority
Blocking
Priority
High
Priority
Low
Security
Status
Confirmed
Status
Duplicate
Status
Invalid
Status
Needs Investigation
Support
To-Merge
Wont fix
old/ci/cd
old/rust
No milestone
No project
No assignees
2 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
continuwuation/continuwuity#1458
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
https://matrix.to/#/#continuwuity:continuwuity.org/$j3rk666MxnS0KwDHwA7fiMArORHhLkye8WbMvH1ijew
This command defaults to querying the local server's remote signing key cache for its own keys, which will always(?) fail. The server name argument should probably be required and error out if the local server's own name is given.
I was just trying to "fix" this by simply erroring out when no argument is given. However, there is actually useful output now when I call it without server argument. It verifies that my keys are available for the notary servers. (example output below). So, the only output that might be a bit more useful is that the error message could be a bit more useful,
no entry in db(or similar), such as:"No notary could find our signing key" (or similar). On a 2nd glance, looking a the code, it seems that it does try to get the signing keys locally when there is no notary server configured. Not sure what the most useful behavior should be.