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

Uri escape deprecated #14

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

graaff
Copy link

@graaff graaff commented Jul 15, 2023

URI.escape has been deprecated for some time and has been removed in ruby 3.0. Use the URI.encode_www_form_component method instead.

See e.g. https://docs.knapsackpro.com/2020/uri-escape-is-obsolete-percent-encoding-your-query-string for background.

Addresses #12

URI.escape has been deprecated for a long time and is no longer
available in ruby 3.0 and later versions.

Fixes claygregory#12
@DavidGoodwin
Copy link

DavidGoodwin commented Oct 31, 2023

Thanks - this fixes the plugin on Debian Bookworm for me (Ruby 3.1)

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