/lib/httparty/hash_conversions.rb:35:in `normalize_param': warning: URI.escape is obsolete

[mrjcleaver]

Since a recent install, I'm seeing hundreds of lines re: httparty URI.escape. I call via the soundcloud gem which calls httparty

The offending line is https://github.com/jnunemaker/httparty/blob/master/lib/httparty/hash_conversions.rb#L35

/var/lib/gems/1.9.1/gems/httparty-0.11.0/lib/httparty/hash_conversions.rb:35:in `normalize_param': warning: URI.escape is obsolete
/var/lib/gems/1.9.1/gems/httparty-0.11.0/lib/httparty/hash_conversions.rb:35:in `normalize_param': warning: URI.escape is obsolete
/var/lib/gems/1.9.1/gems/httparty-0.11.0/lib/httparty/hash_conversions.rb:35:in `normalize_param': warning: URI.escape is obsolete
/var/lib/gems/1.9.1/gems/httparty-0.11.0/lib/httparty/hash_conversions.rb:35:in `normalize_param': warning: URI.escape is obsolete

Should we remove the obsolete call?

[jnunemaker]

I'm open to a pull and explanation. :)

[triptec]

Here's a quite long explanation http://stackoverflow.com/a/13059657.

[jnunemaker]

This is fixed in latest.

[river-of-code]

Hmm... is this fixed? On 0.13.1 (which I believe right now is the latest) I am still seeing the 'obsolete' warning - and URI.escape is still in hash_conversions.rb:35 on the master branch... was this reverted?

[howdoicomputer]

Same here. I'm on 0.13.1 and the problem is still showing up.

[howdoicomputer]

I changed URI.encode to CGI.escape and to get around this. Node, cgi needs to be required.

[Olgagr]

I also see this warnings. HashConversions module still uses URI.encode.

[rusikf]

@Olgagr , @howdoicomputer, @the-robear , @jnunemaker URI.escape is obsolete was fixed in current master( not on release version) and this issue must be closed.

[mrjcleaver]

Any plan to release then? It's been another month and a half. Not fixed in 0.13.3.

[mrjcleaver]

Sorry to pester, but https://rubygems.org/gems/httparty still shows 0.13.3