Project

General

Profile

Backport #10873

backport r48666 lib/uri/generic.rb (URI::Generic.build)

Added by jrafanie (Joe Rafaniello) almost 5 years ago. Updated almost 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
[ruby-core:68201]

Description

Fix URI::Generic.build to properly build IPv6 host values with square brackets.

use hostname= to detect and wrap IPv6 hosts.
Build is accepting URI components and users may not expect
that a host component needs to be wrapped with square brackets
since it's not providing a URI.

https://github.com/ruby/ruby/commit/4ec9793ba88b27028c826c6bc6c49ee59df8d920
https://github.com/ruby/ruby/pull/765


Related issues

Related to Backport21 - Backport #10875: backport r48666 lib/uri/generic.rb (URI::Generic.build)Closed02/20/2015Actions
#2

Updated by usa (Usaku NAKAMURA) almost 5 years ago

  • Related to Backport #10875: backport r48666 lib/uri/generic.rb (URI::Generic.build) added
#3

Updated by usa (Usaku NAKAMURA) almost 5 years ago

  • Status changed from Open to Closed
  • % Done changed from 0 to 100

Applied in changeset r49669.


merge revision(s) 48666: [Backport #10873]

* lib/uri/generic.rb (URI::Generic.build):
  use hostname= to detect and wrap IPv6 hosts.
  Build is accepting URI components and users may not expect
  that a host component needs to be wrapped with square brackets
  since it's not providing a URI.
  Note: initialize with arg_check => true does not wrap IPv6 hosts.
  by Joe Rafaniello <jrafanie@redhat.com>
  https://github.com/ruby/ruby/pull/765 fix GH-765

* test/uri/test_generic.rb: Add more tests

Also available in: Atom PDF