[ruby-trunk - Bug #7924][Open] r39232 以降 net/http で正しく reponse を取得出来ないケースがある

Issue #7924 has been reported by hsbt (Hiroshi SHIBATA).


Bug #7924: r39232 以降 net/http で正しく reponse を取得出来ないケースがある
https://bugs.ruby-lang.org/issues/7924

Author: hsbt (Hiroshi SHIBATA)
Status: Open
Priority: Immediate
Assignee: naruse (Yui NARUSE)
Category: core
Target version: 2.0.0
ruby -v: ruby 2.0.0dev (2013-02-24 trunk 39439) [x86_64-darwin12.2.1]

r39232 以降、tDiary の以下のようなコードが動かなくなりました。

https://github.com/tdiary/tdiary-contrib/blob/master/plugin/flickr.rb#L185

単純に net/http を使用して flickr.com から xml を取得するコードですが、r39232で加えた
変更により、本来 inflate されるべき response.body が gzip の状態のままになっています。

response を inflate するケースの考慮漏れのような気がします。

Issue #7924 has been updated by kazuhiko (Kazuhiko Shiozaki).

accept-encodingの無指定時に、http.rbのget()でaccept-encodingをつけていて、そのためにgeneric_request.rbが「あ、指定してるんならdecode_content=falseでいいよね」としているのが原因。

accept-encodingの無指定時に"gzip;q=1.0,deflate;q=0.6,identity;q=0.3"をセットするコードが、http.rbのget()とhttp/generic_request.rbのinitialize()の両方にあって、後者は前者の可能性を考慮していない。なので、この問題はputでは起きない。

その二箇所以外にaccept-encodingをセットしているコードはなさそうなので、http.rbのget()から「accept-encodingの無指定時に〜」を除けばなおるはず。

Bug #7924: r39232 以降 net/http で正しく reponse を取得出来ないケースがある
https://bugs.ruby-lang.org/issues/7924#change-36837

Author: hsbt (Hiroshi SHIBATA)
Status: Open
Priority: Immediate
Assignee: naruse (Yui NARUSE)
Category: core
Target version: 2.0.0
ruby -v: ruby 2.0.0dev (2013-02-24 trunk 39439) [x86_64-darwin12.2.1]

r39232 以降、tDiary の以下のようなコードが動かなくなりました。

https://github.com/tdiary/tdiary-contrib/blob/master/plugin/flickr.rb#L185

単純に net/http を使用して flickr.com から xml を取得するコードですが、r39232で加えた
変更により、本来 inflate されるべき response.body が gzip の状態のままになっています。

response を inflate するケースの考慮漏れのような気がします。

Issue #7924 has been updated by kazuhiko (Kazuhiko Shiozaki).

問題になった r32932 ruby_2_0_0 にすでにマージされているので、このまま2.0.0を出すとけっこう影響の大きいバグだと思います。


Bug #7924: r39232 以降 net/http で正しく reponse を取得出来ないケースがある
https://bugs.ruby-lang.org/issues/7924#change-36838

Author: hsbt (Hiroshi SHIBATA)
Status: Open
Priority: Immediate
Assignee: naruse (Yui NARUSE)
Category: core
Target version: 2.0.0
ruby -v: ruby 2.0.0dev (2013-02-24 trunk 39439) [x86_64-darwin12.2.1]

r39232 以降、tDiary の以下のようなコードが動かなくなりました。

https://github.com/tdiary/tdiary-contrib/blob/master/plugin/flickr.rb#L185

単純に net/http を使用して flickr.com から xml を取得するコードですが、r39232で加えた
変更により、本来 inflate されるべき response.body が gzip の状態のままになっています。

response を inflate するケースの考慮漏れのような気がします。

Issue #7924 has been updated by drbrain (Eric H.).

I found the bug, I will post a patch with a test momentarily.

Bug #7924: r39232 以降 net/http で正しく reponse を取得出来ないケースがある
https://bugs.ruby-lang.org/issues/7924#change-36842

Author: hsbt (Hiroshi SHIBATA)
Status: Open
Priority: Immediate
Assignee: naruse (Yui NARUSE)
Category: core
Target version: 2.0.0
ruby -v: ruby 2.0.0dev (2013-02-24 trunk 39439) [x86_64-darwin12.2.1]

r39232 以降、tDiary の以下のようなコードが動かなくなりました。

https://github.com/tdiary/tdiary-contrib/blob/master/plugin/flickr.rb#L185

単純に net/http を使用して flickr.com から xml を取得するコードですが、r39232で加えた
変更により、本来 inflate されるべき response.body が gzip の状態のままになっています。

response を inflate するケースの考慮漏れのような気がします。

Issue #7924 has been updated by drbrain (Eric H.).

File net.http.bug7924.patch added

The attached patch removes the duplicated header setting in
Net::HTTP#get and adds a test.

I double checked net/http* for use of accept-encoding or HAVE_ZLIB, now
it only exists in Net::HTTPGenericRequest and Net::HTTPResponse.

Bug #7924: r39232 以降 net/http で正しく reponse を取得出来ないケースがある
https://bugs.ruby-lang.org/issues/7924#change-36844

Author: hsbt (Hiroshi SHIBATA)
Status: Open
Priority: Immediate
Assignee: drbrain (Eric H.)
Category: core
Target version: 2.0.0
ruby -v: ruby 2.0.0dev (2013-02-24 trunk 39439) [x86_64-darwin12.2.1]

r39232 以降、tDiary の以下のようなコードが動かなくなりました。

https://github.com/tdiary/tdiary-contrib/blob/master/plugin/flickr.rb#L185

単純に net/http を使用して flickr.com から xml を取得するコードですが、r39232で加えた
変更により、本来 inflate されるべき response.body が gzip の状態のままになっています。

response を inflate するケースの考慮漏れのような気がします。

Issue #7924 has been updated by hsbt (Hiroshi SHIBATA).

Assignee changed from naruse (Yui NARUSE) to drbrain (Eric H.)

translation of this issue: http://bugs.ruby-lang.org/issues/7831#note-10

Bug #7924: r39232 以降 net/http で正しく reponse を取得出来ないケースがある
https://bugs.ruby-lang.org/issues/7924#change-36843

Author: hsbt (Hiroshi SHIBATA)
Status: Open
Priority: Immediate
Assignee: drbrain (Eric H.)
Category: core
Target version: 2.0.0
ruby -v: ruby 2.0.0dev (2013-02-24 trunk 39439) [x86_64-darwin12.2.1]

r39232 以降、tDiary の以下のようなコードが動かなくなりました。

https://github.com/tdiary/tdiary-contrib/blob/master/plugin/flickr.rb#L185

単純に net/http を使用して flickr.com から xml を取得するコードですが、r39232で加えた
変更により、本来 inflate されるべき response.body が gzip の状態のままになっています。

response を inflate するケースの考慮漏れのような気がします。

Issue #7924 has been updated by hsbt (Hiroshi SHIBATA).

this patch seems good.

Bug #7924: r39232 以降 net/http で正しく reponse を取得出来ないケースがある
https://bugs.ruby-lang.org/issues/7924#change-36849

Author: hsbt (Hiroshi SHIBATA)
Status: Open
Priority: Immediate
Assignee: naruse (Yui NARUSE)
Category: core
Target version: 2.0.0
ruby -v: ruby 2.0.0dev (2013-02-24 trunk 39439) [x86_64-darwin12.2.1]

r39232 以降、tDiary の以下のようなコードが動かなくなりました。

https://github.com/tdiary/tdiary-contrib/blob/master/plugin/flickr.rb#L185

単純に net/http を使用して flickr.com から xml を取得するコードですが、r39232で加えた
変更により、本来 inflate されるべき response.body が gzip の状態のままになっています。

response を inflate するケースの考慮漏れのような気がします。

Issue #7924 has been updated by drbrain (Eric H.).

Assignee changed from drbrain (Eric H.) to naruse (Yui NARUSE)

assigned to Naruse-san for approval

Bug #7924: r39232 以降 net/http で正しく reponse を取得出来ないケースがある
https://bugs.ruby-lang.org/issues/7924#change-36846

Author: hsbt (Hiroshi SHIBATA)
Status: Open
Priority: Immediate
Assignee: naruse (Yui NARUSE)
Category: core
Target version: 2.0.0
ruby -v: ruby 2.0.0dev (2013-02-24 trunk 39439) [x86_64-darwin12.2.1]

r39232 以降、tDiary の以下のようなコードが動かなくなりました。

https://github.com/tdiary/tdiary-contrib/blob/master/plugin/flickr.rb#L185

単純に net/http を使用して flickr.com から xml を取得するコードですが、r39232で加えた
変更により、本来 inflate されるべき response.body が gzip の状態のままになっています。

response を inflate するケースの考慮漏れのような気がします。

Issue #7924 has been updated by kosaki (Motohiro KOSAKI).

Please please make a test. for preventing regression.

Bug #7924: r39232 以降 net/http で正しく reponse を取得出来ないケースがある
https://bugs.ruby-lang.org/issues/7924#change-36853

Author: hsbt (Hiroshi SHIBATA)
Status: Open
Priority: Immediate
Assignee: naruse (Yui NARUSE)
Category: core
Target version: 2.0.0
ruby -v: ruby 2.0.0dev (2013-02-24 trunk 39439) [x86_64-darwin12.2.1]

r39232 以降、tDiary の以下のようなコードが動かなくなりました。

https://github.com/tdiary/tdiary-contrib/blob/master/plugin/flickr.rb#L185

単純に net/http を使用して flickr.com から xml を取得するコードですが、r39232で加えた
変更により、本来 inflate されるべき response.body が gzip の状態のままになっています。

response を inflate するケースの考慮漏れのような気がします。

Issue #7924 has been updated by nagachika (Tomoyuki C.).

make check all green with the patch in my environment
[x86_64-darwin12.2.0].


Bug #7924: r39232 以降 net/http で正しく reponse を取得出来ないケースがある
https://bugs.ruby-lang.org/issues/7924#change-36852

Author: hsbt (Hiroshi SHIBATA)
Status: Open
Priority: Immediate
Assignee: naruse (Yui NARUSE)
Category: core
Target version: 2.0.0
ruby -v: ruby 2.0.0dev (2013-02-24 trunk 39439) [x86_64-darwin12.2.1]

r39232 以降、tDiary の以下のようなコードが動かなくなりました。

https://github.com/tdiary/tdiary-contrib/blob/master/plugin/flickr.rb#L185

単純に net/http を使用して flickr.com から xml を取得するコードですが、r39232で加えた
変更により、本来 inflate されるべき response.body が gzip の状態のままになっています。

response を inflate するケースの考慮漏れのような気がします。

Issue #7924 has been updated by drbrain (Eric H.).

Kosaki-san, the patch contains a test to ensure that decode_content is
enabled when using Net::HTTP#get. The behavior of decode_content is
already tested in r39232.

Is it sufficient?

Bug #7924: r39232 以降 net/http で正しく reponse を取得出来ないケースがある
https://bugs.ruby-lang.org/issues/7924#change-36854

Author: hsbt (Hiroshi SHIBATA)
Status: Open
Priority: Immediate
Assignee: naruse (Yui NARUSE)
Category: core
Target version: 2.0.0
ruby -v: ruby 2.0.0dev (2013-02-24 trunk 39439) [x86_64-darwin12.2.1]

r39232 以降、tDiary の以下のようなコードが動かなくなりました。

https://github.com/tdiary/tdiary-contrib/blob/master/plugin/flickr.rb#L185

単純に net/http を使用して flickr.com から xml を取得するコードですが、r39232で加えた
変更により、本来 inflate されるべき response.body が gzip の状態のままになっています。

response を inflate するケースの考慮漏れのような気がします。

Issue #7924 has been updated by mame (Yusuke E.).

Status changed from Open to Assigned
Assignee changed from naruse (Yui NARUSE) to drbrain (Eric H.)

Looks serious. Got ack from hsbt and nagachika.
Drbrain, could you please commit it to trunk and ruby_2_0_0 in advance?
I’ll ask naruse-san to do post-review, if he could wake up early enough
:wink:


Yusuke E. [email protected]

Bug #7924: r39232 以降 net/http で正しく reponse を取得出来ないケースがある
https://bugs.ruby-lang.org/issues/7924#change-36892

Author: hsbt (Hiroshi SHIBATA)
Status: Assigned
Priority: Immediate
Assignee: drbrain (Eric H.)
Category: core
Target version: 2.0.0
ruby -v: ruby 2.0.0dev (2013-02-24 trunk 39439) [x86_64-darwin12.2.1]

r39232 以降、tDiary の以下のようなコードが動かなくなりました。

https://github.com/tdiary/tdiary-contrib/blob/master/plugin/flickr.rb#L185

単純に net/http を使用して flickr.com から xml を取得するコードですが、r39232で加えた
変更により、本来 inflate されるべき response.body が gzip の状態のままになっています。

response を inflate するケースの考慮漏れのような気がします。

Issue #7924 has been updated by drbrain (Eric H.).

Also, r39464 for ruby_2_0_0 branch.

Bug #7924: r39232 以降 net/http で正しく reponse を取得出来ないケースがある
https://bugs.ruby-lang.org/issues/7924#change-36894

Author: hsbt (Hiroshi SHIBATA)
Status: Closed
Priority: Immediate
Assignee: drbrain (Eric H.)
Category: core
Target version: 2.0.0
ruby -v: ruby 2.0.0dev (2013-02-24 trunk 39439) [x86_64-darwin12.2.1]

r39232 以降、tDiary の以下のようなコードが動かなくなりました。

https://github.com/tdiary/tdiary-contrib/blob/master/plugin/flickr.rb#L185

単純に net/http を使用して flickr.com から xml を取得するコードですが、r39232で加えた
変更により、本来 inflate されるべき response.body が gzip の状態のままになっています。

response を inflate するケースの考慮漏れのような気がします。

Kosaki-san, the patch contains a test to ensure that decode_content is enabled
when using Net::HTTP#get. The behavior of decode_content is already tested in
r39232.

Is it sufficient?

Oops, I missed that. sorry for noise.

Issue #7924 has been updated by naruse (Yui NARUSE).

I see.
Thank you all and Happy Ruby 2.0!

Bug #7924: r39232 以降 net/http で正しく reponse を取得出来ないケースがある
https://bugs.ruby-lang.org/issues/7924#change-36941

Author: hsbt (Hiroshi SHIBATA)
Status: Closed
Priority: Immediate
Assignee: drbrain (Eric H.)
Category: core
Target version: 2.0.0
ruby -v: ruby 2.0.0dev (2013-02-24 trunk 39439) [x86_64-darwin12.2.1]

r39232 以降、tDiary の以下のようなコードが動かなくなりました。

https://github.com/tdiary/tdiary-contrib/blob/master/plugin/flickr.rb#L185

単純に net/http を使用して flickr.com から xml を取得するコードですが、r39232で加えた
変更により、本来 inflate されるべき response.body が gzip の状態のままになっています。

response を inflate するケースの考慮漏れのような気がします。

This forum is not affiliated to the Ruby language, Ruby on Rails framework, nor any Ruby applications discussed here.

| Privacy Policy | Terms of Service | Remote Ruby Jobs