Project

General

Profile

Bug #2688

rubyspec: Net::HTTPHeader#content_range returns a Range object that represents the 'Content-Range' header entry FAILED

Added by mame (Yusuke Endoh) over 10 years ago. Updated over 9 years ago.

Status:
Closed
Priority:
Normal
Target version:
ruby -v:
ruby 1.9.2dev (2010-01-31 trunk 26518) [i686-linux]
Backport:
[ruby-dev:40244]

Description

=begin
なかださん
遠藤です。

class C
include Net::HTTPHeader
def initialize
initialize_http_header({})
end
end
c = C.new
c["Content-Range"] = "bytes 0-499/1234"
p c.content_range

の出力が、r26344 の前は 0..500 で、後は 0..499 になっています。
例によって rubyspec が見つけてくれました。

これはバグ修正でしょうか。
バグ修正だとして、その重要度や影響はどの程度でしょうか。
要するに、1.8.7 やそれ以前にバックポートされるべきだと思いますか。
卜部さんが判断に困っていました。

--
Yusuke ENDOH mame@tsg.ne.jp
=end

#1

Updated by mame (Yusuke Endoh) over 10 years ago

  • Category set to lib
  • Assignee set to nobu (Nobuyoshi Nakada)
  • Priority changed from 3 to Normal
  • Target version set to 1.9.2
  • ruby -v set to ruby 1.9.2dev (2010-01-31 trunk 26518) [i686-linux]

=begin

=end

#2

Updated by nobu (Nobuyoshi Nakada) over 10 years ago

=begin
なかだです。

At Sat, 30 Jan 2010 01:10:11 +0900,
Yusuke ENDOH wrote in [ruby-dev:40244]:

class C
include Net::HTTPHeader
def initialize
initialize_http_header({})
end
end
c = C.new
c["Content-Range"] = "bytes 0-499/1234"
p c.content_range

の出力が、r26344 の前は 0..500 で、後は 0..499 になっています。
例によって rubyspec が見つけてくれました。

これはバグ修正でしょうか。

バグ修正です。0-499で返されるのは先頭500byteですが、str[0..500]
の長さは501byteなので、明らかに矛盾しています。

バグ修正だとして、その重要度や影響はどの程度でしょうか。
要するに、1.8.7 やそれ以前にバックポートされるべきだと思いますか。
卜部さんが判断に困っていました。

少なくとも添付ライブラリやGoogle Code Searchでは、content_range
を使っている例は見つけられませんでした。ほとんど使用されないメソッ
ドなので重要度は低いとも影響は少ないとも言えると思います。

--
--- 僕の前にBugはない。
--- 僕の後ろにBugはできる。
中田 伸悦

=end

#3

Updated by mame (Yusuke Endoh) over 10 years ago

  • Status changed from Open to Closed

=begin
遠藤です。

この件は rubyspec 側で修正したので close します。

--
Yusuke Endoh mame@tsg.ne.jp
=end

Also available in: Atom PDF