Redact SQLAlchemy bind params, attach SQL context to errors, add RFC8523 JTI replay and tests

[cobycloud]

Motivation

• Prevent sensitive bind values from leaking in REST/JSON-RPC payloads while preserving useful, machine-readable diagnostics when SQLAlchemy errors occur.
• Validate redaction end-to-end via ASGI (uvicorn) integration tests and ensure unit-level coverage for SQLAlchemy error metadata.
• Add simple JTI replay detection to exercise RFC8523 replay-protection behavior and fix test DB setup so schema-qualified tables (e.g. authn.tenants) exist during tests.

Description

• Added _format_sqlalchemy_error_data, _safe_params_metadata, and _looks_like_validation_error helpers to tigrbl/runtime/errors/utils.py and exported them for use by the error conversion layer.
• Extended tigrbl/runtime/errors/converters.py to recognize StatementError and to include SQL context or safe redaction metadata for StatementError, OperationalError, and DBAPIError when appropriate.
• Implemented an in-memory _JTI_CACHE with locking and expiry logic in tigrbl_auth/rfc/rfc8523.py and invoked replay detection inside validate_enhanced_jwt_bearer.
• Added integration and unit tests covering SQLAlchemy redaction (pkgs/standards/tigrbl_tests/tests/i9n/test_sqlalchemy_error_redaction_uvicorn.py and pkgs/standards/tigrbl_tests/tests/unit/runtime/test_error_sqlalchemy_context.py) and extended RFC8523 tests to assert JTI replay behavior (pkgs/standards/tigrbl_auth/tests/unit/test_rfc8523_jwt_client_auth.py).
• Fixed tigrbl_auth test setup by attaching a temporary SQLite file as the authn schema in pkgs/standards/tigrbl_auth/tests/conftest.py and cleaning it up after the engine is disposed so schema-qualified inserts succeed.

Testing

• Ran code formatting with uv run --directory pkgs/standards/tigrbl_auth --package tigrbl-auth ruff format . which completed successfully.
• Ran linter/fix with uv run --directory pkgs/standards/tigrbl_auth --package tigrbl-auth ruff check . --fix which reported All checks passed!.
• Executed the tigrbl_auth test suite with uv run --package tigrbl-auth --directory standards/tigrbl_auth pytest and observed 388 passed, 5 skipped, 83 deselected (all tests in that run passed).
• New tigrbl unit and i9n tests for error redaction were added to pkgs/standards/tigrbl_tests; those tests were added but not executed as part of the tigrbl_auth pytest run above.

---

Codex Task

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: f3e8f8553a

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

[chatgpt-codex-connector]

Keep JTI cache alive through skew window

Replay detection uses _JTI_CACHE[jti] = iat and evicts entries based on current_time - seen_at > max_age_seconds. Because validate_enhanced_jwt_bearer allows tokens up to max_age_seconds + clock_skew_seconds, a token issued near the end of the max-age window can be replayed after max_age_seconds elapses but before max_age_seconds + clock_skew_seconds (the JTI entry gets purged, so the replay passes). Consider storing current_time (first-seen) instead of iat, or evict based on max_age_seconds + clock_skew_seconds so replays remain blocked for the entire validity window.

Useful? React with 👍 / 👎.