hanami-router 2.1.0.rc2 → 2.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c3207bbac39b5897bd7fa20d6a5e08d6949944be65e0591bccd97f3003e6478f
-  data.tar.gz: b278f47862cc6be5e1dcb4f93517891a73e7ec1048e4ab2da42f3d41fda3ee9d
+  metadata.gz: bd15cdabe88259102767109851f7eb4253c52fbdd8a77fff7e3e53512953866c
+  data.tar.gz: ca5c14f2abe932cda82da4ebacfe569e8acdfbce915d33e89201dd3af52fd044
 SHA512:
-  metadata.gz: dc876c976c261b4c59741c78d78ac9b5afad73074f934a677d4c6c6189bfa86f567990c2037f3bb200ec738825fe00bb648d6c82b7f1a92adc7559abc331ddb7
-  data.tar.gz: afe72323c1f8cb7ef12a933d3ea4a8934d80ca9f4ea99b85e2c0049195eea546b20c5677689e2c7bd00936c2bf839308d9556d1c2c99ed981f0ef5e1fc05f5ad
+  metadata.gz: 1b8dba8cd1f1a178b84a8222b0f0d671a87c0e8e58332022c4480a850a2b3daf3e1d9a0915299d0ee8249d016610a088c8ee7049d0bb93fbefb25e74586d6dbc
+  data.tar.gz: 1eca7bf5c3e7928f9d5400a87fa9d9089f43312f4d89684c5137c082469c31aeea5cd8f910fea9c08cc58a8cc01d2a659feee0a0c62593ccca1ba1854d70d6c8

data/CHANGELOG.md

@@ -2,6 +2,19 @@
 
 Rack compatible HTTP router for Ruby
 
+## v2.1.0 - 2024-02-27
+
+### Fixed
+
+- [Pat Allan] Fix PATH_INFO and SCRIPT_NAME for Rack apps mounted at the root (keep the leading slash in PATH_INFO, and set SCRIPT_NAME to a blank string)
+- [Pat Allan] Process glob routes and mounted apps together, so that the routes can be handled in the user-specified order (previously, a root-mounted app would handle routes even if matching globs were declared earlier)
+
+### Changed
+
+- [Pat Allan] Pass keyword args through to middleware
+
+## v2.1.0.rc3 - 2024-02-16
+
 ## v2.1.0.rc2 - 2023-11-08
 
 ## v2.1.0.rc1 - 2023-11-02

data/lib/hanami/middleware/app.rb

@@ -21,8 +21,8 @@ module Hanami
         mapping.each do |path, stack|
           builder = Rack::Builder.new
 
-          stack.each do |middleware, args, blk|
-            builder.use(middleware, *args, &blk)
+          stack.each do |middleware, args, kwargs, blk|
+            builder.use(middleware, *args, **kwargs, &blk)
           end
 
           builder.run(router)

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

@@ -23,8 +23,6 @@ module Hanami
 
         # 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.

data/lib/hanami/router/errors.rb

@@ -78,7 +78,7 @@ module Hanami
       # @since 0.5.0
       # @api private
       def initialize(env)
-        super %(Cannot find routable endpoint for: #{env[::Rack::REQUEST_METHOD]} #{env[::Rack::PATH_INFO]})
+        super(%(Cannot find routable endpoint for: #{env[::Rack::REQUEST_METHOD]} #{env[::Rack::PATH_INFO]}))
       end
     end
   end

data/lib/hanami/router/globbed_path.rb

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+module Hanami
+  class Router
+    class GlobbedPath
+      def initialize(http_method, path, to)
+        @http_method = http_method
+        @path = path
+        @to = to
+      end
+
+      def endpoint_and_params(env)
+        return [] unless @http_method == env[::Rack::REQUEST_METHOD]
+
+        if (match = @path.match(env[::Rack::PATH_INFO]))
+          [@to, match.named_captures]
+        else
+          []
+        end
+      end
+    end
+  end
+end

data/lib/hanami/router/mounted_path.rb

@@ -0,0 +1,26 @@
+# frozen_string_literal: true
+
+module Hanami
+  class Router
+    class MountedPath
+      def initialize(prefix, app)
+        @prefix = prefix
+        @app = app
+      end
+
+      def endpoint_and_params(env)
+        return [] unless (match = @prefix.peek_match(env[::Rack::PATH_INFO]))
+
+        if @prefix.to_s == "/"
+          env[::Rack::SCRIPT_NAME] = EMPTY_STRING
+        else
+          env[::Rack::SCRIPT_NAME] = env[::Rack::SCRIPT_NAME].to_s + @prefix.to_s
+          env[::Rack::PATH_INFO] = env[::Rack::PATH_INFO].sub(@prefix.to_s, EMPTY_STRING)
+          env[::Rack::PATH_INFO] = DEFAULT_PREFIX if env[::Rack::PATH_INFO] == EMPTY_STRING
+        end
+
+        [@app, match.named_captures]
+      end
+    end
+  end
+end

data/lib/hanami/router/version.rb

@@ -8,6 +8,6 @@ module Hanami
     #
     # @since 0.1.0
     # @api public
-    VERSION = "2.1.0.rc2"
+    VERSION = "2.1.0"
   end
 end

data/lib/hanami/router.rb

@@ -20,6 +20,8 @@ module Hanami
     require "hanami/router/block"
     require "hanami/router/route"
     require "hanami/router/url_helpers"
+    require "hanami/router/globbed_path"
+    require "hanami/router/mounted_path"
 
     # URL helpers for other Hanami integrations
     #
@@ -84,8 +86,7 @@ module Hanami
       @block_context = block_context
       @fixed = {}
       @variable = {}
-      @globbed = {}
-      @mounted = {}
+      @globs_and_mounts = []
       @blk = blk
       @inspector = inspector
       instance_eval(&blk) if blk
@@ -424,7 +425,7 @@ module Hanami
       path = prefixed_path(at)
       prefix = Segment.fabricate(path, **constraints)
 
-      @mounted[prefix] = @resolver.call(path, app)
+      @globs_and_mounts << MountedPath.new(prefix, @resolver.call(path, app))
       if inspect?
         @inspector.add_route(Route.new(http_method: "*", path: at, to: app, constraints: constraints))
       end
@@ -619,31 +620,12 @@ module Hanami
       @variable[env[::Rack::REQUEST_METHOD]]&.find(env[::Rack::PATH_INFO])
     end
 
-    # @since 2.0.0
-    # @api private
-    def globbed(env)
-      @globbed[env[::Rack::REQUEST_METHOD]]&.each do |path, to|
-        if (match = path.match(env[::Rack::PATH_INFO]))
-          return [to, match.named_captures]
-        end
-      end
-
-      nil
-    end
-
-    # @since 2.0.0
+    # @since 2.1.0
     # @api private
-    def mounted(env)
-      @mounted.each do |prefix, app|
-        next unless (match = prefix.peek_match(env[::Rack::PATH_INFO]))
-
-        # TODO: ensure compatibility with existing env[::Rack::SCRIPT_NAME]
-        # TODO: cleanup this code
-        env[::Rack::SCRIPT_NAME] = env[::Rack::SCRIPT_NAME].to_s + prefix.to_s
-        env[::Rack::PATH_INFO] = env[::Rack::PATH_INFO].sub(prefix.to_s, EMPTY_STRING)
-        env[::Rack::PATH_INFO] = DEFAULT_PREFIX if env[::Rack::PATH_INFO] == EMPTY_STRING
-
-        return [app, match.named_captures]
+    def globbed_or_mounted(env)
+      @globs_and_mounts.each do |path|
+        result = path.endpoint_and_params(env)
+        return result unless result.empty?
       end
 
       nil
@@ -800,7 +782,7 @@ module Hanami
       endpoint = fixed(env)
       return [endpoint, {}] if endpoint
 
-      variable(env) || mounted(env) || globbed(env)
+      variable(env) || globbed_or_mounted(env)
     end
 
     # @since 2.0.0
@@ -843,8 +825,7 @@ module Hanami
     # @since 2.0.0
     # @api private
     def add_globbed_route(http_method, path, to, constraints)
-      @globbed[http_method] ||= []
-      @globbed[http_method] << [Segment.fabricate(path, **constraints), to]
+      @globs_and_mounts << GlobbedPath.new(http_method, Segment.fabricate(path, **constraints), to)
     end
 
     # @since 2.0.0

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: hanami-router
 version: !ruby/object:Gem::Version
-  version: 2.1.0.rc2
+  version: 2.1.0
 platform: ruby
 authors:
 - Luca Guidi
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-11-08 00:00:00.000000000 Z
+date: 2024-02-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack
@@ -169,7 +169,9 @@ files:
 - lib/hanami/router/errors.rb
 - lib/hanami/router/formatter/csv.rb
 - lib/hanami/router/formatter/human_friendly.rb
+- lib/hanami/router/globbed_path.rb
 - lib/hanami/router/inspector.rb
+- lib/hanami/router/mounted_path.rb
 - lib/hanami/router/node.rb
 - lib/hanami/router/params.rb
 - lib/hanami/router/prefix.rb
@@ -196,11 +198,11 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '3.0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">"
+  - - ">="
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
 requirements: []
-rubygems_version: 3.4.21
+rubygems_version: 3.5.6
 signing_key:
 specification_version: 4
 summary: Rack compatible HTTP router for Ruby and Hanami