Skip to content

Mask credentials in expression preview - #299

Merged
mbakgun merged 1 commit into
mainfrom
codex/fix-credential-masking-preview
Jul 8, 2026
Merged

Mask credentials in expression preview#299
mbakgun merged 1 commit into
mainfrom
codex/fix-credential-masking-preview

Conversation

@mbakgun

@mbakgun mbakgun commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Implemented credential masking for expression preview. Added mask_credentials_context in backend/app/services/workflow_executor.py, wired backend/app/api/expressions.py to pass masked credential values into ExpressionEvaluatorService, and added backend tests for the helper and preview API behavior.

Implemented credential masking for expression preview. Added `mask_credentials_context` in `backend/app/services/workflow_executor.py`, wired `backend/app/api/expressions.py` to pass masked credential values into `ExpressionEvaluatorService`, and added backend tests for the helper and preview API behavior.

Validation:
Passed: `uv run ruff check app/api/expressions.py app/services/workflow_executor.py tests/test_expression_evaluator_api.py tests/test_sse_streaming.py`. Passed: `SECRET_KEY=... ENCRYPTION_KEY=... uv run pytest tests/test_expression_evaluator_api.py tests/test_sse_streaming.py` with 38 tests.
@mbakgun
mbakgun enabled auto-merge July 8, 2026 16:14
@mbakgun
mbakgun merged commit 393ea94 into main Jul 8, 2026
2 checks passed
@mbakgun
mbakgun deleted the codex/fix-credential-masking-preview branch July 8, 2026 16:15
@mbakgun

mbakgun commented Jul 8, 2026

Copy link
Copy Markdown
Contributor Author
30922318

@heym-codex 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants