Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

avoid FrozenError when params contains frozen value #1506

Merged
merged 1 commit into from Dec 16, 2018
Merged

Conversation

namusyaka
Copy link
Member

Fixes #1478
Closes #1479

The error can be fixed by using dupped values.
The original code for testing is implemented by @programmarchy in #1479

Fixes #1478
Closes #1479
@namusyaka namusyaka merged commit 9057b70 into master Dec 16, 2018
@namusyaka namusyaka deleted the fix-1478 branch December 16, 2018 07:50
netbsd-srcmastr pushed a commit to NetBSD/pkgsrc that referenced this pull request Feb 6, 2019
## 2.0.5 / 2018-12-22

* Avoid FrozenError when params contains frozen value [#1506](sinatra/sinatra#1506) by Kunpei Sakai

* Add support for Erubi [#1494](sinatra/sinatra#1494) by @tkmru

* `IndifferentHash` monkeypatch warning improvements [#1477](sinatra/sinatra#1477) by Mike Pastore

* Improve development support and documentation and source code by Anusree Prakash, Jordan Owens, @ceclinux and @krororo.
namusyaka added a commit that referenced this pull request Apr 11, 2019
The original fixes are #1506 and #1517.
This commit considers `FrozenError` and `TypeEror` for the case.
namusyaka added a commit that referenced this pull request Apr 11, 2019
The original fixes are #1506 and #1517.
This commit considers `FrozenError` and `TypeEror` for the case.
namusyaka added a commit that referenced this pull request Apr 22, 2019
The original fixes are #1506 and #1517.
This commit considers `FrozenError` and `TypeEror` for the case.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant