Skip to content

feat: add cluster execution owner contract#4

Closed
acedatacloud-dev wants to merge 1 commit into
mainfrom
feat/x402-execution-owner
Closed

feat: add cluster execution owner contract#4
acedatacloud-dev wants to merge 1 commit into
mainfrom
feat/x402-execution-owner

Conversation

@acedatacloud-dev

Copy link
Copy Markdown
Member

Summary

  • add the generic X-Ace-Execution-Owner: x402 execute-only contract
  • skip worker /record only for the recognized owner; unknown/missing owners keep normal recording
  • add case-insensitive dict and real Tornado header tests plus PR CI

Hard dependency

Do not merge or publish this package until https://github.com/AceDataCloud/PlatformGateway/pull/214 is merged and deployed. The public Kong boundary must strip client-supplied owner headers first.

Validation

  • 5 unit tests passed on Python 3.11
  • sdist and wheel built successfully
  • wheel contains acedatacloud_scaffold/execution.py
  • public imports and compileall passed

🤖 对抗评审 (reviewer: Explore, 2 rounds)

  • RISK: case-sensitive mapping lookup → fixed with case-insensitive header iteration and real Tornado coverage
  • RISK: public clients could spoof execution ownership → fixed by hard dependency on Gateway PR #214; this PR remains Draft until that boundary is deployed
  • RISK: unknown owners silently bypass policy → rebutted: unknown owners retain normal billing; added warning for observability
  • RISK: pre-existing logger.catch and duplicate initializer → out of scope; neither is introduced nor exercised by the skip path

@Germey Germey closed this Jul 24, 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.

2 participants