Skip to content

pkg/capabilities/v2/actions/confidentialrelay/types.go: relax validateExecutionID check#2194

Open
cfal wants to merge 2 commits into
mainfrom
relax-exec-id-check
Open

pkg/capabilities/v2/actions/confidentialrelay/types.go: relax validateExecutionID check#2194
cfal wants to merge 2 commits into
mainfrom
relax-exec-id-check

Conversation

@cfal

@cfal cfal commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@cfal cfal requested a review from a team as a code owner June 25, 2026 10:02
@github-actions

Copy link
Copy Markdown

👋 cfal, thanks for creating this pull request!

To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team.

Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks!

@github-actions

github-actions Bot commented Jun 25, 2026

Copy link
Copy Markdown

📊 API Diff Results

No changes detected for module github.com/smartcontractkit/chainlink-common

View full report

nadahalli
nadahalli previously approved these changes Jun 25, 2026
@cfal cfal dismissed stale reviews from prashantkumar1982 and nadahalli via 1b32001 June 26, 2026 10:27
@cfal cfal force-pushed the relax-exec-id-check branch from 0ae4650 to 1b32001 Compare June 26, 2026 10:27
cfal added 2 commits June 26, 2026 17:36
…x and arbitrary length execution IDs

Relaxes the validation constraints for the execution_id field on
SecretsRequestParams and CapabilityRequestParams. The field is
now only validated to be non-empty, and no longer restricted to
a 32-byte hex-encoded string.
@cfal cfal force-pushed the relax-exec-id-check branch from 1b32001 to 14dc1dc Compare June 26, 2026 10:37
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.

3 participants