[openstreetmap/openstreetmap-website] Use Brakeman for static code analysis (#2723)

[Andy Allan]

This is my preferred version of #2229 

I've also reviewed the currently ignored warnings. Many of them are in code that will soon be either removed (amf_controller) or refactored (trace file handling) so I'm not prioritising fixing them at the moment. But at least having Brakeman included will prevent other errors from being accidentally introduced.
You can view, comment on, or merge this pull request online at:

  https://github.com/openstreetmap/openstreetmap-website/pull/2723

-- Commit Summary --

  * Use Brakeman for static code analysis

-- File Changes --

    M .travis.yml (1)
    M CONTRIBUTING.md (8)
    M Gemfile (1)
    M Gemfile.lock (2)
    A config/brakeman.yml (9)

-- Patch Links --

https://github.com/openstreetmap/openstreetmap-website/pull/2723.patch
https://github.com/openstreetmap/openstreetmap-website/pull/2723.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/2723
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20200722/8142da76/attachment.htm>