-
Notifications
You must be signed in to change notification settings - Fork 363
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
Creating apps with emoji names results in a 500 #1375
Comments
We have created an issue in Pivotal Tracker to manage this: https://www.pivotaltracker.com/story/show/166681830 The labels on this github issue will be updated when the story is started. |
I can reproduce this by triggering validation failures on the app in general:
|
This error class was removed in the following commit: cloudfoundry/capi-workspace@cbb3000 Regular `CloudController::Errors::ApiError` exceptions are raised now for this situation and those are caught at a higher level. The existing unit tests weren't catching this because they only explicitly raised earlier error types like InvalidApp that would be caught first. There is still an UnknownError being raised due to a DatabaseError that's caused by us not having enough validations on the AppModel. Github Issue: #1375 [#166681830](#1375) Authored-by: Tim Downey <[email protected]>
fixed:
|
We have created an issue in Pivotal Tracker to manage this: https://www.pivotaltracker.com/story/show/172728700 The labels on this github issue will be updated when the story is started. |
We've done some investigation and found that this is still a problem – it occurs only in MySQL, which is why Seth did not reproduce it above. But we also found that it affects many different fields on many V3 resources (anywhere arbitrary text is stored in a string column in the database). It doesn't seem super high-priority right now, but we wrote a story to explore this further in the future: https://www.pivotaltracker.com/story/show/172728679 |
The above story leads to a story in the VAT backlog. This story has been ported to the CAPI public tracker: https://www.pivotaltracker.com/n/projects/966314/stories/166681830 |
I also stumbled upon this issue while writing a test for anther story..where I chosed as app name '🦄🦄🦄'. |
Reproduction Steps
Underlying Error
Expected Behavior
This request should either succeed or return a human-readable error letting me know why I'm not allowed to have fun app names.
The text was updated successfully, but these errors were encountered: