Skip to content

Rename unprocessable_entity -> unprocessable_content.#52097

Closed
ioquatix wants to merge 1 commit into
rails:mainfrom
ioquatix:rack-3-unprocessable-content
Closed

Rename unprocessable_entity -> unprocessable_content.#52097
ioquatix wants to merge 1 commit into
rails:mainfrom
ioquatix:rack-3-unprocessable-content

Conversation

@ioquatix

@ioquatix ioquatix commented Jun 12, 2024

Copy link
Copy Markdown
Contributor

Motivation / Background

In Rack 3.1, unprocessable_entity was renamed to unprocessable_content to reflect the current naming conventions. See rack/rack#2137 for more context.

The old status code alias/name continues to work but emits a deprecation warning, so we prefer to use the new name where possible while retaining compatibility with old versions of Rack.

Checklist

Before submitting the PR make sure the following are checked:

  • This Pull Request is related to one change. Unrelated changes should be opened in separate PRs.
  • Commit message has a detailed description of what changed and why. If this PR fixes a related issue include it in the commit message. Ex: [Fix #issue-number]
  • Tests are added or updated if you fix a bug or add a feature.
  • CHANGELOG files are updated for the changed libraries if there is a behavior change or additional feature. Minor bug fixes and documentation changes should not be included.

@rails-bot rails-bot Bot added the actionpack label Jun 12, 2024
@ioquatix

Copy link
Copy Markdown
Contributor Author

@tenderlove the alternative is we update Rack 3.1 to remove the deprecation warnings and re-introduce them in Rack 3.2?

@ioquatix

Copy link
Copy Markdown
Contributor Author

After consideration, I think we should do rack/rack#2209 instead.

@ioquatix ioquatix closed this Jun 12, 2024
@ioquatix ioquatix deleted the rack-3-unprocessable-content branch June 12, 2024 04:07
Earlopain added a commit to Earlopain/rails that referenced this pull request Oct 19, 2024
Using `unprocessable_entity` will emit a warning on Rack 3.2.
There have been various PRs in the past for this:
* rails#52252
* rails#52192
* rails#52087
* rails#52097

This to me seems like the least invasive change and is also how headers are currently handled.
Earlopain added a commit to Earlopain/rails that referenced this pull request Mar 3, 2025
Using `unprocessable_entity` will emit a warning on Rack 3.2.
There have been various PRs in the past for this:
* rails#52252
* rails#52192
* rails#52087
* rails#52097

This to me seems like the least invasive change and is also how headers are currently handled.
Earlopain added a commit to Earlopain/rails that referenced this pull request Jul 23, 2025
Using `unprocessable_entity` will emit a warning on Rack 3.2.
There have been various PRs in the past for this:
* rails#52252
* rails#52192
* rails#52087
* rails#52097

This to me seems like the least invasive change and is also how headers are currently handled.
byroot pushed a commit to Earlopain/rails that referenced this pull request Jul 31, 2025
Using `unprocessable_entity` will emit a warning on Rack 3.2.
There have been various PRs in the past for this:
* rails#52252
* rails#52192
* rails#52087
* rails#52097

This to me seems like the least invasive change and is also how headers are currently handled.
viralpraxis pushed a commit to viralpraxis/rails that referenced this pull request Sep 22, 2025
Using `unprocessable_entity` will emit a warning on Rack 3.2.
There have been various PRs in the past for this:
* rails#52252
* rails#52192
* rails#52087
* rails#52097

This to me seems like the least invasive change and is also how headers are currently handled.
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.

1 participant