grizzthedj / smart_proxy_ipam

Foreman Smart Proxy plugin for integration with External IPAM providers
GNU General Public License v3.0
18 stars 20 forks source link

URI.escape is obsolete #57

Open dmgeurts opened 2 years ago

dmgeurts commented 2 years ago
Aug 15 00:37:10 foreman.domain.com smart-proxy[819169]: /usr/local/share/gems/gems/smart_proxy_ipam-0.1.4/lib/smart_proxy_ipam/ipam_helper.rb:106: warning: URI.unescape is obsolete
Aug 15 00:37:10 foreman.domain.com smart-proxy[819169]: /usr/local/share/gems/gems/smart_proxy_ipam-0.1.4/lib/smart_proxy_ipam/ipam_helper.rb:106: warning: URI.escape is obsolete

Looking at the replacements it doesn't seem to be that simple, and as I'm completely new to Ruby and smart_proxy_ipam I'm not sure what the best alternative is. Some background Google-fu has given me the following, but I don't have enough background to pick the right replacement:

https://stackoverflow.com/questions/2824126/whats-the-difference-between-uri-escape-and-cgi-escape/13059657#13059657 https://stackoverflow.com/questions/65423458/ruby-2-7-says-uri-escape-is-obsolete-what-replaces-it