Skip to content

[CRE-6798] add DonName to confidential relay request params - #2476

Draft
bolekk wants to merge 1 commit into
mainfrom
cre-6798-relay-don-name
Draft

bolekk wants to merge 1 commit into
mainfrom
cre-6798-relay-don-name

Conversation

@bolekk

@bolekk bolekk commented Oct 16, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Adds DonName to SecretsRequestParams and CapabilityRequestParams, next to ShardIndex ([CRE-6798] add ShardIndex to confidential relay request params #2470).
  • Names the gateway DON whose shard ShardIndex selects: the node's GatewayConnector DonID minus its _shard-N suffix. May be omitted when the gateway service has a single DON.
  • Routing metadata only: not hash-bound, not checked by Validate.

Deployment Validation

No runtime change until the gateway reads the field.

🤖 Generated with Claude Code

@github-actions

github-actions Bot commented Oct 16, 2026 •

Copy link
Copy Markdown
Contributor

⚠️ API Diff Results - github.com/smartcontractkit/chainlink-common

⚠️ Breaking Changes (2)

pkg/capabilities/v2/actions/confidentialrelay.CapabilityRequestParams (1)
  • ShardIndex — 🗑️ Removed
pkg/capabilities/v2/actions/confidentialrelay.SecretsRequestParams (1)
  • ShardIndex — 🗑️ Removed

✅ Compatible Changes (4)

pkg/capabilities/v2/actions/confidentialrelay.CapabilityRequestParams (2)
  • WorkflowDonName — ➕ Added

  • WorkflowDONShardIndex — ➕ Added

pkg/capabilities/v2/actions/confidentialrelay.SecretsRequestParams (2)
  • WorkflowDonName — ➕ Added

  • WorkflowDONShardIndex — ➕ Added


📄 View full apidiff report

With ShardIndex alone the gateway cannot tell which DON's shard is meant
when a service has several DONs. DonName is the node's GatewayConnector
DonID minus the _shard-N suffix.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@bolekk
bolekk force-pushed the cre-6798-relay-don-name branch from 129ade4 to 0f929eb Compare October 16, 2026 03:16

This branch has not been deployed

No deployments
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