Jira (PUP-11046) Ruby 3 removed URI.escape/unescape

17 views
Skip to first unread message

Josh Cooper (Jira)

unread,
Apr 29, 2021, 6:30:02 PM4/29/21
to puppe...@googlegroups.com
Josh Cooper assigned an issue to Josh Cooper
 
Puppet / Bug PUP-11046
Ruby 3 removed URI.escape/unescape
Change By: Josh Cooper
Assignee: Josh Cooper
Add Comment Add Comment
 
This message was sent by Atlassian Jira (v8.13.2#813002-sha1:c495a97)
Atlassian logo

Josh Cooper (Jira)

unread,
Apr 29, 2021, 6:30:02 PM4/29/21
to puppe...@googlegroups.com
Josh Cooper created an issue
Issue Type: Bug Bug
Assignee: Unassigned
Created: 2021/04/29 3:29 PM
Priority: Normal Normal
Reporter: Josh Cooper

Ruby deprecated URI.escape and unescape in 2.x and removed those methods in 3, because they rely on a deprecated URL parser (RFC2396). However, Puppet needs them to convert potentially unsafe characters, like file paths, into valid URLs.

Josh Cooper (Jira)

unread,
Apr 29, 2021, 7:33:01 PM4/29/21
to puppe...@googlegroups.com
Josh Cooper updated an issue
Change By: Josh Cooper
Sprint: Platform Core KANBAN

Josh Cooper (Jira)

unread,
Apr 29, 2021, 7:33:04 PM4/29/21
to puppe...@googlegroups.com
Josh Cooper updated an issue
Change By: Josh Cooper
Team: Coremunity

Josh Cooper (Jira)

unread,
May 12, 2021, 1:26:05 PM5/12/21
to puppe...@googlegroups.com
Josh Cooper commented on Bug PUP-11046
 
Re: Ruby 3 removed URI.escape/unescape

Passed CI in 39a7d66339

Josh Cooper (Jira)

unread,
May 12, 2021, 1:27:02 PM5/12/21
to puppe...@googlegroups.com
Josh Cooper updated an issue
Change By: Josh Cooper
Fix Version/s: PUP 7.7.0

Josh Cooper (Jira)

unread,
May 12, 2021, 1:27:04 PM5/12/21
to puppe...@googlegroups.com
Josh Cooper updated an issue
Change By: Josh Cooper
Release Notes: Bug Fix
Release Notes Summary: Eliminated calls to URI.escape/unescape which is deprecated in Ruby 2.x and removed in Ruby 3

Claire Cadman (Jira)

unread,
May 18, 2021, 9:56:01 AM5/18/21
to puppe...@googlegroups.com
Claire Cadman updated an issue
Change By: Claire Cadman
Labels: doc-reviewed
Reply all
Reply to author
Forward
0 new messages