quapp-common 0.0.14.dev1__tar.gz → 0.0.14.dev2__tar.gz

{quapp_common-0.0.14.dev1/quapp_common.egg-info → quapp_common-0.0.14.dev2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: quapp-common
-Version: 0.0.14.dev1
+Version: 0.0.14.dev2
 Summary: Quapp common library supporting Quapp Platform for Quantum Computing
 Author-email: "CITYNOW Co. Ltd. " <corp@citynow.vn>
 License: The MIT License (MIT)
@@ -114,6 +114,56 @@ Notes:
 
 ## Recently Changes Highlights
 
+### v0.0.14.dev2 — Fix: log lỗi bị nhân đôi
+
+- **Fix**: mỗi lần lỗi phát **hai** dòng ERROR cùng `event=job_failed`, mỗi dòng
+  kèm một traceback riêng. Nguyên nhân: `build_error_job_response()` (thêm ở
+  `0.0.13.dev9`) tự phát một dòng log tập trung, nhưng 10 call-site vẫn giữ
+  `self.logger.exception(...)` của chúng.
+
+  Ngoài việc làm ồn Function Log và nhân đôi lượng traceback, nó **làm sai metric
+  nếu backend đếm `event=job_failed`** để tính tỉ lệ job thất bại.
+
+  Đã gỡ log ở 10 call-site trong `device.py`, `custom_device.py`,
+  `invocation.py` (×3), `job_fetcher.py` (×2), `job_fetching.py` (×2),
+  `post_processing_task.py`. Dòng log tập trung mang nhiều thông tin hơn bản bị
+  gỡ: `event` / `category` / `error_code` / `phase` + full traceback.
+
+  `phase_failed` (WARNING, mốc timeline) là event khác, không bị tính là trùng.
+
+- **Fix (security)**: credential lọt vào log text qua **hai đường** mà
+  `redact()` không chặn được:
+
+  1. `loguru diagnose=True` (default) in giá trị **biến cục bộ** trong traceback
+     dưới dạng `└ 'value'`. Không có tên key nên `redact()` — vốn match
+     `key: value` — không mask. Token provider nằm trong biến cục bộ lọt nguyên.
+     Đã set `diagnose=False` cho sink text (`backtrace=True` vẫn giữ: nó mở rộng
+     call chain nhưng không in giá trị).
+  2. Placeholder `{exception}` được loguru render **trực tiếp từ exception
+     object**, không đi qua filter. Secret nằm trong `str(exception)` — ví dụ
+     response body của provider bị nhét vào `ValueError` — lọt nguyên. Sink text
+     nay append traceback đã redact ở `extra[_exc_text]` thay cho placeholder đó.
+
+  Sink JSON không bị ảnh hưởng: nó đã dùng `_exc_text` và `diagnose=False`.
+
+- **Docs**: ghi rõ trong docstring nên chọn `QUAPP_LOG_FORMAT` nào. Với
+  Grafana/Loki phải dùng `json`: ở chế độ `text`, traceback là các dòng **riêng
+  không có prefix** `[ConsoleJobLog][<job_id>]`, nên filter theo job_id sẽ mất
+  toàn bộ stack. `both` ghi mỗi record hai lần — chỉ dùng khi debug.
+
+- **Test**: thêm `tests/test_log_secret_leak.py` — 5 case, test qua sink THẬT do
+  `configure_logging()` tạo (không introspect internals). Có một case khoá lại
+  hành vi `diagnose=True` của loguru, để nếu upstream đổi thì ta biết.
+
+- **Test**: thêm `tests/test_error_log_dedupe.py` — khoá hồi quy ở hai mức: đếm
+  số dòng `job_failed` khi chạy thật, **và** quét source để bắt trường hợp ai đó
+  thêm lại `logger.exception()` ngay trước `build_error_job_response()`.
+
+  Kèm test cho trường hợp **nhiều job đồng thời**: 4 job qua
+  `run_in_threadpool` + `circuit_running_pool` phải có số dòng log đều nhau,
+  không job nào nhận log của job khác, và `phase` giữ được khi đi vào thread
+  pool.
+
 ### v0.0.14.dev1 — Error taxonomy & error envelope (Phase 1)
 
 Tiếp nối Phase 0. Bổ sung phân loại lỗi có ngữ nghĩa và envelope dùng chung cho

{quapp_common-0.0.14.dev1 → quapp_common-0.0.14.dev2}/README.md

@@ -73,6 +73,56 @@ Notes:
 
 ## Recently Changes Highlights
 
+### v0.0.14.dev2 — Fix: log lỗi bị nhân đôi
+
+- **Fix**: mỗi lần lỗi phát **hai** dòng ERROR cùng `event=job_failed`, mỗi dòng
+  kèm một traceback riêng. Nguyên nhân: `build_error_job_response()` (thêm ở
+  `0.0.13.dev9`) tự phát một dòng log tập trung, nhưng 10 call-site vẫn giữ
+  `self.logger.exception(...)` của chúng.
+
+  Ngoài việc làm ồn Function Log và nhân đôi lượng traceback, nó **làm sai metric
+  nếu backend đếm `event=job_failed`** để tính tỉ lệ job thất bại.
+
+  Đã gỡ log ở 10 call-site trong `device.py`, `custom_device.py`,
+  `invocation.py` (×3), `job_fetcher.py` (×2), `job_fetching.py` (×2),
+  `post_processing_task.py`. Dòng log tập trung mang nhiều thông tin hơn bản bị
+  gỡ: `event` / `category` / `error_code` / `phase` + full traceback.
+
+  `phase_failed` (WARNING, mốc timeline) là event khác, không bị tính là trùng.
+
+- **Fix (security)**: credential lọt vào log text qua **hai đường** mà
+  `redact()` không chặn được:
+
+  1. `loguru diagnose=True` (default) in giá trị **biến cục bộ** trong traceback
+     dưới dạng `└ 'value'`. Không có tên key nên `redact()` — vốn match
+     `key: value` — không mask. Token provider nằm trong biến cục bộ lọt nguyên.
+     Đã set `diagnose=False` cho sink text (`backtrace=True` vẫn giữ: nó mở rộng
+     call chain nhưng không in giá trị).
+  2. Placeholder `{exception}` được loguru render **trực tiếp từ exception
+     object**, không đi qua filter. Secret nằm trong `str(exception)` — ví dụ
+     response body của provider bị nhét vào `ValueError` — lọt nguyên. Sink text
+     nay append traceback đã redact ở `extra[_exc_text]` thay cho placeholder đó.
+
+  Sink JSON không bị ảnh hưởng: nó đã dùng `_exc_text` và `diagnose=False`.
+
+- **Docs**: ghi rõ trong docstring nên chọn `QUAPP_LOG_FORMAT` nào. Với
+  Grafana/Loki phải dùng `json`: ở chế độ `text`, traceback là các dòng **riêng
+  không có prefix** `[ConsoleJobLog][<job_id>]`, nên filter theo job_id sẽ mất
+  toàn bộ stack. `both` ghi mỗi record hai lần — chỉ dùng khi debug.
+
+- **Test**: thêm `tests/test_log_secret_leak.py` — 5 case, test qua sink THẬT do
+  `configure_logging()` tạo (không introspect internals). Có một case khoá lại
+  hành vi `diagnose=True` của loguru, để nếu upstream đổi thì ta biết.
+
+- **Test**: thêm `tests/test_error_log_dedupe.py` — khoá hồi quy ở hai mức: đếm
+  số dòng `job_failed` khi chạy thật, **và** quét source để bắt trường hợp ai đó
+  thêm lại `logger.exception()` ngay trước `build_error_job_response()`.
+
+  Kèm test cho trường hợp **nhiều job đồng thời**: 4 job qua
+  `run_in_threadpool` + `circuit_running_pool` phải có số dòng log đều nhau,
+  không job nào nhận log của job khác, và `phase` giữ được khi đi vào thread
+  pool.
+
 ### v0.0.14.dev1 — Error taxonomy & error envelope (Phase 1)
 
 Tiếp nối Phase 0. Bổ sung phân loại lỗi có ngữ nghĩa và envelope dùng chung cho

{quapp_common-0.0.14.dev1 → quapp_common-0.0.14.dev2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "quapp-common"
-version = "0.0.14.dev1"
+version = "0.0.14.dev2"
 description = "Quapp common library supporting Quapp Platform for Quantum Computing"
 readme = "README.md"
 authors = [{ name = "CITYNOW Co. Ltd. ", email = "corp@citynow.vn" }]

{quapp_common-0.0.14.dev1 → quapp_common-0.0.14.dev2}/quapp_common/async_tasks/post_processing_task.py

@@ -45,9 +45,6 @@ class PostProcessingTask(AsyncTask):
             update_job_metadata(job_response, self.promise.callback_url.on_done)
 
         except Exception as exception:
-            self.logger.bind(event='job_failed').exception(
-                    f"Post-processing failed: {exception}")
-
             job_response = build_error_job_response(
                     exception, job_response,
                     message='Error when post processing job result',

{quapp_common-0.0.14.dev1 → quapp_common-0.0.14.dev2}/quapp_common/component/backend/invocation.py

@@ -94,10 +94,6 @@ class Invocation(ABC):
                     f"Backend data prepared: device {self.backend_information.device_name}, provider {self.backend_information.provider_tag.value}")
 
         except Exception as exception:
-            self.logger.exception(
-                f"Error when prepare backend data: {exception}",
-                exc_info=True)
-
             job_response = build_error_job_response(exception,
                                                     message='Error when prepare backend data',
                                                     stage='while preparing the backend',
@@ -143,10 +139,6 @@ class Invocation(ABC):
                 device = self._create_device(provider)
 
         except Exception as exception:
-            self.logger.exception(
-                    f"Exception when create provider or device: {exception}",
-                    exc_info=True)
-
             job_response = build_error_job_response(exception,
                                                     message='Error when create provider or device',
                                                     stage='while running the circuit on the device',
@@ -199,9 +191,6 @@ class Invocation(ABC):
             return circuit
 
         except Exception as exception:
-            self.logger.exception(f'Error when prepare circuit: {exception}',
-                                  exc_info=True)
-
             job_response = build_error_job_response(exception, job_response,
                                                     message='Error when prepare circuit',
                                                     stage='while preparing the circuit',

{quapp_common-0.0.14.dev1 → quapp_common-0.0.14.dev2}/quapp_common/component/backend/job_fetcher.py

@@ -128,10 +128,6 @@ class JobFetcher(ABC):
                 job_response.status_code = StatusCode.POLLING
 
         except Exception as exception:
-            self.logger.exception(
-                    f"Exception during job fetch for provider_job_id {self.provider_job_id}: {exception}",
-                    exc_info=True)
-
             job_response = build_error_job_response(
                     exception, job_response,
                     message='Error when fetching job with provider_job_id {0}'.format(
@@ -209,8 +205,6 @@ class JobFetcher(ABC):
             return job_response
 
         except Exception as exception:
-            self.logger.exception(f"Exception during analysis: {exception}",
-                              exc_info=True)
             from ...util.response_utils import build_error_job_response
             job_response = build_error_job_response(
                     exception, job_response,

{quapp_common-0.0.14.dev1 → quapp_common-0.0.14.dev2}/quapp_common/component/backend/job_fetching.py

@@ -90,10 +90,6 @@ class JobFetching(ABC):
                 job_response.status_code = StatusCode.POLLING
 
         except Exception as exception:
-            self.logger.bind(event='job_failed').exception(
-                    "Exception when fetch job with provider_job_id {0}: {1}".format(
-                            self.provider_job_id, str(exception)))
-
             job_response = build_error_job_response(
                     exception, job_response,
                     message='Exception when fetch job with provider_job_id {0}'.format(
@@ -156,10 +152,6 @@ class JobFetching(ABC):
             return job_response
 
         except Exception as exception:
-            self.logger.exception(
-                    "Exception when analyst job result with provider_job_id {0}: {1}".format(
-                            self.provider_job_id, str(exception)))
-
             job_response = build_error_job_response(
                     exception, job_response,
                     message='Exception when analyst job result with provider_job_id {0}'.format(

{quapp_common-0.0.14.dev1 → quapp_common-0.0.14.dev2}/quapp_common/config/logging_config.py

@@ -25,6 +25,17 @@ Ba vấn đề được xử lý ở đây:
 Biến môi trường:
     QUAPP_LOG_LEVEL   DEBUG (default) | INFO | WARNING | ERROR
     QUAPP_LOG_FORMAT  text (default) | json | both
+
+Chọn format nào:
+
+- ``json`` -- dùng cho môi trường có Grafana/Loki. Mỗi record là MỘT dòng, luôn
+  có field ``jobId``, và traceback nằm trong ``exc.stackTrace`` của cùng dòng đó
+  nên filter theo ``jobId`` không làm mất stack.
+- ``text`` -- dễ đọc bằng mắt qua ``docker logs``, nhưng traceback là các dòng
+  RIÊNG không có prefix ``[ConsoleJobLog][<job_id>]``. Filter theo job_id trên
+  Loki sẽ chỉ thấy dòng message và mất toàn bộ traceback.
+- ``both`` -- CHỈ dùng khi debug. Mỗi record được ghi hai lần (một text, một
+  JSON), nên hệ thống thu log sẽ thấy mọi dòng nhân đôi.
 """
 
 import inspect
@@ -149,15 +160,22 @@ def __custom_format(record) -> str:
     level_color = level_colors.get(record['level'].name, 'white')
     phase = f"[{extra['phase']}]" if extra.get('phase') else ''
 
-    # `{exception}` là bắt buộc: khi format là một callable, loguru KHÔNG tự
-    # append traceback. Format cũ thiếu placeholder này, nên mọi
-    # logger.exception(...) trong codebase đều không in ra stack trace nào --
-    # đúng phần thông tin cần nhất khi debug.
+    # Traceback phải được append tường minh: khi format là một callable, loguru
+    # KHÔNG tự thêm. Format cũ thiếu hẳn phần này nên mọi logger.exception()
+    # trong codebase không in ra stack trace nào.
+    #
+    # Dùng extra[_exc_text] (do _prepare_record format sẵn VÀ đã redact) thay vì
+    # placeholder `{exception}` của loguru: `{exception}` được loguru render trực
+    # tiếp từ exception object, không đi qua redact(), nên secret nằm trong
+    # message của exception -- ví dụ response body của provider -- sẽ lọt nguyên
+    # vào log.
+    tail = '{extra[' + EXC_TEXT + ']}' if EXC_TEXT in extra else ''
+
     return (f"<yellow>[ConsoleJobLog][{context}]{phase}</yellow> "
             f"<{level_color}>{{level}}</{level_color}> : "
             f"<green>{{time:YYYY-MM-DD HH:mm:ss.SSS}}</green> : "
             f"<{level_color}>{{message}}</{level_color}>: "
-            f"<magenta>{{process}}</magenta>\n{{exception}}")
+            f"<magenta>{{process}}</magenta>\n{tail}")
 
 
 # --------------------------------------------------------------------- json sink
@@ -273,8 +291,14 @@ def configure_logging(force: bool = False) -> None:
     logger.remove()
 
     if LOG_FORMAT in ('text', 'both'):
+        # diagnose=False là BẮT BUỘC: loguru diagnose in giá trị biến cục bộ
+        # trong traceback dưới dạng `└ 'value'`. Không có tên key nên hàm
+        # redact() -- vốn match `key: value` -- không mask được, và token của
+        # provider nằm trong biến cục bộ sẽ lọt nguyên vào log.
+        # backtrace=True vẫn giữ: nó mở rộng call chain nhưng không in giá trị.
         logger.add(sink=sys.stderr, format=__custom_format, level=LOG_LEVEL,
-                   colorize=True, filter=_prepare_record, enqueue=True)
+                   colorize=True, filter=_prepare_record, enqueue=True,
+                   backtrace=True, diagnose=False)
 
     if LOG_FORMAT in ('json', 'both'):
         logger.add(sink=_json_sink, level=LOG_LEVEL, filter=_prepare_record,

{quapp_common-0.0.14.dev1 → quapp_common-0.0.14.dev2}/quapp_common/model/device/custom_device.py

@@ -56,10 +56,6 @@ class CustomDevice(Device, ABC):
             return job_response
 
         except Exception as exception:
-            self.logger.exception(
-                    f"Exception when analyzing job result: {exception}",
-                    exc_info=True)
-
             job_response = build_error_job_response(
                     exception, job_response,
                     message='Error when analyzing job result',

{quapp_common-0.0.14.dev1 → quapp_common-0.0.14.dev2}/quapp_common/model/device/device.py

@@ -149,9 +149,6 @@ class Device(ABC):
             # của build_error_job_response -- mất traceback, mất phân loại, và
             # log ở mức user thường không bật. Đây lại là đường lỗi quan trọng
             # nhất (submit circuit lên device), nên phải dùng đường chung.
-            self.logger.bind(event='job_failed').exception(
-                    f"Execute job failed: {exception}")
-
             job_response = build_error_job_response(
                     exception, job_response,
                     message='Error when executing job on device',

{quapp_common-0.0.14.dev1 → quapp_common-0.0.14.dev2/quapp_common.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: quapp-common
-Version: 0.0.14.dev1
+Version: 0.0.14.dev2
 Summary: Quapp common library supporting Quapp Platform for Quantum Computing
 Author-email: "CITYNOW Co. Ltd. " <corp@citynow.vn>
 License: The MIT License (MIT)
@@ -114,6 +114,56 @@ Notes:
 
 ## Recently Changes Highlights
 
+### v0.0.14.dev2 — Fix: log lỗi bị nhân đôi
+
+- **Fix**: mỗi lần lỗi phát **hai** dòng ERROR cùng `event=job_failed`, mỗi dòng
+  kèm một traceback riêng. Nguyên nhân: `build_error_job_response()` (thêm ở
+  `0.0.13.dev9`) tự phát một dòng log tập trung, nhưng 10 call-site vẫn giữ
+  `self.logger.exception(...)` của chúng.
+
+  Ngoài việc làm ồn Function Log và nhân đôi lượng traceback, nó **làm sai metric
+  nếu backend đếm `event=job_failed`** để tính tỉ lệ job thất bại.
+
+  Đã gỡ log ở 10 call-site trong `device.py`, `custom_device.py`,
+  `invocation.py` (×3), `job_fetcher.py` (×2), `job_fetching.py` (×2),
+  `post_processing_task.py`. Dòng log tập trung mang nhiều thông tin hơn bản bị
+  gỡ: `event` / `category` / `error_code` / `phase` + full traceback.
+
+  `phase_failed` (WARNING, mốc timeline) là event khác, không bị tính là trùng.
+
+- **Fix (security)**: credential lọt vào log text qua **hai đường** mà
+  `redact()` không chặn được:
+
+  1. `loguru diagnose=True` (default) in giá trị **biến cục bộ** trong traceback
+     dưới dạng `└ 'value'`. Không có tên key nên `redact()` — vốn match
+     `key: value` — không mask. Token provider nằm trong biến cục bộ lọt nguyên.
+     Đã set `diagnose=False` cho sink text (`backtrace=True` vẫn giữ: nó mở rộng
+     call chain nhưng không in giá trị).
+  2. Placeholder `{exception}` được loguru render **trực tiếp từ exception
+     object**, không đi qua filter. Secret nằm trong `str(exception)` — ví dụ
+     response body của provider bị nhét vào `ValueError` — lọt nguyên. Sink text
+     nay append traceback đã redact ở `extra[_exc_text]` thay cho placeholder đó.
+
+  Sink JSON không bị ảnh hưởng: nó đã dùng `_exc_text` và `diagnose=False`.
+
+- **Docs**: ghi rõ trong docstring nên chọn `QUAPP_LOG_FORMAT` nào. Với
+  Grafana/Loki phải dùng `json`: ở chế độ `text`, traceback là các dòng **riêng
+  không có prefix** `[ConsoleJobLog][<job_id>]`, nên filter theo job_id sẽ mất
+  toàn bộ stack. `both` ghi mỗi record hai lần — chỉ dùng khi debug.
+
+- **Test**: thêm `tests/test_log_secret_leak.py` — 5 case, test qua sink THẬT do
+  `configure_logging()` tạo (không introspect internals). Có một case khoá lại
+  hành vi `diagnose=True` của loguru, để nếu upstream đổi thì ta biết.
+
+- **Test**: thêm `tests/test_error_log_dedupe.py` — khoá hồi quy ở hai mức: đếm
+  số dòng `job_failed` khi chạy thật, **và** quét source để bắt trường hợp ai đó
+  thêm lại `logger.exception()` ngay trước `build_error_job_response()`.
+
+  Kèm test cho trường hợp **nhiều job đồng thời**: 4 job qua
+  `run_in_threadpool` + `circuit_running_pool` phải có số dòng log đều nhau,
+  không job nào nhận log của job khác, và `phase` giữ được khi đi vào thread
+  pool.
+
 ### v0.0.14.dev1 — Error taxonomy & error envelope (Phase 1)
 
 Tiếp nối Phase 0. Bổ sung phân loại lỗi có ngữ nghĩa và envelope dùng chung cho

{quapp_common-0.0.14.dev1 → quapp_common-0.0.14.dev2}/quapp_common.egg-info/SOURCES.txt

@@ -92,10 +92,12 @@ quapp_common/util/response_utils.py
 tests/test_circuit_adapter.py
 tests/test_dispatcher_runner_error.py
 tests/test_error_classifier.py
+tests/test_error_log_dedupe.py
 tests/test_error_report.py
 tests/test_invocation_failure.py
 tests/test_job_heartbeat.py
 tests/test_json_parser_utils.py
 tests/test_language_dispatcher.py
+tests/test_log_secret_leak.py
 tests/test_logging_config.py
 tests/test_result_serializer.py

quapp_common-0.0.14.dev2/tests/test_error_log_dedupe.py

@@ -0,0 +1,176 @@
+#  Quapp Platform Project
+#  test_error_log_dedupe.py
+#  Copyright © CITYNOW Co. Ltd. All rights reserved.
+
+"""Mỗi lần lỗi chỉ được phát ĐÚNG MỘT dòng ERROR event=job_failed.
+
+Trước đây call-site log một lần rồi build_error_job_response() log lần nữa, nên
+mỗi lỗi ra 2 dòng ERROR cùng event, mỗi dòng kèm một traceback. Ngoài chuyện ồn,
+nó làm sai metric nếu backend đếm event=job_failed.
+
+Test này cũng khoá luôn hành vi khi nhiều job chạy đồng thời: contextvars phải
+cách ly, không job nào nhận log của job khác.
+"""
+
+import asyncio
+import re
+from collections import Counter
+from pathlib import Path
+
+import pytest
+from loguru import logger
+
+from quapp_common.config import log_context
+from quapp_common.config.log_context import bind_job, phase
+from quapp_common.config.logging_config import _prepare_record, configure_logging
+from quapp_common.config.thread_config import circuit_running_pool
+from quapp_common.enum.runtime_phase import RuntimePhase
+from quapp_common.util.response_utils import build_error_job_response
+
+SOURCE_ROOT = Path(__file__).resolve().parent.parent / 'quapp_common'
+
+# File có khối except gọi build_error_job_response().
+CALL_SITE_FILES = (
+    'model/device/device.py',
+    'model/device/custom_device.py',
+    'component/backend/invocation.py',
+    'component/backend/job_fetcher.py',
+    'component/backend/job_fetching.py',
+    'async_tasks/post_processing_task.py',
+)
+
+
+@pytest.fixture(autouse=True)
+def clean_context():
+    log_context.reset()
+    yield
+    log_context.reset()
+
+
+@pytest.fixture
+def captured():
+    records = []
+    logger.remove()
+    sink_id = logger.add(
+            lambda message: records.append(
+                    dict(message.record['extra'],
+                         _level=message.record['level'].name,
+                         _message=message.record['message'])),
+            level='DEBUG', filter=_prepare_record)
+    yield records
+    logger.remove(sink_id)
+    configure_logging(force=True)
+
+
+def raise_and_catch(exception):
+    try:
+        raise exception
+    except BaseException as caught:
+        return caught
+
+
+# ------------------------------------------------------- mot loi = mot dong
+
+def test_single_job_failed_line_per_error(captured):
+    bind_job('job_1')
+
+    build_error_job_response(raise_and_catch(ValueError('boom')),
+                             message='Error when executing job on device',
+                             phase=RuntimePhase.EXECUTION)
+
+    failed = [r for r in captured if r.get('event') == 'job_failed']
+    assert len(failed) == 1, f'mong doi 1 dong, nhan {len(failed)}'
+    assert failed[0]['_level'] == 'ERROR'
+
+
+def test_job_failed_line_carries_classification(captured):
+    bind_job('job_1')
+
+    build_error_job_response(raise_and_catch(KeyError('shots')),
+                             phase=RuntimePhase.PREPROCESSING)
+
+    failed = next(r for r in captured if r.get('event') == 'job_failed')
+    assert failed['category'] == 'INVALID_INPUT'
+    assert failed['error_code'] == 'INPUT_MISSING_FIELD'
+    assert failed['phase'] == 'PREPROCESSING'
+    assert failed['job_id'] == 'job_1'
+
+
+def test_phase_failed_is_separate_and_single(captured):
+    """phase_failed là mốc timeline, khác job_failed -- không tính là trùng."""
+    bind_job('job_1')
+
+    with pytest.raises(ValueError):
+        with phase(RuntimePhase.COMPILATION):
+            raise ValueError('transpile failed')
+
+    events = Counter(r.get('event') for r in captured)
+    assert events['phase_failed'] == 1
+    assert events['job_failed'] == 0
+
+
+# --------------------------------------------------- khong con log o call-site
+
+@pytest.mark.parametrize('relative', CALL_SITE_FILES)
+def test_no_redundant_logger_exception_before_builder(relative):
+    """Khoá hồi quy ở mức source: không được log exception ngay trước
+    build_error_job_response()."""
+    source = (SOURCE_ROOT / relative).read_text(encoding='utf-8')
+
+    pattern = re.compile(
+            r'self\.logger(?:\.bind\([^)]*\))?\.exception\('
+            r'(?:[^()]|\([^()]*\))*\)\s*\n\s*\n?\s*'
+            r'(?:job_response\s*=\s*)?build_error_job_response',
+            re.MULTILINE)
+
+    assert not pattern.search(source), (
+            f'{relative}: con log exception ngay truoc '
+            f'build_error_job_response -- se ra 2 dong ERROR cho mot loi')
+
+
+# ------------------------------------------------------------ dong thoi
+
+def test_concurrent_jobs_do_not_duplicate_or_mix(captured):
+    """4 job đồng thời: mỗi job đúng số dòng của nó, không lẫn context."""
+
+    def work(job_id):
+        bind_job(job_id, sdk='qiskit')
+        with phase(RuntimePhase.PREPROCESSING):
+            log_context.logger if False else None
+            from quapp_common.config.logging_config import job_logger
+            job_logger().info(f'step for {job_id}')
+        circuit_running_pool.submit(_analysis, job_id).result()
+
+    def _analysis(job_id):
+        from quapp_common.config.logging_config import job_logger
+        with phase(RuntimePhase.ANALYSIS):
+            job_logger().info(f'analysis for {job_id}')
+
+    async def run():
+        from starlette.concurrency import run_in_threadpool
+        await asyncio.gather(*[run_in_threadpool(work, f'job_{i}')
+                               for i in range(4)])
+
+    asyncio.run(run())
+
+    per_job = Counter(r.get('job_id') for r in captured)
+    assert set(per_job) == {'job_0', 'job_1', 'job_2', 'job_3'}
+    assert len(set(per_job.values())) == 1, \
+        f'so dong khong deu giua cac job: {dict(per_job)}'
+
+    # Không dòng nào mang job_id của job khác so với message của nó.
+    mixed = [r for r in captured
+             if r.get('job_id') and 'job_' in r['_message']
+             and r['job_id'] not in r['_message']]
+    assert not mixed, f'{len(mixed)} dong lan context giua cac job'
+
+
+def test_thread_pool_keeps_phase(captured):
+    """ThreadPoolExecutor thuần không copy contextvars; pool phải bọc lại."""
+    bind_job('job_pool')
+
+    def read_phase():
+        return log_context.current_phase_value()
+
+    with phase(RuntimePhase.ANALYSIS):
+        assert circuit_running_pool.submit(read_phase).result() == 'ANALYSIS'

quapp_common-0.0.14.dev2/tests/test_log_secret_leak.py

@@ -0,0 +1,160 @@
+#  Quapp Platform Project
+#  test_log_secret_leak.py
+#  Copyright © CITYNOW Co. Ltd. All rights reserved.
+
+"""Credential không được lọt vào log qua bất kỳ đường nào.
+
+Có hai đường rò khác nhau và cần chặn bằng hai cơ chế khác nhau:
+
+1. Secret nằm trong MESSAGE hoặc trong str(exception) -- dạng ``key: value``.
+   Chặn bằng ``redact()`` trong filter.
+
+2. Secret nằm trong BIẾN CỤC BỘ của frame bị lỗi. loguru ``diagnose=True`` in
+   giá trị biến ra traceback dưới dạng ``└ 'value'`` -- không có tên key nên
+   ``redact()`` không match. Chặn bằng ``diagnose=False`` ở sink.
+"""
+
+import io
+
+import pytest
+from loguru import logger
+
+from quapp_common.config import log_context, logging_config
+from quapp_common.config.log_context import bind_job
+from quapp_common.config.logging_config import (REDACTED, _prepare_record,
+                                                configure_logging, job_logger)
+
+TOKEN = 'glpat-SUPERSECRET-DO-NOT-LEAK'
+
+
+@pytest.fixture(autouse=True)
+def clean_context():
+    log_context.reset()
+    yield
+    log_context.reset()
+    configure_logging(force=True)
+
+
+def raw_sink(**add_kwargs) -> io.StringIO:
+    """Sink THÔ với format mặc định của loguru.
+
+    Chỉ dùng để khoá hành vi của chính loguru. Lưu ý: sink kiểu này KHÔNG được
+    bảo vệ -- format mặc định render `{exception}` trực tiếp từ exception object,
+    bỏ qua redact(). Bảo vệ của ta nằm ở __custom_format + _prepare_record, nên
+    mọi sink phải được tạo qua configure_logging().
+    """
+    buffer = io.StringIO()
+    logger.remove()
+    logger.add(buffer, level='DEBUG', filter=_prepare_record, colorize=False,
+               **add_kwargs)
+    return buffer
+
+
+@pytest.fixture
+def real_sink(monkeypatch) -> io.StringIO:
+    """Sink text THẬT do configure_logging() tạo.
+
+    Sink bind vào sys.stderr lúc logger.add(), nên phải thay sys.stderr trước
+    khi cấu hình lại.
+    """
+    buffer = io.StringIO()
+    monkeypatch.setattr('sys.stderr', buffer)
+    monkeypatch.setattr(logging_config, 'LOG_FORMAT', 'text')
+    configure_logging(force=True)
+    return buffer
+
+
+def _connect(api_token):
+    """api_token là biến cục bộ -- đúng thứ diagnose=True sẽ in ra."""
+    raise ConnectionError('provider refused the connection')
+
+
+# ------------------------------------------------- (1) secret trong message
+
+def test_secret_in_message_is_redacted(real_sink):
+    bind_job('job_1')
+
+    job_logger().info(f"connecting with token: {TOKEN}")
+
+    logger.complete()
+    output = real_sink.getvalue()
+    assert TOKEN not in output
+    assert REDACTED in output
+
+
+def test_secret_in_exception_string_is_redacted(real_sink):
+    """Secret nằm trong str(exception) -- ví dụ response body của provider bị
+    nhét vào ValueError. loguru render `{exception}` trực tiếp từ exception
+    object nên không qua redact(); vì vậy sink text phải dùng traceback đã
+    redact ở extra[_exc_text] thay cho placeholder đó."""
+    bind_job('job_1')
+
+    try:
+        raise ValueError(f"auth failed for apiKey={TOKEN}")
+    except ValueError:
+        job_logger().exception('boom')
+
+    logger.complete()
+    output = real_sink.getvalue()
+    assert TOKEN not in output
+    assert REDACTED in output
+    assert 'Traceback' in output
+
+
+# --------------------------------------- (2) secret trong bien cuc bo
+
+def test_diagnose_true_would_leak_local_variables():
+    """Khoá lại hành vi của loguru để test có ý nghĩa: với diagnose=True thì
+    token LỌT. Nếu loguru đổi hành vi này, test sẽ báo và ta biết có thể bật lại."""
+    buffer = raw_sink(backtrace=True, diagnose=True)
+    bind_job('job_1')
+
+    try:
+        _connect(TOKEN)
+    except ConnectionError:
+        job_logger().exception('provider connection failed')
+
+    assert TOKEN in buffer.getvalue(), \
+        'loguru khong con in gia tri bien cuc bo -- xem lai co can diagnose=False'
+
+
+def test_diagnose_false_does_not_leak_local_variables():
+    buffer = raw_sink(backtrace=True, diagnose=False)
+    bind_job('job_1')
+
+    try:
+        _connect(TOKEN)
+    except ConnectionError:
+        job_logger().exception('provider connection failed')
+
+    assert TOKEN not in buffer.getvalue()
+
+
+
+
+# ------------------------------------------------- json sink dung traceback sach
+
+def test_json_stack_trace_has_no_variable_values():
+    """stackTrace trong JSON lấy từ traceback.format_exception (chuẩn), không
+    kèm giá trị biến."""
+    buffer = raw_sink()
+    bind_job('job_1')
+
+    try:
+        _connect(TOKEN)
+    except ConnectionError:
+        job_logger().exception('provider connection failed')
+
+    records = []
+    logger.remove()
+    logger.add(lambda message: records.append(message.record), level='DEBUG',
+               filter=_prepare_record)
+    bind_job('job_2')
+    try:
+        _connect(TOKEN)
+    except ConnectionError:
+        job_logger().exception('provider connection failed')
+
+    stack = records[-1]['extra'][logging_config.EXC_TEXT]
+    assert 'Traceback' in stack
+    assert TOKEN not in stack