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

hsbt_SHiroshi_SHIBA · February 23, 2013, 11:16pm

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

Bug #7924: r39232 以降 net/http で正しく reponse を取得出来ないケースがある

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 の以下のようなコードが動かなくなりました。

github.com

tdiary/tdiary-contrib/blob/master/plugin/flickr.rb#L185


      
          end
          
          
def flickr_slideshow(tag, id = nil)
            id ||= @conf['flickr.id']
            return unless id
            %Q|<iframe align="center" src="http://www.flickr.com/slideShow/index.gne?user_id=#{id}&amp;tags=#{tag}" frameBorder="0" width="500" scrolling="no" height="500"></iframe>|
          end
          
          
def flickr_slideshow_by_set(set_id)
            return unless set_id
            %Q|<iframe align="center" src="http://www.flickr.com/slideShow/index.gne?set_id=#{set_id}" frameBorder="0" width="500" scrolling="no" height="500"></iframe>|
          end
          
          
module Flickr
            class Request < Hash
              def initialize(api_key, secret = nil)
                self['api_key'] = api_key
                @secret = secret
              end
          
          
    def open

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

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

hsbt_SHiroshi_SHIBA · February 23, 2013, 11:46pm

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 を取得出来ないケースがある

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 の以下のようなコードが動かなくなりました。

github.com

tdiary/tdiary-contrib/blob/master/plugin/flickr.rb#L185


      
          end
          
          
def flickr_slideshow(tag, id = nil)
            id ||= @conf['flickr.id']
            return unless id
            %Q|<iframe align="center" src="http://www.flickr.com/slideShow/index.gne?user_id=#{id}&amp;tags=#{tag}" frameBorder="0" width="500" scrolling="no" height="500"></iframe>|
          end
          
          
def flickr_slideshow_by_set(set_id)
            return unless set_id
            %Q|<iframe align="center" src="http://www.flickr.com/slideShow/index.gne?set_id=#{set_id}" frameBorder="0" width="500" scrolling="no" height="500"></iframe>|
          end
          
          
module Flickr
            class Request < Hash
              def initialize(api_key, secret = nil)
                self['api_key'] = api_key
                @secret = secret
              end
          
          
    def open

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

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

hsbt_SHiroshi_SHIBA · February 23, 2013, 11:47pm

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

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

Bug #7924: r39232 以降 net/http で正しく reponse を取得出来ないケースがある

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 の以下のようなコードが動かなくなりました。

github.com

tdiary/tdiary-contrib/blob/master/plugin/flickr.rb#L185


      
          end
          
          
def flickr_slideshow(tag, id = nil)
            id ||= @conf['flickr.id']
            return unless id
            %Q|<iframe align="center" src="http://www.flickr.com/slideShow/index.gne?user_id=#{id}&amp;tags=#{tag}" frameBorder="0" width="500" scrolling="no" height="500"></iframe>|
          end
          
          
def flickr_slideshow_by_set(set_id)
            return unless set_id
            %Q|<iframe align="center" src="http://www.flickr.com/slideShow/index.gne?set_id=#{set_id}" frameBorder="0" width="500" scrolling="no" height="500"></iframe>|
          end
          
          
module Flickr
            class Request < Hash
              def initialize(api_key, secret = nil)
                self['api_key'] = api_key
                @secret = secret
              end
          
          
    def open

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

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

hsbt_SHiroshi_SHIBA · February 23, 2013, 11:58pm

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 を取得出来ないケースがある

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 の以下のようなコードが動かなくなりました。

github.com

tdiary/tdiary-contrib/blob/master/plugin/flickr.rb#L185


      
          end
          
          
def flickr_slideshow(tag, id = nil)
            id ||= @conf['flickr.id']
            return unless id
            %Q|<iframe align="center" src="http://www.flickr.com/slideShow/index.gne?user_id=#{id}&amp;tags=#{tag}" frameBorder="0" width="500" scrolling="no" height="500"></iframe>|
          end
          
          
def flickr_slideshow_by_set(set_id)
            return unless set_id
            %Q|<iframe align="center" src="http://www.flickr.com/slideShow/index.gne?set_id=#{set_id}" frameBorder="0" width="500" scrolling="no" height="500"></iframe>|
          end
          
          
module Flickr
            class Request < Hash
              def initialize(api_key, secret = nil)
                self['api_key'] = api_key
                @secret = secret
              end
          
          
    def open

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

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

hsbt_SHiroshi_SHIBA · February 24, 2013, 12:06am

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 を取得出来ないケースがある

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 の以下のようなコードが動かなくなりました。

github.com

tdiary/tdiary-contrib/blob/master/plugin/flickr.rb#L185


      
          end
          
          
def flickr_slideshow(tag, id = nil)
            id ||= @conf['flickr.id']
            return unless id
            %Q|<iframe align="center" src="http://www.flickr.com/slideShow/index.gne?user_id=#{id}&amp;tags=#{tag}" frameBorder="0" width="500" scrolling="no" height="500"></iframe>|
          end
          
          
def flickr_slideshow_by_set(set_id)
            return unless set_id
            %Q|<iframe align="center" src="http://www.flickr.com/slideShow/index.gne?set_id=#{set_id}" frameBorder="0" width="500" scrolling="no" height="500"></iframe>|
          end
          
          
module Flickr
            class Request < Hash
              def initialize(api_key, secret = nil)
                self['api_key'] = api_key
                @secret = secret
              end
          
          
    def open

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

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

hsbt_SHiroshi_SHIBA · February 24, 2013, 12:03am

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

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

translation of this issue: Backport #7831: Net::HTTP does not allow users to handle Content-Encoding in responses like 1.x - Backport200 - Ruby Issue Tracking System

Bug #7924: r39232 以降 net/http で正しく reponse を取得出来ないケースがある

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 の以下のようなコードが動かなくなりました。

github.com

tdiary/tdiary-contrib/blob/master/plugin/flickr.rb#L185


      
          end
          
          
def flickr_slideshow(tag, id = nil)
            id ||= @conf['flickr.id']
            return unless id
            %Q|<iframe align="center" src="http://www.flickr.com/slideShow/index.gne?user_id=#{id}&amp;tags=#{tag}" frameBorder="0" width="500" scrolling="no" height="500"></iframe>|
          end
          
          
def flickr_slideshow_by_set(set_id)
            return unless set_id
            %Q|<iframe align="center" src="http://www.flickr.com/slideShow/index.gne?set_id=#{set_id}" frameBorder="0" width="500" scrolling="no" height="500"></iframe>|
          end
          
          
module Flickr
            class Request < Hash
              def initialize(api_key, secret = nil)
                self['api_key'] = api_key
                @secret = secret
              end
          
          
    def open

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

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

hsbt_SHiroshi_SHIBA · February 24, 2013, 12:15am

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

this patch seems good.

Bug #7924: r39232 以降 net/http で正しく reponse を取得出来ないケースがある

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 の以下のようなコードが動かなくなりました。

github.com

tdiary/tdiary-contrib/blob/master/plugin/flickr.rb#L185


      
          end
          
          
def flickr_slideshow(tag, id = nil)
            id ||= @conf['flickr.id']
            return unless id
            %Q|<iframe align="center" src="http://www.flickr.com/slideShow/index.gne?user_id=#{id}&amp;tags=#{tag}" frameBorder="0" width="500" scrolling="no" height="500"></iframe>|
          end
          
          
def flickr_slideshow_by_set(set_id)
            return unless set_id
            %Q|<iframe align="center" src="http://www.flickr.com/slideShow/index.gne?set_id=#{set_id}" frameBorder="0" width="500" scrolling="no" height="500"></iframe>|
          end
          
          
module Flickr
            class Request < Hash
              def initialize(api_key, secret = nil)
                self['api_key'] = api_key
                @secret = secret
              end
          
          
    def open

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

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

hsbt_SHiroshi_SHIBA · February 24, 2013, 12:07am

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 を取得出来ないケースがある

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 の以下のようなコードが動かなくなりました。

github.com

tdiary/tdiary-contrib/blob/master/plugin/flickr.rb#L185


      
          end
          
          
def flickr_slideshow(tag, id = nil)
            id ||= @conf['flickr.id']
            return unless id
            %Q|<iframe align="center" src="http://www.flickr.com/slideShow/index.gne?user_id=#{id}&amp;tags=#{tag}" frameBorder="0" width="500" scrolling="no" height="500"></iframe>|
          end
          
          
def flickr_slideshow_by_set(set_id)
            return unless set_id
            %Q|<iframe align="center" src="http://www.flickr.com/slideShow/index.gne?set_id=#{set_id}" frameBorder="0" width="500" scrolling="no" height="500"></iframe>|
          end
          
          
module Flickr
            class Request < Hash
              def initialize(api_key, secret = nil)
                self['api_key'] = api_key
                @secret = secret
              end
          
          
    def open

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

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

hsbt_SHiroshi_SHIBA · February 24, 2013, 1:38am

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

Please please make a test. for preventing regression.

Bug #7924: r39232 以降 net/http で正しく reponse を取得出来ないケースがある

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 の以下のようなコードが動かなくなりました。

github.com

tdiary/tdiary-contrib/blob/master/plugin/flickr.rb#L185


      
          end
          
          
def flickr_slideshow(tag, id = nil)
            id ||= @conf['flickr.id']
            return unless id
            %Q|<iframe align="center" src="http://www.flickr.com/slideShow/index.gne?user_id=#{id}&amp;tags=#{tag}" frameBorder="0" width="500" scrolling="no" height="500"></iframe>|
          end
          
          
def flickr_slideshow_by_set(set_id)
            return unless set_id
            %Q|<iframe align="center" src="http://www.flickr.com/slideShow/index.gne?set_id=#{set_id}" frameBorder="0" width="500" scrolling="no" height="500"></iframe>|
          end
          
          
module Flickr
            class Request < Hash
              def initialize(api_key, secret = nil)
                self['api_key'] = api_key
                @secret = secret
              end
          
          
    def open

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

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

hsbt_SHiroshi_SHIBA · February 24, 2013, 1:26am

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 を取得出来ないケースがある

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 の以下のようなコードが動かなくなりました。

github.com

tdiary/tdiary-contrib/blob/master/plugin/flickr.rb#L185


      
          end
          
          
def flickr_slideshow(tag, id = nil)
            id ||= @conf['flickr.id']
            return unless id
            %Q|<iframe align="center" src="http://www.flickr.com/slideShow/index.gne?user_id=#{id}&amp;tags=#{tag}" frameBorder="0" width="500" scrolling="no" height="500"></iframe>|
          end
          
          
def flickr_slideshow_by_set(set_id)
            return unless set_id
            %Q|<iframe align="center" src="http://www.flickr.com/slideShow/index.gne?set_id=#{set_id}" frameBorder="0" width="500" scrolling="no" height="500"></iframe>|
          end
          
          
module Flickr
            class Request < Hash
              def initialize(api_key, secret = nil)
                self['api_key'] = api_key
                @secret = secret
              end
          
          
    def open

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

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

hsbt_SHiroshi_SHIBA · February 24, 2013, 1:54am

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 を取得出来ないケースがある

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 の以下のようなコードが動かなくなりました。

github.com

tdiary/tdiary-contrib/blob/master/plugin/flickr.rb#L185


      
          end
          
          
def flickr_slideshow(tag, id = nil)
            id ||= @conf['flickr.id']
            return unless id
            %Q|<iframe align="center" src="http://www.flickr.com/slideShow/index.gne?user_id=#{id}&amp;tags=#{tag}" frameBorder="0" width="500" scrolling="no" height="500"></iframe>|
          end
          
          
def flickr_slideshow_by_set(set_id)
            return unless set_id
            %Q|<iframe align="center" src="http://www.flickr.com/slideShow/index.gne?set_id=#{set_id}" frameBorder="0" width="500" scrolling="no" height="500"></iframe>|
          end
          
          
module Flickr
            class Request < Hash
              def initialize(api_key, secret = nil)
                self['api_key'] = api_key
                @secret = secret
              end
          
          
    def open

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

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

hsbt_SHiroshi_SHIBA · February 24, 2013, 6:48am

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

–
Yusuke E. [email protected]

Bug #7924: r39232 以降 net/http で正しく reponse を取得出来ないケースがある

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 の以下のようなコードが動かなくなりました。

github.com

tdiary/tdiary-contrib/blob/master/plugin/flickr.rb#L185


      
          end
          
          
def flickr_slideshow(tag, id = nil)
            id ||= @conf['flickr.id']
            return unless id
            %Q|<iframe align="center" src="http://www.flickr.com/slideShow/index.gne?user_id=#{id}&amp;tags=#{tag}" frameBorder="0" width="500" scrolling="no" height="500"></iframe>|
          end
          
          
def flickr_slideshow_by_set(set_id)
            return unless set_id
            %Q|<iframe align="center" src="http://www.flickr.com/slideShow/index.gne?set_id=#{set_id}" frameBorder="0" width="500" scrolling="no" height="500"></iframe>|
          end
          
          
module Flickr
            class Request < Hash
              def initialize(api_key, secret = nil)
                self['api_key'] = api_key
                @secret = secret
              end
          
          
    def open

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

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

hsbt_SHiroshi_SHIBA · February 24, 2013, 7:21am

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

Also, r39464 for ruby_2_0_0 branch.

Bug #7924: r39232 以降 net/http で正しく reponse を取得出来ないケースがある

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 の以下のようなコードが動かなくなりました。

github.com

tdiary/tdiary-contrib/blob/master/plugin/flickr.rb#L185


      
          end
          
          
def flickr_slideshow(tag, id = nil)
            id ||= @conf['flickr.id']
            return unless id
            %Q|<iframe align="center" src="http://www.flickr.com/slideShow/index.gne?user_id=#{id}&amp;tags=#{tag}" frameBorder="0" width="500" scrolling="no" height="500"></iframe>|
          end
          
          
def flickr_slideshow_by_set(set_id)
            return unless set_id
            %Q|<iframe align="center" src="http://www.flickr.com/slideShow/index.gne?set_id=#{set_id}" frameBorder="0" width="500" scrolling="no" height="500"></iframe>|
          end
          
          
module Flickr
            class Request < Hash
              def initialize(api_key, secret = nil)
                self['api_key'] = api_key
                @secret = secret
              end
          
          
    def open

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

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

hsbt_SHiroshi_SHIBA · February 24, 2013, 2:04am

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.

hsbt_SHiroshi_SHIBA · February 24, 2013, 4:26pm

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 を取得出来ないケースがある

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 の以下のようなコードが動かなくなりました。

github.com

tdiary/tdiary-contrib/blob/master/plugin/flickr.rb#L185


      
          end
          
          
def flickr_slideshow(tag, id = nil)
            id ||= @conf['flickr.id']
            return unless id
            %Q|<iframe align="center" src="http://www.flickr.com/slideShow/index.gne?user_id=#{id}&amp;tags=#{tag}" frameBorder="0" width="500" scrolling="no" height="500"></iframe>|
          end
          
          
def flickr_slideshow_by_set(set_id)
            return unless set_id
            %Q|<iframe align="center" src="http://www.flickr.com/slideShow/index.gne?set_id=#{set_id}" frameBorder="0" width="500" scrolling="no" height="500"></iframe>|
          end
          
          
module Flickr
            class Request < Hash
              def initialize(api_key, secret = nil)
                self['api_key'] = api_key
                @secret = secret
              end
          
          
    def open

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

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

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

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

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

translation of this issue: Backport #7831: Net::HTTP does not allow users to handle Content-Encoding in responses like 1.x - Backport200 - Ruby Issue Tracking System

this patch seems good.

assigned to Naruse-san for approval

Please please make a test. for preventing regression.

Is it sufficient?

– Yusuke E. [email protected]

Also, r39464 for ruby_2_0_0 branch.

I see. Thank you all and Happy Ruby 2.0!

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

–
Yusuke E. [email protected]

I see.
Thank you all and Happy Ruby 2.0!