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

Fix deprecation message s/Rails 6.1 will return/Rails 7.0 will return/ #41251

Merged
merged 1 commit into from Jan 27, 2021

Conversation

kamipo
Copy link
Member

@kamipo kamipo commented Jan 27, 2021

return_only_media_type_on_content_type will be introduced in Rails 6.2
so the changing of returning Content-Type will happen in a future
version of Rails (probably 7.0).

cc @rafaelfranca

`return_only_media_type_on_content_type` will be introduced in Rails 6.2
so the changing of returning Content-Type will happen in a future
version of Rails (probably 7.0).
@rails-bot rails-bot bot added the actionpack label Jan 27, 2021
@rafaelfranca rafaelfranca merged commit 3921f24 into rails:main Jan 27, 2021
@kamipo kamipo deleted the fix_deprecation_message branch January 27, 2021 23:18
yahonda added a commit to yahonda/rails that referenced this pull request Feb 2, 2021
This pull request addresses these warnings.

```ruby
$ cd actionmailbox
$ bin/test test/controllers/ingresses/relay/inbound_emails_controller_test.rb
Run options: --seed 32561

DEPRECATION WARNING: Rails 7.0 will return Content-Type header without modification. If you want just the MIME type, please use `#media_type` instead. (called from call at /home/yahonda/.rbenv/versions/3.0.0/lib/ruby/gems/3.0.0/gems/rack-2.2.3/lib/rack/tempfile_reaper.rb:15)
...DEPRECATION WARNING: Rails 7.0 will return Content-Type header without modification. If you want just the MIME type, please use `#media_type` instead. (called from call at /home/yahonda/.rbenv/versions/3.0.0/lib/ruby/gems/3.0.0/gems/rack-2.2.3/lib/rack/tempfile_reaper.rb:15)
..

Finished in 0.114883s, 43.5224 runs/s, 87.0447 assertions/s.
5 runs, 10 assertions, 0 failures, 0 errors, 0 skips
$
```

Follow-up rails@84055130713
related to rails#41251
kamipo added a commit that referenced this pull request Feb 5, 2021
`return_only_media_type_on_content_type` will be introduced in Rails 7.0
so the changing of returning Content-Type will happen in a future
version of Rails.

Follow up to #41251.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants