[ci skip] Fix some more documentation links

kaspth · kaspth · commit f2ca082d7b5f · 2021-11-06T03:43:46.000+01:00
diff --git a/actionpack/lib/action_controller/metal/redirecting.rb b/actionpack/lib/action_controller/metal/redirecting.rb
@@ -66,13 +66,13 @@ class UnsafeRedirectError < StandardError; end
     # === Open Redirect protection
     #
     # By default, Rails protects against redirecting to external hosts for your app's safety, so called open redirects.
-    # Note: this was a new default in Rails 7.0, after upgrading opt-in by uncommenting the line with #raise_on_open_redirects in <tt>config/initializers/new_framework_defaults_7_0.rb</tt>
+    # Note: this was a new default in Rails 7.0, after upgrading opt-in by uncommenting the line with +raise_on_open_redirects+ in <tt>config/initializers/new_framework_defaults_7_0.rb</tt>
     #
     # Here #redirect_to automatically validates the potentially-unsafe URL:
     #
     #   redirect_to params[:redirect_url]
     #
-    # Raises #UnsafeRedirectError in the case of an unsafe redirect.
+    # Raises UnsafeRedirectError in the case of an unsafe redirect.
     #
     # To allow any external redirects pass `allow_other_host: true`, though using a user-provided param in that case is unsafe.
     #
@@ -90,7 +90,7 @@ def redirect_to(options = {}, response_options = {})
       self.response_body = "<html><body>You are being <a href=\"#{ERB::Util.unwrapped_html_escape(response.location)}\">redirected</a>.</body></html>"
     end
 
-    # Soft deprecated alias for <tt>redirect_back_or_to</tt> where the fallback_location location is supplied as a keyword argument instead
+    # Soft deprecated alias for #redirect_back_or_to where the +fallback_location+ location is supplied as a keyword argument instead
     # of the first positional argument.
     def redirect_back(fallback_location:, allow_other_host: _allow_other_host, **args)
       redirect_back_or_to fallback_location, allow_other_host: allow_other_host, **args
@@ -164,7 +164,7 @@ def _compute_redirect_to_location(request, options) # :nodoc:
     #   # If request.host is on https://example.com or https://app.example.com, you'd get:
     #   url_from("https://dev.example.com/profile") # => nil
     #
-    # NOTE: there's a similarity with #url_for, which generates an internal URL from various options from within the app, e.g. <tt>url_for(@post)</tt>.
+    # NOTE: there's a similarity with ActionDispatch::Routing::UrlFor#url_for, which generates an internal URL from various options from within the app, e.g. <tt>url_for(@post)</tt>.
     # However, #url_from is meant to take an external parameter to verify as in <tt>url_from(params[:redirect_url])</tt>.
     def url_from(location)
       location = location.presence

Original file line number	Diff line number	Diff line change
`@@ -66,13 +66,13 @@ class UnsafeRedirectError < StandardError; end`
`66`	`66`	`# === Open Redirect protection`
`67`	`67`	`#`
`68`	`68`	`# By default, Rails protects against redirecting to external hosts for your app's safety, so called open redirects.`
`69`		`- # Note: this was a new default in Rails 7.0, after upgrading opt-in by uncommenting the line with #raise_on_open_redirects in <tt>config/initializers/new_framework_defaults_7_0.rb</tt>`
	`69`	`+ # Note: this was a new default in Rails 7.0, after upgrading opt-in by uncommenting the line with +raise_on_open_redirects+ in <tt>config/initializers/new_framework_defaults_7_0.rb</tt>`
`70`	`70`	`#`
`71`	`71`	`# Here #redirect_to automatically validates the potentially-unsafe URL:`
`72`	`72`	`#`
`73`	`73`	`# redirect_to params[:redirect_url]`
`74`	`74`	`#`
`75`		`- # Raises #UnsafeRedirectError in the case of an unsafe redirect.`
	`75`	`+ # Raises UnsafeRedirectError in the case of an unsafe redirect.`
`76`	`76`	`#`
`77`	`77`	# To allow any external redirects pass `allow_other_host: true`, though using a user-provided param in that case is unsafe.
`78`	`78`	`#`
`@@ -90,7 +90,7 @@ def redirect_to(options = {}, response_options = {})`
`90`	`90`	`self.response_body = "<html><body>You are being <a href=\"#{ERB::Util.unwrapped_html_escape(response.location)}\">redirected</a>.</body></html>"`
`91`	`91`	`end`
`92`	`92`
`93`		`- # Soft deprecated alias for <tt>redirect_back_or_to</tt> where the fallback_location location is supplied as a keyword argument instead`
	`93`	`+ # Soft deprecated alias for #redirect_back_or_to where the +fallback_location+ location is supplied as a keyword argument instead`
`94`	`94`	`# of the first positional argument.`
`95`	`95`	`def redirect_back(fallback_location:, allow_other_host: _allow_other_host, **args)`
`96`	`96`	`redirect_back_or_to fallback_location, allow_other_host: allow_other_host, **args`
`@@ -164,7 +164,7 @@ def _compute_redirect_to_location(request, options) # :nodoc:`
`164`	`164`	`# # If request.host is on https://example.com or https://app.example.com, you'd get:`
`165`	`165`	`# url_from("https://dev.example.com/profile") # => nil`
`166`	`166`	`#`
`167`		`- # NOTE: there's a similarity with #url_for, which generates an internal URL from various options from within the app, e.g. <tt>url_for(@post)</tt>.`
	`167`	`+ # NOTE: there's a similarity with ActionDispatch::Routing::UrlFor#url_for, which generates an internal URL from various options from within the app, e.g. <tt>url_for(@post)</tt>.`
`168`	`168`	`# However, #url_from is meant to take an external parameter to verify as in <tt>url_from(params[:redirect_url])</tt>.`
`169`	`169`	`def url_from(location)`
`170`	`170`	`location = location.presence`