Skip to content

Conversation

fineswap
Copy link
Contributor

@fineswap fineswap commented Jan 4, 2015

This is particularly useful in situations where the original URL contains special characters that must be retained, for example a '+' sign, and not converted to its encoded form ('%20', in case of '+').

This is particularly useful in situations where the original URL contains special characters that must be retained, for example a '+' sign, and not converted to its encoded form ('%20', in case of '+').
@fineswap
Copy link
Contributor Author

fineswap commented Jan 4, 2015

Hmm, this pull is actually useless because URI implementation will always convert a plus sign + into a %20 representation. More info: https://www.subbu.org/blog/2008/02/uri-escaping-and-javaneturlencoder

This is stupid... I would like to use a plus sign in the path, is it even possible with URI implementation, @smarek?

@fineswap
Copy link
Contributor Author

fineswap commented Jan 4, 2015

Actually, this seems to be connected to URL encoding in AHC itself, so my previous comment is incorrect. So the pull is still good, I guess...

smarek added a commit that referenced this pull request Jan 4, 2015
Introduce getURI(String) method to convert a String to a URI.
@smarek smarek merged commit a20f795 into android-async-http:master Jan 4, 2015
@fineswap fineswap deleted the normalizeURL branch January 4, 2015 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants