Fix mtls credentials warning (#558)

* fix warning in mtls credential case * add line for multiproc dir in quickstart documentation
fkie-cad · Apr 4, 2024 · eacb5f2 · eacb5f2
1 parent 8258bdc
commit eacb5f2
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/doc/source/getting_started.rst b/doc/source/getting_started.rst
@@ -95,6 +95,15 @@ Run without Logprep Container (default)
      .. code-block:: bash
 
       logprep run quickstart/exampledata/config/pipeline.yml
+
+    If logprep is run with the metrics enabled, the necessary environment variable has to be set first:
+
+    .. code-block:: bash
+
+      export PROMETHEUS_MULTIPROC_DIR="tmp/logprep"
+      logprep run quickstart/exampledata/config/pipeline.yml
+
+      
      
 
 Run with Logprep Container

diff --git a/logprep/util/credentials.py b/logprep/util/credentials.py
@@ -250,7 +250,7 @@ def _match_credentials(cls, credential_mapping: dict) -> "Credentials":
             }:
                 if extra_params:
                     cls._logger.warning(
-                        "Other parameters were given: %s but OAuth token authorization was chosen",
+                        "Other parameters were given: %s but mTLS authorization was chosen",
                         extra_params.keys(),
                     )
                 return MTLSCredentials(client_key=client_key, cert=cert, ca_cert=ca_cert)
@@ -261,7 +261,7 @@ def _match_credentials(cls, credential_mapping: dict) -> "Credentials":
             }:
                 if extra_params:
                     cls._logger.warning(
-                        "Other parameters were given: %s but OAuth token authorization was chosen",
+                        "Other parameters were given: %s but mTLS authorization was chosen",
                         extra_params.keys(),
                     )
                 return MTLSCredentials(client_key=client_key, cert=cert)