Project

General

Profile

Actions

Feature #4551

closed

uri.set_scheme should downcase schemes

Added by drbrain (Eric Hodel) about 13 years ago. Updated about 12 years ago.

Status:
Closed
Target version:
-
[ruby-core:35606]

Description

=begin
From RFC 2396 section 3.1:

"For resiliency, programs interpreting URI should treat upper case letters as equivalent to lower case in scheme names (e.g., allow 'HTTP' as well as 'http')."

Currently programs using URI for HTTP vs HTTPS protocol check which protocol to use with a comparison like:

http.use_ssl = uri.scheme == 'https'

Since URI does not modify the input scheme this makes the check fragile. For example:

URI.parse('httpS://example').scheme # => "httpS"

Since RFC 2396 allows uppercase letters to be treated like lowercase I think #set_scheme should downcase its value.
=end


Files

uri.rb.scheme_downcase.patch (929 Bytes) uri.rb.scheme_downcase.patch drbrain (Eric Hodel), 04/04/2011 07:35 AM

Updated by naruse (Yui NARUSE) about 13 years ago

=begin
Your point is reasonable, but it breaks some cases like
s = 'HTTP://example.com/'
URI(s).to_s == s
=end

Updated by naruse (Yui NARUSE) about 12 years ago

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

Updated by naruse (Yui NARUSE) about 12 years ago

  • Assignee changed from naruse (Yui NARUSE) to drbrain (Eric Hodel)

I agree with this.
Please commit it.

Actions #4

Updated by drbrain (Eric Hodel) about 12 years ago

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

This issue was solved with changeset r35305.
Eric, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.


  • lib/uri/generic.rb (module URI): URI now downcases the scheme to
    follow RFC 2396 section 3.1. [ruby-trunk - Feature #4551]
  • test/uri/test_generic.rb (class URI): Test for above
Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0