mirror of
https://github.com/heartcombo/devise.git
synced 2026-01-08 22:37:57 -05:00
8054ad55c3
In Rack v3.1.0, the symbol for HTTP status code 422 was changed from `:unprocessable_entity` to `:unprocessable_content`. As a result, when using rack 3.2 with the following configuration in `config/initializers/devise.rb`, a warning is shown on login failure: ```ruby # config/initializers/devise.rb Devise.setup do |config| ... config.responder.error_status = :unprocessable_entity ``` Warning message: ```sh /path-to-app/vendor/bundle/ruby/3.4.0/gems/devise-4.9.4/lib/devise/failure_app.rb:80: warning: Status code :unprocessable_entity is deprecated and will be removed in a future version of Rack. Please use :unprocessable_content instead. ``` This warning can be resolved by updating the config as follows: ```diff # config/initializers/devise.rb Devise.setup do |config| ... + config.responder.error_status = :unprocessable_content - config.responder.error_status = :unprocessable_entity ``` This fixes the root cause of the warning for new apps by adjusting the generated config during `$ rails generate devise:install` depending on the rack version, so new apps using newer Rack versions generate `error_status = :unprocessable_content` instead of `:unprocessable_entity`. Existing apps are handled by [latest versions of Rails, which will now transparently convert the code under the hood to avoid the Rack warning](https://github.com/rails/rails/pull/53383), and Devise will use that translation layer when available in the failure app to prevent the warning there as well (since that isn't covered by Rails automatic conversion). Signed-off-by: Carlos Antonio da Silva <carlosantoniodasilva@gmail.com>
41 lines
1.1 KiB
Ruby
41 lines
1.1 KiB
Ruby
# frozen_string_literal: true
|
|
|
|
require 'test_helper'
|
|
|
|
require "generators/devise/devise_generator"
|
|
|
|
class DeviseGeneratorTest < Rails::Generators::TestCase
|
|
tests Devise::Generators::DeviseGenerator
|
|
destination File.expand_path("../../tmp", __FILE__)
|
|
|
|
setup do
|
|
prepare_destination
|
|
copy_routes
|
|
end
|
|
|
|
test "route generation for simple model names" do
|
|
run_generator %w(monster name:string)
|
|
assert_file "config/routes.rb", /devise_for :monsters/
|
|
end
|
|
|
|
test "route generation for namespaced model names" do
|
|
run_generator %w(monster/goblin name:string)
|
|
match = /devise_for :goblins, class_name: "Monster::Goblin"/
|
|
assert_file "config/routes.rb", match
|
|
end
|
|
|
|
test "route generation with skip routes" do
|
|
run_generator %w(monster name:string --skip-routes)
|
|
match = /devise_for :monsters, skip: :all/
|
|
assert_file "config/routes.rb", match
|
|
end
|
|
|
|
def copy_routes
|
|
routes = File.expand_path("../../rails_app/config/routes.rb", __FILE__)
|
|
destination = File.join(destination_root, "config")
|
|
|
|
FileUtils.mkdir_p(destination)
|
|
FileUtils.cp routes, destination
|
|
end
|
|
end
|