From: akostadinov@...
Date: 2015-02-24T16:27:47+00:00
Subject: [ruby-core:68287] [Ruby trunk - Bug #10900] [Open] GzipReader does not define `#external_encoding`

Issue #10900 has been reported by Aleksandar Kostadinov.

----------------------------------------
Bug #10900: GzipReader does not define `#external_encoding`
https://bugs.ruby-lang.org/issues/10900

* Author: Aleksandar Kostadinov
* Status: Open
* Priority: Normal
* Assignee: 
* ruby -v: 2.0.0p353
* Backport: 2.0.0: UNKNOWN, 2.1: UNKNOWN, 2.2: UNKNOWN
----------------------------------------
Hello,

I want to store/load some **yaml** and other text data in gz for space efficiency. I thought it's most efficient to read/write directly to/from a gzip stream. But seems like the **GzipReader** io object does not support everything that a regular file io object does. I spotted the **#external_encoding** method. So for example to load YAML directly from gzip I use the following method:

~~~
    def load_yaml_from_gz(gz)
      loaded = nil
      Zlib::GzipReader.open(gz) do |f|
        class << f
          def external_encoding
            Encoding::UTF_8
          end
        end
        loaded = YAML.load(f, gz)
      end
      return loaded
    end
~~~

I need the monkey-patching because otherwise the YAML.load method fails for no **#external_encoding** method defined.

To avoid monkey patching just to read some text from a gzip archive I propose that GzipReader implements **#external_encoding** the same way as **File** does.



-- 
https://bugs.ruby-lang.org/