hanami-router 2.0.0 → 2.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d1d74dfb9252f6462cb018bd567261aad94f83ea6837bb78f3d597dd9b62d0c5
-  data.tar.gz: 30c7f9425d0ad82d0c3059ed8a2d8fcc5e1a1a11092caf5f725338748451fddc
+  metadata.gz: d211b5c2f94d5e609ae9de76707c3740d387771f0864fab1d8a04f1f81942663
+  data.tar.gz: 354ce7d52713736379e88271b84e930b6b203299a355e9568c64572380286efe
 SHA512:
-  metadata.gz: 1f351789e9d4f24ef65cfdb6e19868e56c612b2f222dfd2f250f15bddfdcf00fb9d7146a5b4bd978fa2cf5b067b2dc1a96f9cf0430f028240a4c6543075cd0a1
-  data.tar.gz: 7019f2ef7c57177041495e7614992b117548d5d0eeace16d4784a9dc07b678d1d49e0987a6f750fd51b342cd5b86136fe393cef2d1fa126250a07ff8887bc1fe
+  metadata.gz: e8ef3013b9ec171da3eeef8ee92564cc55ad010c62a2133a386325c31ee2501331a92808b37503671d4b75b8d0a8d5eb38ce0bac7b24099bffea689f14cc51e0
+  data.tar.gz: bdf66d398a5e1f32b279ffb22cb4ba6719b2565631880a18e4e2d82469b293930b0f29265d483495d7b7c23dfe4b842aa0cadb34a083d32bad945a2bfc4e8100

data/CHANGELOG.md

@@ -2,6 +2,16 @@
 
 Rack compatible HTTP router for Ruby
 
+## v2.0.1 - 2022-12-06
+
+### Added
+
+- [Armin, Luca Guidi] Introduce `Hanami::Middleware::BodyParser::FormParser` to parse multipart file upload
+
+### Fixed
+
+- [Luca Guidi] Return HTTP response header `Allow` when returning `405` HTTP status
+
 ## v2.0.0 - 2022-11-22
 
 ### Fixed

data/README.md

@@ -331,22 +331,22 @@ curl http://localhost:2300/authors/1 \
 require "hanami/router"
 
 router = Hanami::Router.new do
-  get "/books/:id", to: "books#show", as: :book
+  get "/books/:id", to: "books.show", as: :book
 end
 
 route = router.recognize("/books/23")
 route.verb      # "GET"
-route.action    # => "books#show"
+route.endpoint  # => "books.show"
 route.params    # => {:id=>"23"}
 route.routable? # => true
 
 route = router.recognize(:book, id: 23)
 route.verb      # "GET"
-route.action    # => "books#show"
+route.endpoint  # => "books.show"
 route.params    # => {:id=>"23"}
 route.routable? # => true
 
-route = router.recognize("/books/23", method: :post)
+route = router.recognize("/books/23", {}, method: :post)
 route.verb      # "POST"
 route.routable? # => false
 ```

data/hanami-router.gemspec

@@ -21,8 +21,8 @@ Gem::Specification.new do |spec|
   spec.required_ruby_version = ">= 3.0"
 
   spec.add_dependency "rack",               "~> 2.0"
-  spec.add_dependency "mustermann",         "~> 1.0"
-  spec.add_dependency "mustermann-contrib", "~> 1.0"
+  spec.add_dependency "mustermann",         "~> 3.0"
+  spec.add_dependency "mustermann-contrib", "~> 3.0"
 
   spec.add_development_dependency "bundler",   ">= 1.6", "< 3"
   spec.add_development_dependency "rake",      "~> 13"

data/lib/hanami/middleware/body_parser/form_parser.rb

@@ -0,0 +1,42 @@
+# frozen_string_literal: true
+
+require_relative "parser"
+require "rack/multipart"
+
+module Hanami
+  module Middleware
+    class BodyParser
+      # @since 2.0.1
+      # @api private
+      class FormParser < Parser
+        # @since 2.0.1
+        # @api private
+        MIME_TYPES = [
+          "multipart/form-data"
+        ].freeze
+
+        # @since 2.0.1
+        # @api private
+        def self.mime_types
+          MIME_TYPES
+        end
+
+        # Parse a multipart body payload (form file uploading)
+        #
+        # @param body [String] a multipart body
+        #
+        # @return [Hash] the parsed multipart body
+        #
+        # @raise [Hanami::Middleware::BodyParser::BodyParsingError] when the body can't be parsed.
+        #
+        # @since 2.0.1
+        # @api private
+        def parse(*, env)
+          ::Rack::Multipart.parse_multipart(env)
+        rescue StandardError => exception
+          raise BodyParsingError.new(exception.message)
+        end
+      end
+    end
+  end
+end

data/lib/hanami/middleware/body_parser/json_parser.rb

@@ -25,7 +25,7 @@ module Hanami
         #
         # @since 1.3.0
         # @api private
-        def parse(body)
+        def parse(body, *)
           JSON.parse(body)
         rescue StandardError => exception
           raise BodyParsingError.new(exception.message)

data/lib/hanami/middleware/body_parser/parser.rb

@@ -34,6 +34,7 @@ module Hanami
         # Parse raw HTTP request body
         #
         # @param body [String] HTTP request body
+        # @param env [Hash] Rack env
         #
         # @return [Hash] the result of the parsing
         #
@@ -54,7 +55,7 @@ module Hanami
         #       raise Hanami::Middleware::BodyParser::BodyParsingError.new(exception.message)
         #     end
         #   end
-        def parse(body) # rubocop:disable Lint/UnusedMethodArgument
+        def parse(body, env = {}) # rubocop:disable Lint/UnusedMethodArgument
           raise NoMethodError
         end
       end

data/lib/hanami/middleware/body_parser.rb

@@ -45,7 +45,7 @@ module Hanami
         env[RACK_INPUT].rewind # somebody might try to read this stream
 
         if (parser = @parsers[media_type(env)])
-          env[Router::ROUTER_PARSED_BODY] = parser.parse(body)
+          env[Router::ROUTER_PARSED_BODY] = parser.parse(body, env)
           env[ROUTER_PARAMS] = _symbolize(env[Router::ROUTER_PARSED_BODY])
         end
 

data/lib/hanami/router/version.rb

@@ -6,7 +6,8 @@ module Hanami
     #
     # @return [String]
     #
+    # @since 0.1.0
     # @api public
-    VERSION = "2.0.0"
+    VERSION = "2.0.1"
   end
 end

data/lib/hanami/router.rb

@@ -652,10 +652,15 @@ module Hanami
     # @since 2.0.0
     # @api private
     def not_allowed(env)
-      (_not_allowed_fixed(env) ||
-      _not_allowed_variable(env)) and return [HTTP_STATUS_NOT_ALLOWED,
-                                              {::Rack::CONTENT_LENGTH => HTTP_BODY_NOT_ALLOWED_LENGTH},
-                                              [HTTP_BODY_NOT_ALLOWED]]
+      http_methods = _not_allowed_fixed(env) || _not_allowed_variable(env)
+      return if http_methods.nil?
+
+      [HTTP_STATUS_NOT_ALLOWED,
+       {
+         ::Rack::CONTENT_LENGTH => HTTP_BODY_NOT_ALLOWED_LENGTH,
+         "Allow" => http_methods.join(", ")
+       },
+       [HTTP_BODY_NOT_ALLOWED]]
     end
 
     # @since 2.0.0
@@ -934,28 +939,32 @@ module Hanami
     # @since 2.0.0
     # @api private
     def _not_allowed_fixed(env)
-      found = false
+      found = []
 
-      @fixed.each_value do |routes|
-        break if found
+      @fixed.each do |http_method, routes|
+        next if routes.fetch(env[::Rack::PATH_INFO], nil).nil?
 
-        found = routes.key?(env[::Rack::PATH_INFO])
+        found << http_method
       end
 
+      return nil if found.empty?
+
       found
     end
 
     # @since 2.0.0
     # @api private
     def _not_allowed_variable(env)
-      found = false
+      found = []
 
-      @variable.each_value do |routes|
-        break if found
+      @variable.each do |http_method, routes|
+        next if routes.find(env[::Rack::PATH_INFO]).nil?
 
-        found = routes.find(env[::Rack::PATH_INFO])
+        found << http_method
       end
 
+      return nil if found.empty?
+
       found
     end
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: hanami-router
 version: !ruby/object:Gem::Version
-  version: 2.0.0
+  version: 2.0.1
 platform: ruby
 authors:
 - Luca Guidi
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-11-22 00:00:00.000000000 Z
+date: 2022-12-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack
@@ -30,28 +30,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '3.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '3.0'
 - !ruby/object:Gem::Dependency
   name: mustermann-contrib
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '3.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '3.0'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -157,6 +157,7 @@ files:
 - lib/hanami/middleware/body_parser.rb
 - lib/hanami/middleware/body_parser/class_interface.rb
 - lib/hanami/middleware/body_parser/errors.rb
+- lib/hanami/middleware/body_parser/form_parser.rb
 - lib/hanami/middleware/body_parser/json_parser.rb
 - lib/hanami/middleware/body_parser/parser.rb
 - lib/hanami/middleware/error.rb
@@ -199,7 +200,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.7
+rubygems_version: 3.3.3
 signing_key:
 specification_version: 4
 summary: Rack compatible HTTP router for Ruby and Hanami