route_translator 15.0.0 → 15.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5a393716835c2c542d7f158782dcf1dcbaf163185bb4a1f85ff1be5507accbbf
-  data.tar.gz: 9ea86dcbebec6a9cea19b280da5a821cfccd6951e5d4c5b79ca43582053c8869
+  metadata.gz: 5cca7e784d193668fe69a3b9eae57116700bc4699c1930a79f720de4ce206fbe
+  data.tar.gz: a852fa9a6c8c00bad0223199b1e3ec8159f5bf70774a5bf7b531519c25ccaf4c
 SHA512:
-  metadata.gz: b1fe1869d56070ea8de79618f8c6cd8fe5e6360b6be50637ac94ac9a13db13d6ce44f807a4745b4973cef5126049b1d7314d2c46401d7aa6af1d081cb63c9f84
-  data.tar.gz: d91df79e0dabed0f438757b364e467d85c1d1691c324e9098b5c1446805e6735a594e5d425d48345c47dcdb01f6438a30358a09a52195de93a46c425b5ccc944
+  metadata.gz: d85e32c58543f4ef062777a7257f5377f661acc37e4a003c05076824d487020f0c70742c7256262b824596b4247f88fab3ab3ae2f12edccd1610c3cff3236091
+  data.tar.gz: bc6d14c007375d9e0196f965781431c890793588293a23fe4e8ea95c669eca7afab4488e9a3bee3b0f95be72c044081b68fc1f0c5da4651ad68eca5eac64979f

data/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## 15.1.0 / 2025-06-15
+
+* [BUGFIX] Ensure `@localized` is always reset after yield ([#333](https://github.com/enriclluelles/route_translator/issues/333))
+* [ENHANCEMENT] Optimize host locale detection ([#337](https://github.com/enriclluelles/route_translator/pull/337))
+* [ENHANCEMENT] Add `locale_from_request` method ([#337](https://github.com/enriclluelles/route_translator/pull/337))
+
 ## 15.0.0 / 2025-06-12
 
 * [FEATURE] Drop Ruby < 3.1 support

data/README.md

@@ -385,6 +385,39 @@ scope ':country/:locale' do
 end
 ```
 
+## Using RouteTranslator with Devise
+
+When using RouteTranslator with Devise, you may notice that some authentication-related flash messages (such as errors after failed logins) are shown in the wrong language. This happens because Devise builds some messages in middleware, after your controller’s actions have completed. If you use RouteTranslator’s `around_action` (or legacy `around_filter`) to set the locale, the locale is reset after the controller yields, and middleware like Devise will see the default locale.
+
+This only affects you if Devise is mounted inside a localized block (for example, `localized { devise_for :users }`). If you do not use localized Devise routes, you do not need to change anything.
+
+To ensure Devise’s middleware uses the correct locale, some users suggest replacing RouteTranslator’s `around_action` with a `before_action`. However, this is discouraged, as it can leave your application in the wrong locale after the request and may cause subtle bugs.
+
+A better approach is to explicitly set the locale for Devise’s failure responses. Starting from Devise version 4.9.4, you can customize how Devise determines the locale for error messages, thanks to the ability to override the `i18n_locale` method in the failure app. This allows you to set the locale for Devise’s middleware without changing RouteTranslator’s recommended usage.
+
+Here’s an example:
+
+```rb
+# config/initializers/devise.rb
+
+Devise.setup do |config|
+  # ...
+
+  ActiveSupport.on_load(:devise_failure_app) do
+    def i18n_locale
+      RouteTranslator.locale_from_request(request)
+    end
+  end
+end
+```
+
+### Summary
+
+- You only need this workaround if Devise is mounted in a localized block.
+- Do not replace RouteTranslator’s `around_action` with a `before_action`.
+- Instead, set the locale for Devise failure responses as shown above.
+- The solution above requires Devise version 4.9.4 or higher, which allows customizing the `i18n_locale` method in the failure app.
+
 ## Testing
 Testing your controllers with routes-translator is easy, just add a locale parameter as `String` for your localized routes. Otherwise, an `ActionController::UrlGenerationError` will raise.
 

data/lib/route_translator/extensions/action_controller.rb

@@ -9,7 +9,8 @@ module RouteTranslator
     private
 
     def set_locale_from_url
-      locale_from_url = RouteTranslator.locale_from_params(params) || RouteTranslator::Host.locale_from_host(request.host)
+      locale_from_url = RouteTranslator.locale_from_request(request)
+
       if locale_from_url
         old_locale  = I18n.locale
         I18n.locale = locale_from_url

data/lib/route_translator/extensions/mapper.rb

@@ -8,6 +8,7 @@ module ActionDispatch
       def localized
         @localized = true
         yield
+      ensure
         @localized = false
       end
 

data/lib/route_translator/host.rb

@@ -18,11 +18,11 @@ module RouteTranslator
     module_function
 
     def locale_from_host(host)
-      locales = RouteTranslator.config.host_locales.each_with_object([]) do |(pattern, locale), result|
-        result << locale.to_sym if host&.match?(regex_for(pattern))
-      end
-      locales &= I18n.available_locales
-      locales.first&.to_sym
+      available_locales = I18n.available_locales
+
+      RouteTranslator.config.host_locales.find do |pattern, locale|
+        host&.match?(regex_for(pattern)) && available_locales.include?(locale&.to_sym)
+      end&.last&.to_sym
     end
 
     def lambdas_for_locale(locale)

data/lib/route_translator/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module RouteTranslator
-  VERSION = '15.0.0'
+  VERSION = '15.1.0'
 end

data/lib/route_translator.rb

@@ -81,6 +81,10 @@ module RouteTranslator
     locale if I18n.available_locales.include?(locale)
   end
 
+  def locale_from_request(request)
+    locale_from_params(request.params) || Host.locale_from_host(request.host)
+  end
+
   def deprecator
     @deprecator ||= ActiveSupport::Deprecation.new(RouteTranslator::VERSION, 'RouteTranslator')
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: route_translator
 version: !ruby/object:Gem::Version
-  version: 15.0.0
+  version: 15.1.0
 platform: ruby
 authors:
 - Geremia Taglialatela
@@ -88,7 +88,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.7
+rubygems_version: 3.6.9
 specification_version: 4
 summary: Translate your Rails routes in a simple manner
 test_files: []