Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Validate NameOrSnowflake in Path<NameOrSnowflake> declarations #1217

Closed
auguwu opened this issue Nov 24, 2023 · 0 comments · Fixed by #1501
Closed

Validate NameOrSnowflake in Path<NameOrSnowflake> declarations #1217

auguwu opened this issue Nov 24, 2023 · 0 comments · Fixed by #1501
Assignees
@auguwu
Labels
bug Something isn't working rust Pull requests that update Rust code
Milestone
v0.1.0-beta

Comments

@auguwu
Copy link
Member

auguwu commented Nov 24, 2023

Since Names are user input, they should be validated:

/workspaces/Noelware/Products/charted/charted (feat/repo-endpoints*) » curl http://localhost:3651/users/@Me                                                             noel@floofbox
{"success":false,"errors":[{"code":"ENTITY_NOT_FOUND","message":"User with ID or name [@Me] was not found."}]}%
@auguwu auguwu added bug Something isn't working rust Pull requests that update Rust code labels Nov 24, 2023
@auguwu auguwu added this to the v0.1.0-beta milestone Nov 24, 2023
@auguwu auguwu self-assigned this Nov 24, 2023
@auguwu auguwu mentioned this issue Nov 24, 2023
Closed
54 tasks
@auguwu auguwu mentioned this issue Jan 26, 2024
Merged
@auguwu auguwu linked a pull request Jan 26, 2024 that will close this issue
Smaller Codebase #1501
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@auguwu auguwu

Labels
bug Something isn't working rust Pull requests that update Rust code
Projects
None yet
Milestone
v0.1.0-beta
Development

Successfully merging a pull request may close this issue.

Smaller Codebase charted-dev/charted
1 participant
@auguwu