diff options
author | Sam Davies <seivadmas@gmail.com> | 2016-03-03 12:25:28 -0300 |
---|---|---|
committer | Sam Davies <seivadmas@gmail.com> | 2016-03-03 21:23:19 -0300 |
commit | fea7c9fed65b9dd305562483c1cfd80104d521b5 (patch) | |
tree | 63ed6da70a1ea9a833fc9b6217a0ba31db92ee77 /activesupport/lib/active_support/core_ext/hash | |
parent | 3009fa1e089ff1133ce95c767063ca9f9cdb130f (diff) | |
download | rails-fea7c9fed65b9dd305562483c1cfd80104d521b5.tar.gz rails-fea7c9fed65b9dd305562483c1cfd80104d521b5.tar.bz2 rails-fea7c9fed65b9dd305562483c1cfd80104d521b5.zip |
Do not destructively mutate passed options hash in route definitions
- Fixes #24030
An example scope might be specified as such:
```ruby
HTML = { constraints: { format: :html } }.freeze
scope HTML do
get 'x'
end
```
This currently raises an error because the mapper attempts to
destructively modify the passed options hash. This is dangerous because
this options hash might even be shared with other scopes.
We should instead always instantiate a new object instead of modifying
the passed options.
Diffstat (limited to 'activesupport/lib/active_support/core_ext/hash')
0 files changed, 0 insertions, 0 deletions