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

Obsolete references to URI.escape and URI.unescape #71

Closed
jacobjlevine opened this issue Jun 12, 2020 · 18 comments
Closed

Obsolete references to URI.escape and URI.unescape #71

jacobjlevine opened this issue Jun 12, 2020 · 18 comments

Comments

@jacobjlevine
Copy link

I'm seeing the following two deprecation warnings (Ruby 2.7.1, Rails 6.0.3.1), both from /app/vendor/bundle/ruby/2.7.0/gems/roxml-4.1.0/lib/roxml/xml/references.rb:50:

warning: URI.unescape is obsolete
warning: URI.escape is obsolete

URI.unescape(convention.call(URI.escape(what, /\/|::/)))

@jacobjlevine
Copy link
Author

@Empact any chance we can get this merged in?

@ibmandeep
Copy link

@Empact Please merge this PR this should solve the issue.

@ibrahim-saifee
Copy link
Contributor

This PR #77 might resolve the issue. As the URI.escape and URI.unescape is deprecated for Ruby 2.7.x so, we need to use URI.encode_www_form_component and URI.decode_www_form_component for URL encoding and decoding respectively.

https://ruby-doc.org/stdlib-2.5.3/libdoc/uri/rdoc/URI/Escape.html#method-i-escape
https://ruby-doc.org/stdlib-2.5.3/libdoc/uri/rdoc/URI/Escape.html#method-i-unescape

@mmplisskin
Copy link

Any possibility for a review on this @Empact

@nehresma
Copy link

@Empact first, thank you very much for this gem. It has been very helpful to a number of us in the community over the years.

There are several somewhat large and often used downstream projects (like quickbooks-ruby) using this gem that are impacted by this particular issue. Those projects are unable to upgrade to Ruby 3 because URI.escape and URI.unescape have been removed. Would you have a few minutes to merge this in and release an updated gem? Or would you be open to considering adding someone else in the community as a collaborator on this github repository and the rubygems.org gem so that an update could be released?

Thanks again for this gem and for your consideration.

@anthonyhamou
Copy link

@Empact first, thank you very much for this gem. It has been very helpful to a number of us in the community over the years.

Could you please help us ? it is urgent !!

@jacobjlevine
Copy link
Author

Hey folks! I just got in touch with @Empact, and he's open to giving an additional contributor admin permissions for the gem. Anyone open to taking on that responsibility?

@campbecf
Copy link

I'd be willing to contribute to the project. Some of the issues on this repo are blockers for my company. How do I go about getting in touch with @Empact ?

@lostapathy
Copy link
Collaborator

I have commit to this repo but have been hesitant to merge much since we can't cut a release. If somebody gets the ability to push new releases to rubygems, I'd be happy to help more with the maintenance here.

@jacobjlevine
Copy link
Author

@campbecf @lostapathy I just sent @Empact a follow-up email encouraging him to participate in this thread directly. Hopefully that way we can consolidate this into a single conversation.

@Empact
Copy link
Owner

Empact commented Aug 18, 2021

Hello everyone, sorry about the neglect of this library but I'm not currently using it so it's low on my list of priorities. @lostapathy can merge, and I've added @campbecf as well. I'm happy to extend the ability to cut releases as well.

@Empact
Copy link
Owner

Empact commented Aug 18, 2021

Closed via #77

@Empact Empact closed this as completed Aug 18, 2021
@jacobjlevine
Copy link
Author

Hey @Empact @lostapathy @campbecf! Looks like this has been merged into master but hasn't been cut as a release yet. Would it be possible to make that happen to unblock those of us looking to explore upgrading to Ruby 3? Thanks!

@lostapathy
Copy link
Collaborator

Unfortunately I don't have the ability to cut a new gem - I think only @Empact has that.

@anthonyhamou
Copy link

Hello Ben @Empact we need you to cut roxml as a release please. Can we rely on you ? It is very important ! Thx

@Empact
Copy link
Owner

Empact commented Sep 9, 2021

Thanks everyone! Please see 4.2.0, and let me know if you run into any issues:
https://github.com/Empact/roxml/releases/tag/v4.2.0
https://rubygems.org/gems/roxml

@pmsiam0
Copy link

pmsiam0 commented Dec 2, 2022

This PR #77 might resolve the issue. As the URI.escape and URI.unescape is deprecated for Ruby 2.7.x so, we need to use URI.encode_www_form_component and URI.decode_www_form_component for URL encoding and decoding respectively.

https://ruby-doc.org/stdlib-2.5.3/libdoc/uri/rdoc/URI/Escape.html#method-i-escape https://ruby-doc.org/stdlib-2.5.3/libdoc/uri/rdoc/URI/Escape.html#method-i-unescape

This one isn't worked in my case cause I use ruby 3.0.4p208 and my case it is not working can anyone help me to do this in my code?

@jacobjlevine
Copy link
Author

@pmsiam0 what version of the gem are you using (you may need to check your Gemfile.lock to see the exact version)? I believe 4.2.0 fixed the issue, and I've been using various versions of Ruby 3 over the past year with no problem.

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 a pull request may close this issue.

10 participants