faraday-gzip 2.0.1 → 3.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 44ad7c4e9c57927b182f8da9719aeb71455cd664607582af5be4440d660c9015
-  data.tar.gz: cfd6528ad8d82ce76f1b62cddbd63e2af086dab192a62c1dc14e6603ae5f5a18
+  metadata.gz: 5183c6f188272ea7e213adf03bb2d637c348e9f273b1b8829255f135a95b8c92
+  data.tar.gz: ce73797d7aa191ef07c376af0193ea704a653237146affec2806031194d064c7
 SHA512:
-  metadata.gz: 61a096ffcf59316c2c7a1e9be64f159e879887a4a56dbfe495268d4c70a7636944885307277dd62c1a705df063676a6526921a65d01a0436cf01331349a01828
-  data.tar.gz: 174bd0148dfcdc7e33acddb9c086629b37f8179905081c45ea4c35fb2e916c6f07c994a8065fce6b900aca34de1624e162330b185766f8c18a9d3edb6098c6f3
+  metadata.gz: 8f9add78a8ff2c274ef021ae4e8ebfc1f525847ba4d05ca6ccf52cba1bc9f9d891e4627a354fdbf724393cb420e1225c33521d55f15f0c292490c8466e6ba202
+  data.tar.gz: 2edf0cd82a52d2bf33d9024ed3363b4735dc7a7645f847bf1aca250947d14663d696d2bc64bb10fc83019a677a2b701d933766ed1afcc7f76d554a30b4e6c5f9

data/CHANGELOG.md

@@ -1,5 +1,24 @@
 # Changelog
 
+## 3.0.4 (06-Apr-2025)
+
+* Require StringIO that might not always be readily available
+
+## 3.0.3 (25-Feb-2025)
+
+* Minor code fixes, make some methods more solid
+
+## 3.0.1 / 3.0.2 (01-Nov-2024)
+
+* Minor fixes in gemspec
+
+## 3.0.0 (29-Oct-2024)
+
+* **Breaking change**: Drop support for Ruby 2, require 3.0+
+* **Breaking change**: Drop support for Faraday v1. If you need to support Faraday v1, stay on [faraday-gzip version 2](https://github.com/bodrovis/faraday-gzip/tree/v2).
+* Various code tweaks
+* Remove JRuby 9.3 from CI matrix
+
 ## 2.0.1 (02-Jan-2024)
 
 * Handle cases when body is `nil` (thanks, @bendangelo)

data/README.md

@@ -2,54 +2,58 @@
 
 ![CI](https://github.com/bodrovis/faraday-gzip/actions/workflows/ci.yaml/badge.svg)
 [![Gem](https://img.shields.io/gem/v/faraday-gzip.svg?style=flat-square)](https://rubygems.org/gems/faraday-gzip)
+![Gem Total Downloads](https://img.shields.io/gem/dt/faraday-gzip)
 
-The `Gzip` middleware for Faraday 1 and 2 adds the necessary `Accept-Encoding` headers and automatically decompresses the response. If the "Accept-Encoding" header wasn't set in the request, this sets it to "gzip,deflate" and appropriately handles the compressed response from the server. This resembles what Ruby does internally in Net::HTTP#get. If [Brotli](https://github.com/miyucy/brotli) is added to the Gemfile, it will also add "br" to the header.
+The `Gzip` middleware for Faraday 1 and 2 adds the necessary `Accept-Encoding` headers and automatically decompresses the response. If the "Accept-Encoding" header isn't set in the request, it defaults to `gzip,deflate` and appropriately handles the server's compressed response. This functionality resembles what Ruby does internally in `Net::HTTP#get`. If [Brotli](https://github.com/miyucy/brotli) is included in your Gemfile, the middleware also adds `br` to the header for Brotli support.
 
 ## Prerequisites
 
-This gem is tested with Ruby 2.6+ and JRuby 9.3+. Faraday 1 and 2 is supported.
+* faraday-gzip v3 supports only Faraday v2 and is tested with Ruby 3.0+ and JRuby 9.4
+* [faraday-gzip v2](https://github.com/bodrovis/faraday-gzip/tree/v2) supports Faraday v1 and v2 and is tested with Ruby 2.7+ and JRuby 9.4.
 
 ## Installation
 
 Add this line to your application's Gemfile:
 
 ```ruby
-gem 'faraday-gzip'
+gem 'faraday-gzip', '~> 3'
 ```
 
 And then execute:
 
-```shell
+```
 bundle install
 ```
 
 Or install it yourself as:
 
-```shell
+```
 gem install faraday-gzip
 ```
 
 ## Usage
 
+To enable the middleware in your Faraday connection, add it as shown below:
+
 ```ruby
-require 'faraday/gzip' # <=== add this line
+require 'faraday/gzip' # <=== Add this line
 
 conn = Faraday.new(...) do |f|
-  f.request :gzip # <=== add this line
-  #...
+  f.request :gzip # <=== Add this line
+  # Additional configuration...
 end
 ```
 
 ## Development
 
-After checking out the repo, run `bin/setup` to install dependencies.
-
-Then, run `bin/test` to run the tests.
-
-To install this gem onto your local machine, run `rake build`.
+To contribute or make changes:
 
-To release a new version, make a commit with a message such as "Bumped to 0.0.2" and then run `rake release`.
-See how it works [here](https://bundler.io/guides/creating_gem.html#releasing-the-gem).
+* Clone the repo
+* Run `bundle` to install dependencies
+* Implement your feature
+* Write and run tests using `rspec .`
+* Use rake build to build the gem locally if needed
+* Create a new PR with your changes
 
 ## Contributing
 
@@ -57,4 +61,4 @@ Bug reports and pull requests are welcome on GitHub.
 
 ## License
 
-The gem is available as open source under the terms of the [MIT License](https://opensource.org/licenses/MIT).
+This gem is available as open source under the terms of the [MIT License](https://opensource.org/licenses/MIT).

data/lib/faraday/gzip/middleware.rb

@@ -1,16 +1,19 @@
 # frozen_string_literal: true
 
 require 'zlib'
+require 'stringio'
 
+# Middleware to automatically decompress response bodies. If the
+# "Accept-Encoding" header wasn't set in the request, this sets it to
+# "gzip,deflate" and appropriately handles the compressed response from the
+# server. This resembles what Ruby 1.9+ does internally in Net::HTTP#get.
+# Based on https://github.com/lostisland/faraday_middleware/blob/main/lib/faraday_middleware/gzip.rb
 module Faraday
+  # Main module
   module Gzip
-    # Middleware to automatically decompress response bodies. If the
-    # "Accept-Encoding" header wasn't set in the request, this sets it to
-    # "gzip,deflate" and appropriately handles the compressed response from the
-    # server. This resembles what Ruby 1.9+ does internally in Net::HTTP#get.
-    # Based on https://github.com/lostisland/faraday_middleware/blob/main/lib/faraday_middleware/gzip.rb
-
+    # Faraday middleware for decompression
     class Middleware < Faraday::Middleware
+      # System method required by Faraday
       def self.optional_dependency(lib = nil)
         lib ? require(lib) : yield
         true
@@ -20,6 +23,8 @@ module Faraday
 
       BROTLI_SUPPORTED = optional_dependency 'brotli'
 
+      # Returns supported encodings, adds brotli if the corresponding
+      # dependency is present
       def self.supported_encodings
         encodings = %w[gzip deflate]
         encodings << 'br' if BROTLI_SUPPORTED
@@ -31,37 +36,46 @@ module Faraday
       CONTENT_LENGTH = 'Content-Length'
       SUPPORTED_ENCODINGS = supported_encodings.join(',').freeze
 
+      # Main method to process the response
       def call(env)
         env[:request_headers][ACCEPT_ENCODING] ||= SUPPORTED_ENCODINGS
+
         @app.call(env).on_complete do |response_env|
-          if empty_body?(response_env)
-            reset_body(response_env) { |body| raw_body(body) }
-          else
-            case response_env[:response_headers][CONTENT_ENCODING]
-            when 'gzip'
-              reset_body(response_env) { |body| uncompress_gzip(body) }
-            when 'deflate'
-              reset_body(response_env) { |body| inflate(body) }
-            when 'br'
-              reset_body(response_env) { |body| brotli_inflate(body) }
-            end
-          end
+          reset_body(response_env, find_processor(response_env))
+        end
+      end
+
+      # Finds a proper processor
+      def find_processor(response_env)
+        if empty_body?(response_env)
+          ->(body) { raw_body(body) }
+        else
+          processors[response_env[:response_headers][CONTENT_ENCODING]]
         end
       end
 
-      def reset_body(env)
-        env[:body] = yield(env[:body])
+      # Calls the proper processor to decompress body
+      def reset_body(env, processor)
+        return if processor.nil?
+
+        env[:body] = processor.call(env[:body])
         env[:response_headers].delete(CONTENT_ENCODING)
 
         env[:response_headers][CONTENT_LENGTH] = env[:body].nil? ? 0 : env[:body].length
       end
 
+      # Process gzip
       def uncompress_gzip(body)
         io = StringIO.new(body)
         gzip_reader = Zlib::GzipReader.new(io, encoding: 'ASCII-8BIT')
-        gzip_reader.read
+        begin
+          gzip_reader.read
+        ensure
+          gzip_reader.close
+        end
       end
 
+      # Process deflate
       def inflate(body)
         # Inflate as a DEFLATE (RFC 1950+RFC 1951) stream
         Zlib::Inflate.inflate(body)
@@ -76,10 +90,12 @@ module Faraday
         end
       end
 
+      # Process brotli
       def brotli_inflate(body)
         Brotli.inflate(body)
       end
 
+      # Do not process anything, leave body as is
       def raw_body(body)
         body
       end
@@ -89,6 +105,15 @@ module Faraday
       def empty_body?(response_env)
         response_env[:body].nil? || response_env[:body].empty?
       end
+
+      # Method providing the processors
+      def processors
+        @processors ||= {
+          'gzip' => ->(body) { uncompress_gzip(body) },
+          'deflate' => ->(body) { inflate(body) },
+          'br' => ->(body) { brotli_inflate(body) }
+        }
+      end
     end
   end
 end

data/lib/faraday/gzip/version.rb

@@ -2,6 +2,6 @@
 
 module Faraday
   module Gzip
-    VERSION = '2.0.1'
+    VERSION = '3.0.4'
   end
 end

metadata

@@ -1,14 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: faraday-gzip
 version: !ruby/object:Gem::Version
-  version: 2.0.1
+  version: 3.0.4
 platform: ruby
 authors:
 - Ilya Krukowski
-autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-01-02 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday
@@ -16,14 +15,20 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '2.0'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '3'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '2.0'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '3'
 - !ruby/object:Gem::Dependency
   name: zlib
   requirement: !ruby/object:Gem::Requirement
@@ -108,20 +113,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.32'
-- !ruby/object:Gem::Dependency
-  name: rubocop-packaging
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 0.5.0
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 0.5.0
 - !ruby/object:Gem::Dependency
   name: rubocop-performance
   requirement: !ruby/object:Gem::Requirement
@@ -142,14 +133,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.11'
+        version: '3.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.11'
+        version: '3.0'
 description: 'Faraday plugin to automatically set compression headers (GZip, Deflate,
   Brotli) and decompress the response.
 
@@ -172,11 +163,9 @@ licenses:
 metadata:
   bug_tracker_uri: https://github.com/bodrovis/faraday-gzip/issues
   changelog_uri: https://github.com/bodrovis/faraday-gzip/blob/master/CHANGELOG.md
-  documentation_uri: http://www.rubydoc.info/gems/faraday-gzip/2.0.1
-  homepage_uri: https://github.com/bodrovis/faraday-gzip
+  documentation_uri: http://www.rubydoc.info/gems/faraday-gzip/3.0.4
   source_code_uri: https://github.com/bodrovis/faraday-gzip
   rubygems_mfa_required: 'true'
-post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -184,18 +173,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.6'
-  - - "<"
-    - !ruby/object:Gem::Version
-      version: '4'
+      version: '3.0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.3
-signing_key:
+rubygems_version: 3.6.7
 specification_version: 4
 summary: Automatically sets compression headers and decompresses the response
 test_files: []