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
refactor: fix ProjectName validation, custom Path extractor for parsing it #1354
Conversation
When the entire workspace is compiled together, like in CI clippy check, logger enables the sqlx/postgres feature. When ProjectName in common is compiled, it is usually done with only the sqlx/sqlite feature. Enabling postgres from logger makes the sqlx derive macro break for some reason. This fixes it.
Might save some time by performing the constant time and light-weight checks before iterating over all characters or checking for profanities. Also remove double negations to make the boolean expression more readable.
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.
This looks great to me, thanks! And thanks for the tests and thorough comments! Regarding the deployer handlers, as far as I can tell it looks like we're good.
ee129b9
to
5d55f6e
Compare
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.
One suggestion. Btw, nicely done with the new tests
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.
LGTM
b9ac3be
to
7852e9a
Compare
Description of change
Custom axum extractor that prints the parse error message for the custom deser.
Supersedes of #1236, Check only the last commits.
How has this been tested? (if applicable)
On normal API calls
On interaction in init (should this be red?)