Skip to content

refactor(external-call): redesign extension service branch - #14

Draft
angelol wants to merge 13 commits into
external-call/05-extension-servicefrom
angelol/external-call-05-extension-service-redesign
Draft

refactor(external-call): redesign extension service branch#14
angelol wants to merge 13 commits into
external-call/05-extension-servicefrom
angelol/external-call-05-extension-service-redesign

Conversation

@angelol

@angelol angelol commented May 22, 2026

Copy link
Copy Markdown

Summary

Restacks the extension-service branch on top of the redesigned 04 protocol carrier and cleans up the external-call HTTP client surface.

Key branch-05 changes:

  • keeps public extension config wiring deferred to branch 06
  • adds/keeps the extension service config and HTTP client implementation
  • uses the fixed X-Request-Id request header
  • sends a stable Idempotency-Key across retries
  • handles Retry-After as a minimum delay
  • bounds HTTP response body buffering
  • sanitizes client error messages and token-file errors
  • reports retry exhaustion accurately
  • validates service port/API version config without over-engineering it
  • removes implementation-detail test noise

This branch is stacked on the 04 redesign, so the diff against canonical external-call/05-extension-service also includes the 04 protocol-carrier redesign that is being reviewed separately in PR #13.

Validation

  • Ran git diff --check external-call/05-extension-service...HEAD.
  • Earlier focused validation for the branch-05 fixes included HttpExtensionServiceClientTest and participant scalafmt checks during implementation.

Full sbt/protobuf validation was not rerun in this PR-opening step.

@angelol
angelol force-pushed the angelol/external-call-05-extension-service-redesign branch from c89a458 to bbf3171 Compare May 28, 2026 09:16
canton-machine and others added 2 commits May 29, 2026 09:45
* [main] Update 2026-05-28.15

Reference commit: 18597a3

* Updated docker image for Canton builds
…sults (digital-asset#526)

* feat(external-call): Add Canton protocol support for external calls

- Extend ActionDescription with external call results
- Update participant_transaction.proto
- Update ViewParticipantData
- Update NodeHashBuilder for external call hashing

* refactor(external-call): redesign protocol carrier after review

---------

Co-authored-by: Tino Rusch <tino.rusch@gmail.com>
@angelol
angelol force-pushed the angelol/external-call-05-extension-service-redesign branch 2 times, most recently from 2a0fd26 to f8176d8 Compare June 2, 2026 22:36
* [main] Update 2026-06-16.22

Reference commit: a3def04

* Updated docker image for Canton builds
@angelol
angelol force-pushed the angelol/external-call-05-extension-service-redesign branch from 7aee556 to 8595212 Compare June 22, 2026 11:12
@angelol
angelol force-pushed the angelol/external-call-05-extension-service-redesign branch from becd47b to 140a808 Compare June 28, 2026 17:56
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