Bug #10402: URI regression in 2.2.0-preview1 (bad URI(is not URI?): URI::InvalidURIError)

• Status: Closed
• Assignee: naruse (Yui NARUSE)
• Target version: -
• ruby -v: ruby 2.2.0preview1 (2014-09-17 trunk 47616) [x86_64-darwin13]
• Backport: 2.0.0: UNKNOWN, 2.1: UNKNOWN

[JuanitoFatas (Juanito Fatas)]

$ ruby -v -ruri -e "URI 'http://example.com/?a[]=b&a[]=c'"
ruby 2.1.3p242 (2014-09-19 revision 47630) [x86_64-darwin13.0]

$ ruby -v -ruri -e "URI 'http://example.com/?a[]=b&a[]=c'"
ruby 2.2.0preview1 (2014-09-17 trunk 47616) [x86_64-darwin13]
/Users/Juan/.rubies/ruby-2.2.0-preview1/lib/ruby/2.2.0/uri/rfc3986_parser.rb:47:in `split': bad URI(is not URI?): http://example.com/?a[]=b&a[]=c (URI::InvalidURIError)
	from /Users/Juan/.rubies/ruby-2.2.0-preview1/lib/ruby/2.2.0/uri/rfc3986_parser.rb:53:in `parse'
	from /Users/Juan/.rubies/ruby-2.2.0-preview1/lib/ruby/2.2.0/uri/common.rb:226:in `parse'
	from /Users/Juan/.rubies/ruby-2.2.0-preview1/lib/ruby/2.2.0/uri/common.rb:713:in `URI'
	from -e:1:in `<main>'

Found in tarcieri/http gem's test.

[nobu (Nobuyoshi Nakada)]

• Related to Feature #2542: URI lib should be updated to RFC 3986 added

[nobu (Nobuyoshi Nakada)]

• Description updated (diff)

c.f. https://bugs.ruby-lang.org/issues/2542#note-20

[nobu (Nobuyoshi Nakada)]

• Status changed from Open to Assigned
• Assignee set to naruse (Yui NARUSE)

[naruse (Yui NARUSE)]

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

Applied in changeset r48244.

---

• lib/uri/rfc3986_parser.rb (URI::RFC3986_Parser::RFC3986_URI):
  allow '[' and ']' for URI input (and escape). [Bug #10402]

• lib/uri/generic.rb (URI#query=): escape '[', '', and ']'.

[jimpo (Jim Posen)]

The patch caused this bug https://bugs.ruby-lang.org/issues/11275.

[usa (Usaku NAKAMURA)]

• Related to Bug #11275: RFC3986_Parser accepts invalid URIs containing % added