Move translation service config to config_for yml

app/lib/translation_service.rb

@@ -8,17 +8,27 @@ class TranslationService
   class UnexpectedResponseError < Error; end
 
   def self.configured
-    if ENV['DEEPL_API_KEY'].present?
+    if configuration.deepl[:api_key].present?
-      TranslationService::DeepL.new(ENV.fetch('DEEPL_PLAN', 'free'), ENV['DEEPL_API_KEY'])
+      TranslationService::DeepL.new(
-    elsif ENV['LIBRE_TRANSLATE_ENDPOINT'].present?
+        configuration.deepl[:plan],
-      TranslationService::LibreTranslate.new(ENV['LIBRE_TRANSLATE_ENDPOINT'], ENV['LIBRE_TRANSLATE_API_KEY'])
+        configuration.deepl[:api_key]
+      )
+    elsif configuration.libre_translate[:endpoint].present?
+      TranslationService::LibreTranslate.new(
+        configuration.libre_translate[:endpoint],
+        configuration.libre_translate[:api_key]
+      )
     else
       raise NotConfiguredError
     end
   end
 
   def self.configured?
-    ENV['DEEPL_API_KEY'].present? || ENV['LIBRE_TRANSLATE_ENDPOINT'].present?
+    configuration.deepl[:api_key].present? || configuration.libre_translate[:endpoint].present?
+  end
+
+  def self.configuration
+    Rails.configuration.x.translation
   end
 
   def languages

config/application.rb

@@ -112,6 +112,8 @@ module Mastodon
       end
     end
 
+    config.x.translation = config_for(:translation)
+
     config.to_prepare do
       Doorkeeper::AuthorizationsController.layout 'modal'
       Doorkeeper::AuthorizedApplicationsController.layout 'admin'

config/translation.yml

@@ -0,0 +1,7 @@
+shared:
+  deepl:
+    api_key: <%= ENV.fetch('DEEPL_API_KEY', nil) %>
+    plan: <%= ENV.fetch('DEEPL_PLAN', 'free') %>
+  libre_translate:
+    api_key: <%= ENV.fetch('LIBRE_TRANSLATE_API_KEY', nil) %>
+    endpoint: <%= ENV.fetch('LIBRE_TRANSLATE_ENDPOINT', nil) %>