Improve error message for invalid version #88068
Assignees
Comments
|
@iambriccardo I assigned you since your team owns this endpoint and it looks like you've edited this file recently – feel free to assign to someone else if they'd be more able to help. I can also do this, might take me more time though since I would have to figure out where to make the change. |
Closed
8 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The
OrganizationArtifactbundleAssembleendpoint performs validation on the version string:sentry/src/sentry/api/endpoints/organization_artifactbundle_assemble.py
Line 41 in 90c4228
If the provided version fails the validation (for example, in the case where the version is an empty string), the endpoint returns a
400status with the message'' does not match '^[^/]+$'.This message should be improved. In its current state, the error message is vague since it does not mention which field failed validation. It would also likely be better if we provided human-readable information rather than just a regex. A better error message could look like the following:
The better error message would have avoided confusion in getsentry/sentry-cli#2385.
The text was updated successfully, but these errors were encountered: