-
Notifications
You must be signed in to change notification settings - Fork 152
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Better WebAuthn and other error responses #2608
base: master
Are you sure you want to change the base?
Conversation
b9ed69d
to
031acc0
Compare
/// debug format with spaces | ||
/// | ||
/// Probably shouldn't use this with any of the complex types because it'll get weird quick! | ||
pub fn variant_as_nice_string(&self) -> String { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are we using this anywhere?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah, aded it over here as a first run to get a nice response to the user
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wouldn't it be better to just have a to_str that matches rather than a regex? Seems a bit overkill ...
Fixes that thing where if you don't have UV you should get a slightly nicer error.
Checklist