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.

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

Inform users about known issues #675

Closed
wants to merge 1 commit into from

Conversation

Dalzhim
Copy link

@Dalzhim Dalzhim commented Mar 12, 2019

Core functionnality of this library is broken in cases where URLs contain the %25 escape sequence. It is a major issue for users who need to handle any valid URL and it must be disclosed to those considering adoption of this library.

Core functionnality of this library is broken in cases where URLs contain the %25 escape sequence. It is a major issue for users who need to handle any valid URL and it must be disclosed to those considering adoption of this library.
@Dalzhim
Copy link
Author

Dalzhim commented Mar 12, 2019

This PR is meant to document known issues with history versions prior to solving issue #505

@Dalzhim Dalzhim mentioned this pull request Mar 25, 2019
1 task
@mjackson mjackson closed this Mar 26, 2019
@OliverJAsh
Copy link
Contributor

This is a good change until we've fixed #505.

@Dalzhim Dalzhim deleted the Dalzhim-document-505 branch March 26, 2019 20:47
@lock lock bot locked as resolved and limited conversation to collaborators May 25, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants