Skip to content

Hide Deeploy mandatory secrets in dAuth - #478

Closed
aledefra wants to merge 3 commits into
developfrom
codex/dauth-secret-redaction-storage
Closed

Hide Deeploy mandatory secrets in dAuth#478
aledefra wants to merge 3 commits into
developfrom
codex/dauth-secret-redaction-storage

Conversation

@aledefra

@aledefra aledefra commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • extract approved mandatory Deeploy secret paths from materialized deploy plans
  • replace those values with the fixed dAuth placeholder before dispatch and persistence
  • write the job-scoped secret bundle to DAUTH_JOB_SECRETS before create dispatch and before update delete/redeploy
  • sanitize return_request responses for the same mandatory paths

Replaces accidentally closed #468 after its stacked base, #458, was squash-merged and deleted. The branch now contains only this PR's two commits on top of develop.

Notion: https://app.notion.com/p/38bd59ab6657807cb291cafcd16b1954

Test plan

  • python3 -m py_compile extensions/business/deeploy/deeploy_mixin.py extensions/business/deeploy/deeploy_manager_api.py extensions/business/deeploy/tests/test_create_requests.py extensions/business/deeploy/tests/test_process_request.py extensions/business/deeploy/tests/test_update_requests.py
  • git diff --check

Blocked locally:

  • PYTHONPATH=/Users/alessandro/programmazione/ratio1/edge_node:/Users/alessandro/programmazione/ratio1/naeural_core:/Users/alessandro/programmazione/ratio1/ratio1_sdk python3 -m unittest extensions.business.deeploy.tests.test_create_requests extensions.business.deeploy.tests.test_process_request
  • blocked because system Python misses SDK EVM dependency eth_account

@aledefra

aledefra commented Aug 4, 2026

Copy link
Copy Markdown
Contributor Author

Superseded by #471, which now targets develop and includes these commits as part of the end-to-end dAuth secret persistence and worker-resolution flow.

@aledefra aledefra closed this Aug 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant