Move API URL to env

superdesk · Feb 21, 2025 · 3fbe988 · 3fbe988
1 parent 6517a8a
commit 3fbe988
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 3 deletions.
diff --git a/server/settings.py b/server/settings.py
@@ -55,6 +55,7 @@ def env(variable, fallback_value=None):
 
 SECRET_KEY = env("SECRET_KEY", "")
 STT_FIN_API_KEY = env("STT_FIN_API_KEY", "")
+STT_FIN_API_URL = env("STT_FIN_API_URL", "")
 
 NO_TAKES = True
 

diff --git a/server/stt/spellcheckers/stt_fin.py b/server/stt/spellcheckers/stt_fin.py
@@ -17,7 +17,7 @@
 
 logger = logging.getLogger(__name__)
 OPT_API_KEY = "STT_FIN_API_KEY"
-API_URL = "https://api.lingsoft.fi/lmc/{method}/"
+OPT_API_URL_KEY = "STT_FIN_API_URL"
 
 
 class SttFin(SpellcheckerBase):
@@ -34,10 +34,11 @@ class SttFin(SpellcheckerBase):
     def __init__(self, app):
         super().__init__(app)
         self.api_key = self.config.get(OPT_API_KEY, os.environ.get(OPT_API_KEY))
+        self.api_url = self.config.get(OPT_API_URL_KEY, os.environ.get(OPT_API_URL_KEY))
 
     def check(self, text, language=None):
         try:
-            check_url = API_URL.format(method="proof")
+            check_url = self.api_url.format(method="proof")
             data = {
                 "text": text,
                 "language": "fi",
@@ -114,7 +115,7 @@ def check(self, text, language=None):
 
     def suggest(self, text, language=None):
         try:
-            check_url = API_URL.format(method="proof")
+            check_url = self.api_url.format(method="proof")
             data = {
                 "text": text,
                 "language": "fi",
@@ -152,6 +153,13 @@ def available(self):
                 )
             )
             return False
+        if not self.api_url:
+            logger.warning(
+                "API URL is not set for {label}, please set {opt} variable to use it".format(
+                    label=self.label, opt=OPT_API_URL_KEY
+                )
+            )
+            return False
         return True