PR review changes

Author: Hmnt39

cognite/extractorutils/configtools/loaders.py

@@ -387,10 +387,19 @@ def _reload_file(self) -> None:
         try:
             with open(self.config_path, encoding="utf-8") as stream:
                 self._config_text = stream.read()
-        except UnicodeDecodeError as e:
-            _logger.info(f"Using locale default encoding : {e}")
-            with open(self.config_path) as stream:
-                self._config_text = stream.read()
+        except UnicodeDecodeError:
+            _logger.warning(
+                f"Config file '{self.config_path}' is not valid UTF-8. Falling back to system default encoding."
+            )
+            try:
+                with open(self.config_path) as stream:
+                    self._config_text = stream.read()
+            except Exception as e:
+                _logger.error(
+                    f"Failed to read '{self.config_path}' with both UTF-8 and system default encoding. "
+                    f"The file may be corrupt or in an unsupported format. Final error: {e}"
+                )
+                raise RuntimeError("Unable to read configuration file.") from e
 
     @property
     def cognite_client(self) -> CogniteClient | None:

tests/tests_unit/test_configtools.py

@@ -749,4 +749,4 @@ def test_configresolver_fallback_encoding(tmp_path: Path, caplog: pytest.LogCapt
         assert config.logger.file is not None
         assert config.logger.file.path is not None
         assert "café" in config.logger.file.path
-        assert any("Using locale default encoding" in r.message for r in caplog.records)
+        assert any("Falling back to system default encoding." in r.message for r in caplog.records)