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

Publish API (Symfony) fails for URLs with trailing spaces #201

Closed
tripu opened this issue Jul 1, 2015 · 1 comment
Closed

Publish API (Symfony) fails for URLs with trailing spaces #201

tripu opened this issue Jul 1, 2015 · 1 comment
Assignees
@tripu
Labels
bug

Comments

@tripu
Copy link
Member

tripu commented Jul 1, 2015

Reported by @tidoust:
“The bug I reported on specberus for links with trailing spaces apparently also exists in the "publish" tool used by Echidna (that tool checks the editor draft URL and complains that it is invalid when there is a trailing space).”

Possible action: sanitise (trim) those URLs in publisher.js before sending them to the Symfony endpoint.

Related: w3c/specberus#207.
@tripu tripu added the bug label Jul 1, 2015
This was referenced Jul 1, 2015
Closed
Closed
@tripu tripu self-assigned this Jul 14, 2015
@deniak deniak mentioned this issue Aug 13, 2015
Merged
@tripu tripu reopened this Aug 14, 2015
@tripu
Copy link
Member Author

tripu commented Aug 14, 2015

Closed automatically; reopening to double-check.
Note to self: confirm after deployment that this was fixed by w3c/specberus#230.

@deniak deniak mentioned this issue Aug 17, 2015
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tripu tripu

Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@tripu