Merge pull request rails#53131 from seanpdoyle/allow-browser-browser-blocker-private

kamipo · web-flow · commit 137f738f576f · 2024-10-01T22:18:18.000+09:00
Make `ActionController::AllowBrowser::BrowserBlocker` private
diff --git a/actionpack/CHANGELOG.md b/actionpack/CHANGELOG.md
@@ -1,5 +1,9 @@
 ## Rails 8.0.0.beta1 (September 26, 2024) ##
 
+*   Make `ActionController::AllowBrowser::BrowserBlocker` private
+
+    *Sean Doyle*
+
 *   Fix non-GET requests not updating cookies in `ActionController::TestCase`.
 
     *Jon Moss*, *Hartley McGuire*
diff --git a/actionpack/lib/action_controller/metal/allow_browser.rb b/actionpack/lib/action_controller/metal/allow_browser.rb
@@ -60,7 +60,7 @@ def allow_browser(versions:, block:)
         end
       end
 
-      class BrowserBlocker
+      class BrowserBlocker # :nodoc:
         SETS = {
           modern: { safari: 17.2, chrome: 120, firefox: 121, opera: 106, ie: false }
         }

Original file line number	Diff line number	Diff line change
`@@ -60,7 +60,7 @@ def allow_browser(versions:, block:)`
`60`	`60`	`end`
`61`	`61`	`end`
`62`	`62`
`63`		`- class BrowserBlocker`
	`63`	`+ class BrowserBlocker # :nodoc:`
`64`	`64`	`SETS = {`
`65`	`65`	`modern: { safari: 17.2, chrome: 120, firefox: 121, opera: 106, ie: false }`
`66`	`66`	`}`