actionpack 8.0.4.1 → 8.1.0.beta1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 56c9a5bb5fecff6688e9769fc32a9e4ae9400700c9e9059f654c0d286ef53941
-  data.tar.gz: 05f6cdd661fae12b4da50bed7c863720fe9aae79673324ba596bd85f9b9ae1f6
+  metadata.gz: ed45caa459109b7c6009c7b63f8be21dfe1965dc7bc745e8e7588579a9384391
+  data.tar.gz: aabc44311112b16f6e9b8c3f34f391d1fc1abe91cb1b0df7e920dd0068cbc8b0
 SHA512:
-  metadata.gz: d7c75cb9c14e8bdf0a96465d506cf644a6ddf1bd580a3beff1e95e9859a8c0d4a06ab24dfca786386cdf8964451996bdf31ff041f3295734921e512d2a05af5d
-  data.tar.gz: ccf3cd6ad220aca06944695be9fbf22a15de5bcedce1e9e86d4645453ae86b103473ce06bf326b74c7d669f7b4fe7f88138790819817f82841c4af1541ba9a57
+  metadata.gz: 05dc45165c2451cf7a0bd23c7b5baf55ba3e970cde0877211b208f227b98e538237c373135f5d8c3ba905c18af5834d69086ab8e223a11f7d77fed3f0e067746
+  data.tar.gz: 43cba6f2e00ce49bc0b4ef8172eb26b6571325dcfa5c8b5deebcec90cfad8f6d8d7d0804459dc29e5fd36073aa8f1b09348999e6258197360be3306d9c41134e

data/CHANGELOG.md

@@ -1,297 +1,360 @@
-## Rails 8.0.4.1 (March 23, 2026) ##
+## Rails 8.1.0.beta1 (September 04, 2025) ##
 
-*   No changes.
+*   Remove deprecated support to a route to multiple paths.
 
+    *Rafael Mendonça França*
 
-## Rails 8.0.4 (October 28, 2025) ##
+*   Remove deprecated support for using semicolons as a query string separator.
 
-*   Submit test requests using `as: :html` with `Content-Type: x-www-form-urlencoded`
+    Before:
 
-    *Sean Doyle*
+    ```ruby
+    ActionDispatch::QueryParser.each_pair("foo=bar;baz=quux").to_a
+    # => [["foo", "bar"], ["baz", "quux"]]
+    ```
 
+    After:
 
-## Rails 8.0.3 (September 22, 2025) ##
+    ```ruby
+    ActionDispatch::QueryParser.each_pair("foo=bar;baz=quux").to_a
+    # => [["foo", "bar;baz=quux"]]
+    ```
 
-*   URL helpers for engines mounted at the application root handle `SCRIPT_NAME` correctly.
+    *Rafael Mendonça França*
 
-    Fixed an issue where `SCRIPT_NAME` is not applied to paths generated for routes in an engine
-    mounted at "/".
+*   Remove deprecated support to skipping over leading brackets in parameter names in the parameter parser.
 
-    *Mike Dalessio*
+    Before:
 
-*   Fix `Rails.application.reload_routes!` from clearing almost all routes.
+    ```ruby
+    ActionDispatch::ParamBuilder.from_query_string("[foo]=bar") # => { "foo" => "bar" }
+    ActionDispatch::ParamBuilder.from_query_string("[foo][bar]=baz") # => { "foo" => { "bar" => "baz" } }
+    ```
 
-    When calling `Rails.application.reload_routes!` inside a middleware of
-    a Rake task, it was possible under certain conditions that all routes would be cleared.
-    If ran inside a middleware, this would result in getting a 404 on most page you visit.
-    This issue was only happening in development.
+    After:
 
-    *Edouard Chin*
+    ```ruby
+    ActionDispatch::ParamBuilder.from_query_string("[foo]=bar") # => { "[foo]" => "bar" }
+    ActionDispatch::ParamBuilder.from_query_string("[foo][bar]=baz") # => { "[foo]" => { "bar" => "baz" } }
+    ```
 
-*   Address `rack 3.2` deprecations warnings.
+    *Rafael Mendonça França*
 
-    ```
-    warning: Status code :unprocessable_entity is deprecated and will be removed in a future version of Rack.
-    Please use :unprocessable_content instead.
-    ```
+*   Deprecate `Rails.application.config.action_dispatch.ignore_leading_brackets`.
 
-    Rails API will transparently convert one into the other for the foreseeable future.
+    *Rafael Mendonça França*
 
-    *Earlopain*, *Jean Boussier*
+*   Raise `ActionController::TooManyRequests` error from `ActionController::RateLimiting`
 
-*   Support hash-source in Content Security Policy.
+    Requests that exceed the rate limit raise an `ActionController::TooManyRequests` error.
+    By default, Action Dispatch rescues the error and responds with a `429 Too Many Requests` status.
 
-    *madogiwa*
+    *Sean Doyle*
 
-*   Always return empty body for HEAD requests in `PublicExceptions` and
-    `DebugExceptions`.
+*   Add .md/.markdown as Markdown extensions and add a default `markdown:` renderer:
 
-    This is required by `Rack::Lint` (per RFC9110).
+    ```ruby
+    class Page
+      def to_markdown
+        body
+      end
+    end
 
-    *Hartley McGuire*
+    class PagesController < ActionController::Base
+      def show
+        @page = Page.find(params[:id])
 
+        respond_to do |format|
+          format.html
+          format.md { render markdown: @page }
+        end
+      end
+    end
+    ```
 
-## Rails 8.0.2.1 (August 13, 2025) ##
+    *DHH*
 
-*   No changes.
+*   Add headers to engine routes inspection command
 
-## Rails 8.0.2 (March 12, 2025) ##
+    *Petrik de Heus*
 
-*   Improve `with_routing` test helper to not rebuild the middleware stack.
+*   Add "Copy as text" button to error pages
 
-    Otherwise some middleware configuration could be lost.
+    *Mikkel Malmberg*
 
-    *Édouard Chin*
+*   Add `scope:` option to `rate_limit` method.
 
-*   Add resource name to the `ArgumentError` that's raised when invalid `:only` or `:except` options are given to `#resource` or `#resources`
+    Previously, it was not possible to share a rate limit count between several controllers, since the count was by
+    default separate for each controller.
 
-    This makes it easier to locate the source of the problem, especially for routes drawn by gems.
+    Now, the `scope:` option solves this problem.
 
-    Before:
-    ```
-    :only and :except must include only [:index, :create, :new, :show, :update, :destroy, :edit], but also included [:foo, :bar]
-    ```
+    ```ruby
+    class APIController < ActionController::API
+      rate_limit to: 2, within: 2.seconds, scope: "api"
+    end
 
-    After:
-    ```
-    Route `resources :products` - :only and :except must include only [:index, :create, :new, :show, :update, :destroy, :edit], but also included [:foo, :bar]
+    class API::PostsController < APIController
+      # ...
+    end
+
+    class API::UsersController < APIController
+      # ...
+    end
     ```
 
-    *Jeremy Green*
+    *ArthurPV*, *Kamil Hanus*
 
-*   Fix `url_for` to handle `:path_params` gracefully when it's not a `Hash`.
+*   Add support for `rack.response_finished` callbacks in ActionDispatch::Executor.
 
-    Prevents various security scanners from causing exceptions.
+    The executor middleware now supports deferring completion callbacks to later
+    in the request lifecycle by utilizing Rack's `rack.response_finished` mechanism,
+    when available. This enables applications to define `rack.response_finished` callbacks
+    that may rely on state that would be cleaned up by the executor's completion callbacks.
 
-    *Martin Emde*
+    *Adrianna Chang*, *Hartley McGuire*
 
-*   Fix `ActionDispatch::Executor` to unwrap exceptions like other error reporting middlewares.
+*   Produce a log when `rescue_from` is invoked.
 
-    *Jean Boussier*
+    *Steven Webb*, *Jean Boussier*
 
+*   Allow hosts redirects from `hosts` Rails configuration
 
-## Rails 8.0.1 (December 13, 2024) ##
+    ```ruby
+    config.action_controller.allowed_redirect_hosts << "example.com"
+    ```
 
-*   Add `ActionDispatch::Request::Session#store` method to conform Rack spec.
+    *Kevin Robatel*
 
-    *Yaroslav*
+*   `rate_limit.action_controller` notification has additional payload
 
+    additional values: count, to, within, by, name, cache_key
 
-## Rails 8.0.0.1 (December 10, 2024) ##
+    *Jonathan Rochkind*
 
-*   Add validation to content security policies to disallow spaces and semicolons.
-    Developers should use multiple arguments, and different directive methods instead.
+*   Add JSON support to the built-in health controller.
 
-    [CVE-2024-54133]
+    The health controller now responds to JSON requests with a structured response
+    containing status and timestamp information. This makes it easier for monitoring
+    tools and load balancers to consume health check data programmatically.
 
-    *Gannon McGibbon*
+    ```ruby
+    # /up.json
+    {
+      "status": "up",
+      "timestamp": "2025-09-19T12:00:00Z"
+    }
+    ```
 
+    *Francesco Loreti*, *Juan Vásquez*
 
-## Rails 8.0.0 (November 07, 2024) ##
+*   Allow to open source file with a crash from the browser.
 
-*   No changes.
+    *Igor Kasyanchuk*
 
+*   Always check query string keys for valid encoding just like values are checked.
 
-## Rails 8.0.0.rc2 (October 30, 2024) ##
+    *Casper Smits*
 
-*   Fix routes with `::` in the path.
+*   Always return empty body for HEAD requests in `PublicExceptions` and
+    `DebugExceptions`.
 
-    *Rafael Mendonça França*
+    This is required by `Rack::Lint` (per RFC9110).
 
-*   Maintain Rack 2 parameter parsing behaviour.
+    *Hartley McGuire*
 
-    *Matthew Draper*
+*   Add comprehensive support for HTTP Cache-Control request directives according to RFC 9111.
 
+    Provides a `request.cache_control_directives` object that gives access to request cache directives:
 
-## Rails 8.0.0.rc1 (October 19, 2024) ##
+    ```ruby
+    # Boolean directives
+    request.cache_control_directives.only_if_cached?  # => true/false
+    request.cache_control_directives.no_cache?        # => true/false
+    request.cache_control_directives.no_store?        # => true/false
+    request.cache_control_directives.no_transform?    # => true/false
+
+    # Value directives
+    request.cache_control_directives.max_age          # => integer or nil
+    request.cache_control_directives.max_stale        # => integer or nil (or true for valueless max-stale)
+    request.cache_control_directives.min_fresh        # => integer or nil
+    request.cache_control_directives.stale_if_error   # => integer or nil
+
+    # Special helpers for max-stale
+    request.cache_control_directives.max_stale?         # => true if max-stale present (with or without value)
+    request.cache_control_directives.max_stale_unlimited? # => true only for valueless max-stale
+    ```
 
-*   Remove `Rails.application.config.action_controller.allow_deprecated_parameters_hash_equality`.
+    Example usage:
 
-    *Rafael Mendonça França*
+    ```ruby
+    def show
+      if request.cache_control_directives.only_if_cached?
+        @article = Article.find_cached(params[:id])
+        return head(:gateway_timeout) if @article.nil?
+      else
+        @article = Article.find(params[:id])
+      end
+
+      render :show
+    end
+    ```
 
-*   Improve `ActionController::TestCase` to expose a binary encoded `request.body`.
+    *egg528*
 
-    The rack spec clearly states:
+*   Add assert_in_body/assert_not_in_body as the simplest way to check if a piece of text is in the response body.
 
-    > The input stream is an IO-like object which contains the raw HTTP POST data.
-    > When applicable, its external encoding must be “ASCII-8BIT” and it must be opened in binary mode.
+    *DHH*
 
-    Until now its encoding was generally UTF-8, which doesn't accurately reflect production
-    behavior.
+*   Include cookie name when calculating maximum allowed size.
 
-    *Jean Boussier*
+    *Hartley McGuire*
 
-*   Update `ActionController::AllowBrowser` to support passing method names to `:block`
+*   Implement `must-understand` directive according to RFC 9111.
 
-    ```ruby
-    class ApplicationController < ActionController::Base
-      allow_browser versions: :modern, block: :handle_outdated_browser
+    The `must-understand` directive indicates that a cache must understand the semantics of the response status code, or discard the response. This directive is enforced to be used only with `no-store` to ensure proper cache behavior.
 
-      private
-        def handle_outdated_browser
-          render file: Rails.root.join("public/custom-error.html"), status: :not_acceptable
+    ```ruby
+    class ArticlesController < ApplicationController
+      def show
+        @article = Article.find(params[:id])
+
+        if @article.special_format?
+          must_understand
+          render status: 203 # Non-Authoritative Information
+        else
+          fresh_when @article
         end
+      end
     end
     ```
 
-    *Sean Doyle*
-
-*   Raise an `ArgumentError` when invalid `:only` or `:except` options are passed into `#resource` and `#resources`.
-
-    *Joshua Young*
-
-## Rails 8.0.0.beta1 (September 26, 2024) ##
-
-*   Fix non-GET requests not updating cookies in `ActionController::TestCase`.
+    *heka1024*
 
-    *Jon Moss*, *Hartley McGuire*
+*   The JSON renderer doesn't escape HTML entities or Unicode line separators anymore.
 
-*   Update `ActionController::Live` to use a thread-pool to reuse threads across requests.
+    Using `render json:` will no longer escape `<`, `>`, `&`, `U+2028` and `U+2029` characters that can cause errors
+    when the resulting JSON is embedded in JavaScript, or vulnerabilities when the resulting JSON is embedded in HTML.
 
-    *Adam Renberg Tamm*
+    Since the renderer is used to return a JSON document as `application/json`, it's typically not necessary to escape
+    those characters, and it improves performance.
 
-*   Introduce safer, more explicit params handling method with `params#expect` such that
-    `params.expect(table: [ :attr ])` replaces `params.require(:table).permit(:attr)`
+    Escaping will still occur when the `:callback` option is set, since the JSON is used as JavaScript code in this
+    situation (JSONP).
 
-    Ensures params are filtered with consideration for the expected
-    types of values, improving handling of params and avoiding ignorable
-    errors caused by params tampering.
+    You can use the `:escape` option or set `config.action_controller.escape_json_responses` to `true` to restore the
+    escaping behavior.
 
     ```ruby
-    # If the url is altered to ?person=hacked
-    # Before
-    params.require(:person).permit(:name, :age, pets: [:name])
-    # raises NoMethodError, causing a 500 and potential error reporting
-
-    # After
-    params.expect(person: [ :name, :age, pets: [[:name]] ])
-    # raises ActionController::ParameterMissing, correctly returning a 400 error
+    class PostsController < ApplicationController
+      def index
+        render json: Post.last(30), escape: true
+      end
+    end
     ```
 
-    You may also notice the new double array `[[:name]]`. In order to
-    declare when a param is expected to be an array of parameter hashes,
-    this new double array syntax is used to explicitly declare an array.
-    `expect` requires you to declare expected arrays in this way, and will
-    ignore arrays that are passed when, for example, `pet: [:name]` is used.
-
-    In order to preserve compatibility, `permit` does not adopt the new
-    double array syntax and is therefore more permissive about unexpected
-    types. Using `expect` everywhere is recommended.
+    *Étienne Barrié*, *Jean Boussier*
 
-    We suggest replacing `params.require(:person).permit(:name, :age)`
-    with the direct replacement `params.expect(person: [:name, :age])`
-    to prevent external users from manipulating params to trigger 500
-    errors. A 400 error will be returned instead, using public/400.html
+*   Load lazy route sets before inserting test routes
 
-    Usage of `params.require(:id)` should likewise be replaced with
-    `params.expect(:id)` which is designed to ensure that `params[:id]`
-    is a scalar and not an array or hash, also requiring the param.
+    Without loading lazy route sets early, we miss `after_routes_loaded` callbacks, or risk
+    invoking them with the test routes instead of the real ones if another load is triggered by an engine.
 
-    ```ruby
-    # Before
-    User.find(params.require(:id)) # allows an array, altering behavior
+    *Gannon McGibbon*
 
-    # After
-    User.find(params.expect(:id)) # expect only returns non-blank permitted scalars (excludes Hash, Array, nil, "", etc)
-    ```
+*   Raise `AbstractController::DoubleRenderError` if `head` is called after rendering.
 
-    *Martin Emde*
+    After this change, invoking `head` will lead to an error if response body is already set:
 
-*   System Testing: Disable Chrome's search engine choice by default in system tests.
+    ```ruby
+    class PostController < ApplicationController
+      def index
+        render locals: {}
+        head :ok
+      end
+    end
+    ```
 
-    *glaszig*
+    *Iaroslav Kurbatov*
 
-*   Fix `Request#raw_post` raising `NoMethodError` when `rack.input` is `nil`.
+*   The Cookie Serializer can now serialize an Active Support SafeBuffer when using message pack.
 
-    *Hartley McGuire*
+    Such code would previously produce an error if an application was using messagepack as its cookie serializer.
 
-*   Remove `racc` dependency by manually writing `ActionDispatch::Journey::Scanner`.
+    ```ruby
+    class PostController < ApplicationController
+      def index
+        flash.notice = t(:hello_html) # This would try to serialize a SafeBuffer, which was not possible.
+      end
+    end
+    ```
 
-    *Gannon McGibbon*
+    *Edouard Chin*
 
-*   Speed up `ActionDispatch::Routing::Mapper::Scope#[]` by merging frame hashes.
+*   Fix `Rails.application.reload_routes!` from clearing almost all routes.
 
-    *Gannon McGibbon*
+    When calling `Rails.application.reload_routes!` inside a middleware of
+    a Rake task, it was possible under certain conditions that all routes would be cleared.
+    If ran inside a middleware, this would result in getting a 404 on most page you visit.
+    This issue was only happening in development.
 
-*   Allow bots to ignore `allow_browser`.
+    *Edouard Chin*
 
-    *Matthew Nguyen*
+*   Add resource name to the `ArgumentError` that's raised when invalid `:only` or `:except` options are given to `#resource` or `#resources`
 
-*   Deprecate drawing routes with multiple paths to make routing faster.
-    You may use `with_options` or a loop to make drawing multiple paths easier.
+    This makes it easier to locate the source of the problem, especially for routes drawn by gems.
 
-    ```ruby
-    # Before
-    get "/users", "/other_path", to: "users#index"
+    Before:
+    ```
+    :only and :except must include only [:index, :create, :new, :show, :update, :destroy, :edit], but also included [:foo, :bar]
+    ```
 
-    # After
-    get "/users", to: "users#index"
-    get "/other_path", to: "users#index"
+    After:
+    ```
+    Route `resources :products` - :only and :except must include only [:index, :create, :new, :show, :update, :destroy, :edit], but also included [:foo, :bar]
     ```
 
-    *Gannon McGibbon*
+    *Jeremy Green*
 
-*   Make `http_cache_forever` use `immutable: true`
+*   A route pointing to a non-existing controller now returns a 500 instead of a 404.
 
-    *Nate Matykiewicz*
+    A controller not existing isn't a routing error that should result
+    in a 404, but a programming error that should result in a 500 and
+    be reported.
 
-*   Add `config.action_dispatch.strict_freshness`.
+    Until recently, this was hard to untangle because of the support
+    for dynamic `:controller` segment in routes, but since this is
+    deprecated and will be removed in Rails 8.1, we can now easily
+    not consider missing controllers as routing errors.
 
-    When set to `true`, the `ETag` header takes precedence over the `Last-Modified` header when both are present,
-    as specified by RFC 7232, Section 6.
+    *Jean Boussier*
 
-    Defaults to `false` to maintain compatibility with previous versions of Rails, but is enabled as part of
-    Rails 8.0 defaults.
+*   Add `check_collisions` option to `ActionDispatch::Session::CacheStore`.
 
-    *heka1024*
+    Newly generated session ids use 128 bits of randomness, which is more than
+    enough to ensure collisions can't happen, but if you need to harden sessions
+    even more, you can enable this option to check in the session store that the id
+    is indeed free you can enable that option. This however incurs an extra write
+    on session creation.
 
-*   Support `immutable` directive in Cache-Control
+    *Shia*
 
-    ```ruby
-    expires_in 1.minute, public: true, immutable: true
-    # Cache-Control: public, max-age=60, immutable
-    ```
+*   In ExceptionWrapper, match backtrace lines with built templates more often,
+    allowing improved highlighting of errors within do-end blocks in templates.
+    Fix for Ruby 3.4 to match new method labels in backtrace.
 
-    *heka1024*
+    *Martin Emde*
 
-*   Add `:wasm_unsafe_eval` mapping for `content_security_policy`
+*   Allow setting content type with a symbol of the Mime type.
 
     ```ruby
     # Before
-    policy.script_src "'wasm-unsafe-eval'"
+    response.content_type = "text/html"
 
     # After
-    policy.script_src :wasm_unsafe_eval
+    response.content_type = :html
     ```
 
-    *Joe Haig*
-
-*   Add `display_capture` and `keyboard_map` in `permissions_policy`
-
-    *Cyril Blaecke*
-
-*   Add `connect` route helper.
-
-    *Samuel Williams*
+    *Petrik de Heus*
 
-Please check [7-2-stable](https://github.com/rails/rails/blob/7-2-stable/actionpack/CHANGELOG.md) for previous changes.
+Please check [8-0-stable](https://github.com/rails/rails/blob/8-0-stable/actionpack/CHANGELOG.md) for previous changes.

data/lib/abstract_controller/asset_paths.rb

@@ -7,8 +7,10 @@ module AbstractController
     extend ActiveSupport::Concern
 
     included do
-      config_accessor :asset_host, :assets_dir, :javascripts_dir,
-        :stylesheets_dir, :default_asset_host_protocol, :relative_url_root
+      singleton_class.delegate :asset_host, :asset_host=, :assets_dir, :assets_dir=, :javascripts_dir, :javascripts_dir=,
+        :stylesheets_dir, :stylesheets_dir=, :default_asset_host_protocol, :default_asset_host_protocol=, :relative_url_root, :relative_url_root=, to: :config
+      delegate :asset_host, :asset_host=, :assets_dir, :assets_dir=, :javascripts_dir, :javascripts_dir=,
+        :stylesheets_dir, :stylesheets_dir=, :default_asset_host_protocol, :default_asset_host_protocol=, :relative_url_root, :relative_url_root=, to: :config
     end
   end
 end

data/lib/abstract_controller/base.rb

@@ -3,7 +3,6 @@
 # :markup: markdown
 
 require "abstract_controller/error"
-require "active_support/configurable"
 require "active_support/descendants_tracker"
 require "active_support/core_ext/module/anonymous"
 require "active_support/core_ext/module/attr_internal"
@@ -47,7 +46,7 @@ module AbstractController
     # Returns the formats that can be processed by the controller.
     attr_internal :formats
 
-    include ActiveSupport::Configurable
+    class_attribute :config, instance_predicate: false, default: ActiveSupport::OrderedOptions.new
     extend ActiveSupport::DescendantsTracker
 
     class << self
@@ -65,6 +64,7 @@ module AbstractController
         unless klass.instance_variable_defined?(:@abstract)
           klass.instance_variable_set(:@abstract, false)
         end
+        klass.config = ActiveSupport::InheritableOptions.new(config)
         super
       end
 
@@ -121,6 +121,10 @@ module AbstractController
         @controller_path ||= name.delete_suffix("Controller").underscore unless anonymous?
       end
 
+      def configure # :nodoc:
+        yield config
+      end
+
       # Refresh the cached action_methods when a new action_method is added.
       def method_added(name)
         super
@@ -190,6 +194,10 @@ module AbstractController
       true
     end
 
+    def config # :nodoc:
+      @_config ||= self.class.config.inheritable_copy
+    end
+
     def inspect # :nodoc:
       "#<#{self.class.name}:#{'%#016x' % (object_id << 1)}>"
     end

data/lib/abstract_controller/caching.rb

@@ -32,13 +32,16 @@ module AbstractController
     included do
       extend ConfigMethods
 
-      config_accessor :default_static_extension
+      singleton_class.delegate :default_static_extension, :default_static_extension=, to: :config
+      delegate :default_static_extension, :default_static_extension=, to: :config
       self.default_static_extension ||= ".html"
 
-      config_accessor :perform_caching
+      singleton_class.delegate :perform_caching, :perform_caching=, to: :config
+      delegate :perform_caching, :perform_caching=, to: :config
       self.perform_caching = true if perform_caching.nil?
 
-      config_accessor :enable_fragment_cache_logging
+      singleton_class.delegate :enable_fragment_cache_logging, :enable_fragment_cache_logging=, to: :config
+      delegate :enable_fragment_cache_logging, :enable_fragment_cache_logging=, to: :config
       self.enable_fragment_cache_logging = false
 
       class_attribute :_view_cache_dependencies, default: []

data/lib/abstract_controller/helpers.rb

@@ -90,7 +90,7 @@ module AbstractController
       #--
       # Implemented by Resolution#modules_for_helpers.
 
-      # :method: all_helpers_from_path
+      # :method: # all_helpers_from_path
       # :call-seq: all_helpers_from_path(path)
       #
       # Returns a list of helper names in a given path.

data/lib/abstract_controller/logger.rb

@@ -9,7 +9,8 @@ module AbstractController
     extend ActiveSupport::Concern
 
     included do
-      config_accessor :logger
+      singleton_class.delegate :logger, :logger=, to: :config
+      delegate :logger, :logger=, to: :config
       include ActiveSupport::Benchmarkable
     end
   end