@tomhughes commented on this pull request.
> @@ -76,11 +76,13 @@ def sanitize(text)
end
def linkify(text, mode = :urls)
- if text.html_safe?
- Rinku.auto_link(text, mode, tag_builder.tag_options(:rel => "nofollow
noopener noreferrer")).html_safe
- else
- Rinku.auto_link(text, mode, tag_builder.tag_options(:rel => "nofollow
noopener noreferrer"))
- end
+ link_attr = tag_builder.tag_options(:rel => "nofollow noopener
noreferrer")
+ Rinku.auto_link(ERB::Util.html_escape(text), mode, link_attr) do |url|
+ %r{^https?://([^/]*)(.*)$}.match(url) do |m|
The settings file can always use the multiline array syntax if the list is too
unwieldy.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5844#discussion_r2012760194
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/5844/review/2714879...@github.com>_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev
