Skip to content
This repository has been archived by the owner on Oct 23, 2024. It is now read-only.

Multiple forward slashes allowed in app ID #3660

Closed
philipnrmn opened this issue Apr 5, 2016 · 0 comments · Fixed by mesosphere/marathon-ui#762
Closed

Multiple forward slashes allowed in app ID #3660

philipnrmn opened this issue Apr 5, 2016 · 0 comments · Fixed by mesosphere/marathon-ui#762
Assignees
@pierluigi

Comments

@philipnrmn
Copy link
Contributor

  • Open Create modal
  • enter an ID with multiple forward slashes eg //////group////app-id
  • no invalid input warning appears
  • enter a cmd value and submit the application
  • multiple slashes are converted to single slashes, eg /group/app-id

Expected behaviour:

  • entering multiple forward slashes in the UI should result in a warning
  • app IDs with multiple forward slashes should not be accepted and transformed by the service

screen shot 2016-04-05 at 16 16 00
@philipnrmn philipnrmn added gui and removed analyze labels Apr 5, 2016
@philipnrmn philipnrmn changed the title Multiple forward slashes allows in app ID Multiple forward slashes allowed in app ID Apr 5, 2016
pierluigi pushed a commit to mesosphere/marathon-ui that referenced this issue Apr 21, 2016
Disallow multiple forward slashes in app id validation
363ca4d 
This fixes d2iq-archive/marathon#3660
@pierluigi pierluigi mentioned this issue Apr 21, 2016
Merged
@pierluigi pierluigi self-assigned this Apr 21, 2016
@d2iq-archive d2iq-archive locked and limited conversation to collaborators Mar 27, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@pierluigi pierluigi

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Disallow multiple forward slashes mesosphere/marathon-ui
3 participants
@Poltergeist @pierluigi @philipnrmn