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

Autolinks extension for http:// and https:// links includes trailing spaces #62

Closed
vsch opened this issue Feb 27, 2017 · 2 comments
Closed

Autolinks extension for http:// and https:// links includes trailing spaces #62

vsch opened this issue Feb 27, 2017 · 2 comments
Labels
🪲 bug 🚬 fixed for next release 🎉 fixed

Comments

@vsch
Copy link
Owner

vsch commented Feb 27, 2017 •
edited

A link which solely of http:// or https:// links includes trailing spaces.
@vsch vsch added the ☹ bug label Feb 27, 2017
@vsch
Copy link
Owner Author

vsch commented Feb 27, 2017

Actually this bug affects all links that contain nothing but the protocol prefix. The fix consists of trimming end of the link text during processing.

@vsch
Copy link
Owner Author

vsch commented Feb 27, 2017

It affects not only spaces. org.nibor.autolink includes the next character after the protocol prefix. so https://, will also be parsed as an auto-link

vsch added a commit that referenced this issue Feb 27, 2017
@vsch
fix #60, #62
2c152ae
@vsch vsch closed this as completed Feb 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
🪲 bug 🚬 fixed for next release 🎉 fixed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@vsch