@hlfan commented on this pull request.
> @@ -81,7 +81,8 @@
<% if params[:oauth_return_url] %>
<a class="btn btn-primary" href="<%= params[:oauth_return_url] %>"><%= t
".continue_authorization" %></a>
<% else %>
- <a class="btn btn-primary start-mapping" href="<%= edit_path %>"><%= t
".start_mapping" %></a>
+ <%= editor_query = "?editor=#{params[:editor].to_s.split(/\W/)[0]}" if
params[:editor].present?
Brakeman complained, again
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5887#discussion_r2076024995
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/5887/review/2819227...@github.com>_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev
