python3-commons 0.25.0__tar.gz → 0.25.2__tar.gz

{python3_commons-0.25.0 → python3_commons-0.25.2}/.pre-commit-config.yaml

@@ -1,12 +1,12 @@
 repos:
   - repo: https://github.com/astral-sh/uv-pre-commit
-    rev: 0.12.10
+    rev: 0.12.23
     hooks:
       - id: uv-lock
   #      - id: uv-export
 
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.16.6
+    rev: v0.16.10
     hooks:
       # Run the linter.
       - id: ruff-check

python3_commons-0.25.2/.python-version

@@ -0,0 +1 @@
+==3.14.8

{python3_commons-0.25.0 → python3_commons-0.25.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python3-commons
-Version: 0.25.0
+Version: 0.25.2
 Summary: Re-usable Python3 code
 Author-email: Oleg Korsak <kamikaze.is.waiting.you@gmail.com>
 License-Expression: GPL-3.0-only
@@ -12,13 +12,13 @@ Requires-Python: <3.15.0,>=3.14.0
 Description-Content-Type: text/markdown
 License-File: LICENSE
 License-File: AUTHORS.rst
-Requires-Dist: msgpack~=1.2.2
-Requires-Dist: msgspec~=0.21.1
+Requires-Dist: msgpack~=1.2.3
+Requires-Dist: msgspec~=0.22.0
 Requires-Dist: pydantic-settings~=2.15.0
 Provides-Extra: all
 Requires-Dist: python3_commons[api-client,audit,authn,authz,cache,database,object-storage,soap-client]; extra == "all"
 Provides-Extra: api-client
-Requires-Dist: aiohttp[speedups]<3.15.0,>=3.14.3; extra == "api-client"
+Requires-Dist: aiohttp[speedups]<3.15.0,>=3.14.4; extra == "api-client"
 Requires-Dist: certifi~=2026.7.22; extra == "api-client"
 Requires-Dist: python3_commons[audit,object-storage]; extra == "api-client"
 Provides-Extra: audit
@@ -26,7 +26,7 @@ Requires-Dist: lxml~=6.1.3; extra == "audit"
 Requires-Dist: zeep[async]~=4.3.3; extra == "audit"
 Requires-Dist: python3_commons[object-storage]; extra == "audit"
 Provides-Extra: authn
-Requires-Dist: aiohttp[speedups]<3.15.0,>=3.14.3; extra == "authn"
+Requires-Dist: aiohttp[speedups]<3.15.0,>=3.14.4; extra == "authn"
 Requires-Dist: certifi~=2026.7.22; extra == "authn"
 Requires-Dist: python3_commons[api-client]; extra == "authn"
 Provides-Extra: authz
@@ -35,12 +35,12 @@ Requires-Dist: python3_commons[api-client]; extra == "authz"
 Provides-Extra: cache
 Requires-Dist: valkey[libvalkey]~=6.1.1; extra == "cache"
 Provides-Extra: database
-Requires-Dist: asyncpg~=0.31.0; extra == "database"
-Requires-Dist: SQLAlchemy[asyncio]~=2.1.1; extra == "database"
+Requires-Dist: asyncpg~=0.32.0; extra == "database"
+Requires-Dist: SQLAlchemy[asyncio]~=2.1.4; extra == "database"
 Provides-Extra: object-storage
 Requires-Dist: object-storage-client~=0.1.12; extra == "object-storage"
 Provides-Extra: soap-client
-Requires-Dist: aiohttp[speedups]<3.15.0,>=3.14.3; extra == "soap-client"
+Requires-Dist: aiohttp[speedups]<3.15.0,>=3.14.4; extra == "soap-client"
 Requires-Dist: certifi~=2026.7.22; extra == "soap-client"
 Requires-Dist: lxml~=6.1.3; extra == "soap-client"
 Requires-Dist: requests~=2.34.2; extra == "soap-client"
@@ -113,7 +113,7 @@ HTTP error status codes and network failures are mapped to built-in Python excep
 Audit logging captures the outgoing request (formatted as an executable `curl` command) and the incoming response body, writing them asynchronously to Object Storage.
 
 The destination path is resolved as follows:
-- If `ApiClientSettings.audit_path` is configured (via `API_CLIENT_AUDIT_PATH`), it is used directly as the audit root for `api_client` (without prepending `"audit/"`):
+- If `ApiClientSettings.audit_path` is configured (via `API_CLIENT_AUDIT_PATH`), it is used directly as the audit root for `api_client` (without prepending `"audit/"`). Note that `API_CLIENT_AUDIT_PATH` is a full object storage path like `s3://bucket/audit`:
   `<audit_path>/<YYYY/MM/DD>/<audit_name>/<uri>/<method>_<timestamp>_<request_id>_{request,response}.txt`
 - Otherwise, if `ObjectStorageSettings.work_path` is configured (via `OBJECT_STORAGE_WORK_PATH`), it is used as the root with `"audit/"` prepended:
   `<work_path>/audit/<YYYY/MM/DD>/<audit_name>/<uri>/<method>_<timestamp>_<request_id>_{request,response}.txt`
@@ -123,8 +123,8 @@ The destination path is resolved as follows:
 To enable audit logging:
 1. **Configure Object Storage & S3 credentials**:
    ```bash
-   # API client audit path or Object storage root
-   export API_CLIENT_AUDIT_PATH="my-audit-bucket/audit-logs"
+   # API client audit path (full object storage path) or Object storage root
+   export API_CLIENT_AUDIT_PATH="s3://bucket/audit"
    # or export OBJECT_STORAGE_WORK_PATH="my-audit-bucket"
 
    # S3 credentials
@@ -141,7 +141,7 @@ To enable audit logging:
    from python3_commons.conf import api_client_settings, object_storage_settings, s3_settings
    from pydantic import SecretStr
 
-   api_client_settings.audit_path = "my-audit-bucket/audit-logs"
+   api_client_settings.audit_path = "s3://bucket/audit"
    # or object_storage_settings.work_path = "my-audit-bucket"
 
    s3_settings.access_key_id = SecretStr("your-access-key-id")
@@ -373,7 +373,7 @@ Typed application settings powered by Pydantic Settings:
 - `DBSettings` / `db_settings`: PostgreSQL database settings (`DB_` environment prefix, with `DB_PASS` alias for password).
 - `S3Settings` / `s3_settings`: S3 credentials and connection configuration (`S3_` environment prefix: `ACCESS_KEY_ID`, `SECRET_ACCESS_KEY`, `REGION`).
 - `ObjectStorageSettings` / `object_storage_settings`: Object storage root configuration (`OBJECT_STORAGE_` environment prefix: `WORK_PATH`).
-- `ApiClientSettings` / `api_client_settings`: API client configuration (`API_CLIENT_` environment prefix: `AUDIT_PATH`).
+- `ApiClientSettings` / `api_client_settings`: API client configuration (`API_CLIENT_` environment prefix: `AUDIT_PATH` full object storage path like `s3://bucket/audit`).
 - `OIDCSettings`: OpenID Connect configuration (`OIDC_` environment prefix).
 - `ValkeySettings` / `valkey_settings`: Valkey/Redis cache configuration (`VALKEY_` environment prefix).
 

{python3_commons-0.25.0 → python3_commons-0.25.2}/README.md

@@ -64,7 +64,7 @@ HTTP error status codes and network failures are mapped to built-in Python excep
 Audit logging captures the outgoing request (formatted as an executable `curl` command) and the incoming response body, writing them asynchronously to Object Storage.
 
 The destination path is resolved as follows:
-- If `ApiClientSettings.audit_path` is configured (via `API_CLIENT_AUDIT_PATH`), it is used directly as the audit root for `api_client` (without prepending `"audit/"`):
+- If `ApiClientSettings.audit_path` is configured (via `API_CLIENT_AUDIT_PATH`), it is used directly as the audit root for `api_client` (without prepending `"audit/"`). Note that `API_CLIENT_AUDIT_PATH` is a full object storage path like `s3://bucket/audit`:
   `<audit_path>/<YYYY/MM/DD>/<audit_name>/<uri>/<method>_<timestamp>_<request_id>_{request,response}.txt`
 - Otherwise, if `ObjectStorageSettings.work_path` is configured (via `OBJECT_STORAGE_WORK_PATH`), it is used as the root with `"audit/"` prepended:
   `<work_path>/audit/<YYYY/MM/DD>/<audit_name>/<uri>/<method>_<timestamp>_<request_id>_{request,response}.txt`
@@ -74,8 +74,8 @@ The destination path is resolved as follows:
 To enable audit logging:
 1. **Configure Object Storage & S3 credentials**:
    ```bash
-   # API client audit path or Object storage root
-   export API_CLIENT_AUDIT_PATH="my-audit-bucket/audit-logs"
+   # API client audit path (full object storage path) or Object storage root
+   export API_CLIENT_AUDIT_PATH="s3://bucket/audit"
    # or export OBJECT_STORAGE_WORK_PATH="my-audit-bucket"
 
    # S3 credentials
@@ -92,7 +92,7 @@ To enable audit logging:
    from python3_commons.conf import api_client_settings, object_storage_settings, s3_settings
    from pydantic import SecretStr
 
-   api_client_settings.audit_path = "my-audit-bucket/audit-logs"
+   api_client_settings.audit_path = "s3://bucket/audit"
    # or object_storage_settings.work_path = "my-audit-bucket"
 
    s3_settings.access_key_id = SecretStr("your-access-key-id")
@@ -324,7 +324,7 @@ Typed application settings powered by Pydantic Settings:
 - `DBSettings` / `db_settings`: PostgreSQL database settings (`DB_` environment prefix, with `DB_PASS` alias for password).
 - `S3Settings` / `s3_settings`: S3 credentials and connection configuration (`S3_` environment prefix: `ACCESS_KEY_ID`, `SECRET_ACCESS_KEY`, `REGION`).
 - `ObjectStorageSettings` / `object_storage_settings`: Object storage root configuration (`OBJECT_STORAGE_` environment prefix: `WORK_PATH`).
-- `ApiClientSettings` / `api_client_settings`: API client configuration (`API_CLIENT_` environment prefix: `AUDIT_PATH`).
+- `ApiClientSettings` / `api_client_settings`: API client configuration (`API_CLIENT_` environment prefix: `AUDIT_PATH` full object storage path like `s3://bucket/audit`).
 - `OIDCSettings`: OpenID Connect configuration (`OIDC_` environment prefix).
 - `ValkeySettings` / `valkey_settings`: Valkey/Redis cache configuration (`VALKEY_` environment prefix).
 

{python3_commons-0.25.0 → python3_commons-0.25.2}/pyproject.toml

@@ -18,8 +18,8 @@ classifiers = [
 keywords = []
 requires-python = ">=3.14.0,<3.15.0"
 dependencies = [
-    "msgpack~=1.2.2",
-    "msgspec~=0.21.1",
+    "msgpack~=1.2.3",
+    "msgspec~=0.22.0",
     "pydantic-settings~=2.15.0"
 ]
 
@@ -28,7 +28,7 @@ all = [
     "python3_commons[api-client,audit,authn,authz,cache,database,object-storage,soap-client]"
 ]
 api-client = [
-    "aiohttp[speedups]>=3.14.3,<3.15.0",
+    "aiohttp[speedups]>=3.14.4,<3.15.0",
     "certifi~=2026.7.22",
     "python3_commons[audit,object-storage]"
 ]
@@ -38,7 +38,7 @@ audit = [
     "python3_commons[object-storage]"
 ]
 authn = [
-    "aiohttp[speedups]>=3.14.3,<3.15.0",
+    "aiohttp[speedups]>=3.14.4,<3.15.0",
     "certifi~=2026.7.22",
     "python3_commons[api-client]"
 ]
@@ -50,14 +50,14 @@ cache = [
     "valkey[libvalkey]~=6.1.1"
 ]
 database = [
-    "asyncpg~=0.31.0",
-    "SQLAlchemy[asyncio]~=2.1.1",
+    "asyncpg~=0.32.0",
+    "SQLAlchemy[asyncio]~=2.1.4",
 ]
 object-storage = [
     "object-storage-client~=0.1.12",
 ]
 soap-client = [
-    "aiohttp[speedups]>=3.14.3,<3.15.0",
+    "aiohttp[speedups]>=3.14.4,<3.15.0",
     "certifi~=2026.7.22",
     "lxml~=6.1.3",
     "requests~=2.34.2",

{python3_commons-0.25.0 → python3_commons-0.25.2}/src/python3_commons/object_storage.py

@@ -78,9 +78,9 @@ def build_url(work_path: str, path: str = '') -> str:
     base_no_scheme = base.split('://', 1)[-1].lstrip('/')
 
     if path.startswith(f'{base_no_scheme}/') or path == base_no_scheme:
-        scheme = base.split('://', 1)[0] + '://' if '://' in base else ''
+        prefix = base[: len(base) - len(base_no_scheme)]
 
-        return f'{scheme}{path}'
+        return f'{prefix}{path}'
 
     parsed = urlparse(base)
     base_path = parsed.path.lstrip('/')

{python3_commons-0.25.0 → python3_commons-0.25.2}/src/python3_commons.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python3-commons
-Version: 0.25.0
+Version: 0.25.2
 Summary: Re-usable Python3 code
 Author-email: Oleg Korsak <kamikaze.is.waiting.you@gmail.com>
 License-Expression: GPL-3.0-only
@@ -12,13 +12,13 @@ Requires-Python: <3.15.0,>=3.14.0
 Description-Content-Type: text/markdown
 License-File: LICENSE
 License-File: AUTHORS.rst
-Requires-Dist: msgpack~=1.2.2
-Requires-Dist: msgspec~=0.21.1
+Requires-Dist: msgpack~=1.2.3
+Requires-Dist: msgspec~=0.22.0
 Requires-Dist: pydantic-settings~=2.15.0
 Provides-Extra: all
 Requires-Dist: python3_commons[api-client,audit,authn,authz,cache,database,object-storage,soap-client]; extra == "all"
 Provides-Extra: api-client
-Requires-Dist: aiohttp[speedups]<3.15.0,>=3.14.3; extra == "api-client"
+Requires-Dist: aiohttp[speedups]<3.15.0,>=3.14.4; extra == "api-client"
 Requires-Dist: certifi~=2026.7.22; extra == "api-client"
 Requires-Dist: python3_commons[audit,object-storage]; extra == "api-client"
 Provides-Extra: audit
@@ -26,7 +26,7 @@ Requires-Dist: lxml~=6.1.3; extra == "audit"
 Requires-Dist: zeep[async]~=4.3.3; extra == "audit"
 Requires-Dist: python3_commons[object-storage]; extra == "audit"
 Provides-Extra: authn
-Requires-Dist: aiohttp[speedups]<3.15.0,>=3.14.3; extra == "authn"
+Requires-Dist: aiohttp[speedups]<3.15.0,>=3.14.4; extra == "authn"
 Requires-Dist: certifi~=2026.7.22; extra == "authn"
 Requires-Dist: python3_commons[api-client]; extra == "authn"
 Provides-Extra: authz
@@ -35,12 +35,12 @@ Requires-Dist: python3_commons[api-client]; extra == "authz"
 Provides-Extra: cache
 Requires-Dist: valkey[libvalkey]~=6.1.1; extra == "cache"
 Provides-Extra: database
-Requires-Dist: asyncpg~=0.31.0; extra == "database"
-Requires-Dist: SQLAlchemy[asyncio]~=2.1.1; extra == "database"
+Requires-Dist: asyncpg~=0.32.0; extra == "database"
+Requires-Dist: SQLAlchemy[asyncio]~=2.1.4; extra == "database"
 Provides-Extra: object-storage
 Requires-Dist: object-storage-client~=0.1.12; extra == "object-storage"
 Provides-Extra: soap-client
-Requires-Dist: aiohttp[speedups]<3.15.0,>=3.14.3; extra == "soap-client"
+Requires-Dist: aiohttp[speedups]<3.15.0,>=3.14.4; extra == "soap-client"
 Requires-Dist: certifi~=2026.7.22; extra == "soap-client"
 Requires-Dist: lxml~=6.1.3; extra == "soap-client"
 Requires-Dist: requests~=2.34.2; extra == "soap-client"
@@ -113,7 +113,7 @@ HTTP error status codes and network failures are mapped to built-in Python excep
 Audit logging captures the outgoing request (formatted as an executable `curl` command) and the incoming response body, writing them asynchronously to Object Storage.
 
 The destination path is resolved as follows:
-- If `ApiClientSettings.audit_path` is configured (via `API_CLIENT_AUDIT_PATH`), it is used directly as the audit root for `api_client` (without prepending `"audit/"`):
+- If `ApiClientSettings.audit_path` is configured (via `API_CLIENT_AUDIT_PATH`), it is used directly as the audit root for `api_client` (without prepending `"audit/"`). Note that `API_CLIENT_AUDIT_PATH` is a full object storage path like `s3://bucket/audit`:
   `<audit_path>/<YYYY/MM/DD>/<audit_name>/<uri>/<method>_<timestamp>_<request_id>_{request,response}.txt`
 - Otherwise, if `ObjectStorageSettings.work_path` is configured (via `OBJECT_STORAGE_WORK_PATH`), it is used as the root with `"audit/"` prepended:
   `<work_path>/audit/<YYYY/MM/DD>/<audit_name>/<uri>/<method>_<timestamp>_<request_id>_{request,response}.txt`
@@ -123,8 +123,8 @@ The destination path is resolved as follows:
 To enable audit logging:
 1. **Configure Object Storage & S3 credentials**:
    ```bash
-   # API client audit path or Object storage root
-   export API_CLIENT_AUDIT_PATH="my-audit-bucket/audit-logs"
+   # API client audit path (full object storage path) or Object storage root
+   export API_CLIENT_AUDIT_PATH="s3://bucket/audit"
    # or export OBJECT_STORAGE_WORK_PATH="my-audit-bucket"
 
    # S3 credentials
@@ -141,7 +141,7 @@ To enable audit logging:
    from python3_commons.conf import api_client_settings, object_storage_settings, s3_settings
    from pydantic import SecretStr
 
-   api_client_settings.audit_path = "my-audit-bucket/audit-logs"
+   api_client_settings.audit_path = "s3://bucket/audit"
    # or object_storage_settings.work_path = "my-audit-bucket"
 
    s3_settings.access_key_id = SecretStr("your-access-key-id")
@@ -373,7 +373,7 @@ Typed application settings powered by Pydantic Settings:
 - `DBSettings` / `db_settings`: PostgreSQL database settings (`DB_` environment prefix, with `DB_PASS` alias for password).
 - `S3Settings` / `s3_settings`: S3 credentials and connection configuration (`S3_` environment prefix: `ACCESS_KEY_ID`, `SECRET_ACCESS_KEY`, `REGION`).
 - `ObjectStorageSettings` / `object_storage_settings`: Object storage root configuration (`OBJECT_STORAGE_` environment prefix: `WORK_PATH`).
-- `ApiClientSettings` / `api_client_settings`: API client configuration (`API_CLIENT_` environment prefix: `AUDIT_PATH`).
+- `ApiClientSettings` / `api_client_settings`: API client configuration (`API_CLIENT_` environment prefix: `AUDIT_PATH` full object storage path like `s3://bucket/audit`).
 - `OIDCSettings`: OpenID Connect configuration (`OIDC_` environment prefix).
 - `ValkeySettings` / `valkey_settings`: Valkey/Redis cache configuration (`VALKEY_` environment prefix).
 

{python3_commons-0.25.0 → python3_commons-0.25.2}/src/python3_commons.egg-info/requires.txt

@@ -1,12 +1,12 @@
-msgpack~=1.2.2
-msgspec~=0.21.1
+msgpack~=1.2.3
+msgspec~=0.22.0
 pydantic-settings~=2.15.0
 
 [all]
 python3_commons[api-client,audit,authn,authz,cache,database,object-storage,soap-client]
 
 [api-client]
-aiohttp[speedups]<3.15.0,>=3.14.3
+aiohttp[speedups]<3.15.0,>=3.14.4
 certifi~=2026.7.22
 python3_commons[audit,object-storage]
 
@@ -16,7 +16,7 @@ zeep[async]~=4.3.3
 python3_commons[object-storage]
 
 [authn]
-aiohttp[speedups]<3.15.0,>=3.14.3
+aiohttp[speedups]<3.15.0,>=3.14.4
 certifi~=2026.7.22
 python3_commons[api-client]
 
@@ -28,14 +28,14 @@ python3_commons[api-client]
 valkey[libvalkey]~=6.1.1
 
 [database]
-asyncpg~=0.31.0
-SQLAlchemy[asyncio]~=2.1.1
+asyncpg~=0.32.0
+SQLAlchemy[asyncio]~=2.1.4
 
 [object-storage]
 object-storage-client~=0.1.12
 
 [soap-client]
-aiohttp[speedups]<3.15.0,>=3.14.3
+aiohttp[speedups]<3.15.0,>=3.14.4
 certifi~=2026.7.22
 lxml~=6.1.3
 requests~=2.34.2

python3_commons-0.25.2/src/python3_commons.egg-info/scm_version.json

@@ -0,0 +1,8 @@
+{
+  "tag": "0.25.2",
+  "distance": 0,
+  "node": "g54b958326632a148dc22f407b5f2bec9db2e1ccb",
+  "dirty": false,
+  "branch": "HEAD",
+  "node_date": "2026-10-07"
+}

{python3_commons-0.25.0 → python3_commons-0.25.2}/tests/unit/test_api_client.py

@@ -11,7 +11,8 @@ from python3_commons.api_client import request
 from python3_commons.conf import ApiClientSettings, api_client_settings
 
 
-def test_api_client_settings():
+def test_api_client_settings(monkeypatch):
+    monkeypatch.delenv('API_CLIENT_AUDIT_PATH', raising=False)
     settings = ApiClientSettings()
     assert settings.audit_path is None
 
@@ -19,6 +20,10 @@ def test_api_client_settings():
     assert custom.audit_path == 'custom-audit-path'
     assert hasattr(api_client_settings, 'audit_path')
 
+    monkeypatch.setenv('API_CLIENT_AUDIT_PATH', 'env-audit-path')
+    env_settings = ApiClientSettings()
+    assert env_settings.audit_path == 'env-audit-path'
+
 
 @pytest.mark.asyncio
 async def test_api_client_request_success(mocker):
@@ -130,6 +135,50 @@ async def test_api_client_audit_logging_with_audit_path(mocker, monkeypatch):
     assert resp_call[0][1].endswith('_response.txt')
 
 
+@pytest.mark.asyncio
+async def test_api_client_audit_logging_with_s3_audit_path(mocker, monkeypatch):
+    monkeypatch.setattr('python3_commons.api_client.api_client_settings.audit_path', 's3://bucket/audit')
+
+    mock_response = mocker.MagicMock(spec=ClientResponse)
+    mock_response.ok = True
+    mock_response.status = HTTPStatus.OK
+    mock_response.text = AsyncMock(return_value='{"result": "success"}')
+
+    mock_ctx = AsyncMock()
+    mock_ctx.__aenter__.return_value = mock_response
+    mock_ctx.__aexit__.return_value = None
+
+    session = mocker.MagicMock(spec=ClientSession)
+    session.get.return_value = mock_ctx
+    session.cookie_jar.filter_cookies.return_value = {}
+
+    mock_write_audit = mocker.patch('python3_commons.api_client.audit.write_audit_data', new_callable=AsyncMock)
+
+    async with request(
+        session,
+        base_url='https://api.example.com',
+        uri='/data',
+        method='get',
+        headers={'Authorization': 'Bearer token'},
+        audit_name='test_service',
+    ) as resp:
+        assert resp == mock_response
+
+    assert mock_write_audit.call_count == 2
+
+    req_call = mock_write_audit.call_args_list[0]
+    assert req_call[0][1].startswith('s3://bucket/audit/')
+    assert not req_call[0][1].startswith('audit/')
+    assert 'test_service/data/get_' in req_call[0][1]
+    assert req_call[0][1].endswith('_request.txt')
+
+    resp_call = mock_write_audit.call_args_list[1]
+    assert resp_call[0][1].startswith('s3://bucket/audit/')
+    assert not resp_call[0][1].startswith('audit/')
+    assert 'test_service/data/get_' in resp_call[0][1]
+    assert resp_call[0][1].endswith('_response.txt')
+
+
 @pytest.mark.asyncio
 async def test_api_client_audit_logging_disabled_by_default(mocker):
     mock_response = mocker.MagicMock(spec=ClientResponse)

{python3_commons-0.25.0 → python3_commons-0.25.2}/tests/unit/test_audit.py

@@ -34,11 +34,15 @@ def test_object_storage_settings_fields():
     assert not hasattr(object_storage_settings, 'audit_path')
 
 
-def test_api_client_settings_fields():
+def test_api_client_settings_fields(monkeypatch):
     settings = ApiClientSettings(audit_path='audit-dir')
     assert settings.audit_path == 'audit-dir'
     assert hasattr(api_client_settings, 'audit_path')
 
+    monkeypatch.setenv('API_CLIENT_AUDIT_PATH', 'env-audit-dir')
+    env_settings = ApiClientSettings()
+    assert env_settings.audit_path == 'env-audit-dir'
+
 
 @pytest.mark.asyncio
 async def test_write_audit_data_with_work_path(mocker, monkeypatch):
@@ -88,6 +92,21 @@ async def test_write_audit_data_key_starts_with_audit_path(mocker, monkeypatch):
     assert call_kwargs['path'] == 'my-audit-root/2026/01/01/request.txt'
 
 
+@pytest.mark.asyncio
+async def test_write_audit_data_with_full_s3_audit_path(mocker, monkeypatch):
+    monkeypatch.setattr(object_storage_settings, 'work_path', None)
+    monkeypatch.setattr(api_client_settings, 'audit_path', 's3://bucket/audit')
+    mock_put = mocker.patch('python3_commons.audit.object_storage.put_object', new_callable=AsyncMock)
+
+    settings = S3Settings(secret_access_key=SecretStr('secret'))
+    await audit.write_audit_data(settings, 's3://bucket/audit/2026/01/01/request.txt', b'test-data')
+
+    assert mock_put.call_count == 1
+    call_kwargs = mock_put.call_args[1]
+    assert call_kwargs['work_path'] == 's3://bucket/audit'
+    assert call_kwargs['path'] == 's3://bucket/audit/2026/01/01/request.txt'
+
+
 @pytest.mark.asyncio
 async def test_write_audit_data_default_work_path_audit(mocker, monkeypatch):
     monkeypatch.setattr(object_storage_settings, 'work_path', None)

{python3_commons-0.25.0 → python3_commons-0.25.2}/tests/unit/test_object_storage.py

@@ -28,6 +28,8 @@ def test_build_url():
     # Local paths
     assert object_storage.build_url('file:///var/data', 'file.txt') == 'file:///var/data/file.txt'
     assert object_storage.build_url('/var/data', 'file.txt') == '/var/data/file.txt'
+    assert object_storage.build_url('/var/data', 'var/data/file.txt') == '/var/data/file.txt'
+    assert object_storage.build_url('file:///var/data', 'var/data/file.txt') == 'file:///var/data/file.txt'
     # Empty path
     assert object_storage.build_url('s3://my-bucket', '') == 's3://my-bucket'