Skip to content

Comments

Add headers to cancel nexus task#2798

Open
Quinn-With-Two-Ns wants to merge 1 commit intotemporalio:masterfrom
Quinn-With-Two-Ns:issue-2797
Open

Add headers to cancel nexus task#2798
Quinn-With-Two-Ns wants to merge 1 commit intotemporalio:masterfrom
Quinn-With-Two-Ns:issue-2797

Conversation

@Quinn-With-Two-Ns
Copy link
Contributor

@Quinn-With-Two-Ns Quinn-With-Two-Ns commented Feb 24, 2026

Add headers to cancel nexus task. I do have a test we can add along side this, it is decently long for how simple this change is so I left it off for now, but if we want I can add it.

closes #2797


Note

Low Risk
Single-field addition to the cancel Nexus task request; low chance of regressions beyond minor behavior changes in tests that assert exact request contents.

Overview
Propagates the original Nexus headers when the test server enqueues a CancelOperationRequest for an in-flight Nexus operation, by copying NexusOperationScheduledEventAttributes.nexus_header into the cancel Request.

This ensures cancel tasks carry the same header context as start tasks (e.g., for auth/metadata parity) rather than sending an empty header map.

Written by Cursor Bugbot for commit 3d98352. This will update automatically on new commits. Configure here.

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.

Nexus operation cancel task does not include header

1 participant