Bug #3594: URI class doesn't do file URL's properly.

• Status: Closed
• Assignee: akira (akira yamada)
• Target version: -
• ruby -v: ruby 1.8.7 (2010-01-10 patchlevel 249) [x86_64-linux]
• Backport: 2.3: UNKNOWN, 2.4: UNKNOWN

[NeilW (Neil Wilson)]

=begin
URI doesn't seem to output file URLs properly where there is an absolute path part.

For example.

require 'uri'

my_url = URI.parse("file:///var/lib/libvirt/images/")
p my_url.to_s

"file:/var/lib/libvirt/images/"

The double slash '//' should always appear according to the file URL specification.
=end

[akr (Akira Tanaka)]

• Project changed from Ruby to Ruby master
• Assignee set to akira (akira yamada)

[shyouhei (Shyouhei Urabe)]

• Status changed from Open to Assigned

[mame (Yusuke Endoh)]

• Description updated (diff)

Is this specified in RFC or any spec?

--
Yusuke Endoh [e-mail]

[naruse (Yui NARUSE)]

mame (Yusuke Endoh) wrote:

Is this specified in RFC or any spec?

There's no available RFC of file URI scheme.

see also:

• http://en.wikipedia.org/wiki/File_URI_scheme
• http://suika.fam.cx/~wakaba/wiki/sw/n/file

[mame (Yusuke Endoh)]

RFC 8089 (The "file" URI Scheme) has been published in 2017. It says:

      file-URI       = file-scheme ":" file-hier-part

      file-scheme    = "file"

      file-hier-part = ( "//" auth-path )
                     / local-path

      auth-path      = [ file-auth ] path-absolute

      local-path     = path-absolute

and RFC 3986 says:

      path-absolute = "/" [ segment-nz *( "/" segment ) ]

So, "file:/var/lib/libvirt/images/" does not conform the spec, I think.

The maintainer of lib/uri.rb is Akira Yamada, who seems inactive. Does anyone want to investigate and fix this issue?

[mame (Yusuke Endoh)]

• Status changed from Assigned to Rejected

Sorry, I misread the spec. The URI is valid because it can be interpreted as file-scheme ":" local-path. So, there is no problem. Closing.

[naruse (Yui NARUSE)]

• Status changed from Rejected to Closed

I found the description in URL spec.
Therefore I fixed it at r60970.