inngest-js icon indicating copy to clipboard operation
inngest-js copied to clipboard

INN-2882 Add selective header forwarding when sending events

Open jpwilliams opened this issue 1 year ago • 4 comments

Summary

Read the incoming headers tracestate and traceparent. If it is set, copy the header to any outgoing event-sending requests.

This only covers step.sendEvent(); we'll add later support for client.send() as we need async hooks for this which has scattered runtime support.

Checklist

  • [ ] ~Added a docs PR that references this PR~ N/A
  • [ ] Added unit/integration tests
  • [ ] Added changesets if applicable

Related

  • INN-2882

jpwilliams avatar Mar 18 '24 14:03 jpwilliams

🦋 Changeset detected

Latest commit: 7ed02b4658c007655d9a552da876807c6d1f0c33

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
inngest Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

changeset-bot[bot] avatar Mar 18 '24 14:03 changeset-bot[bot]

A user has added the prerelease/inngest label, so this PR will be published to npm with the tag pr-520. It will be updated with the latest changes as you push commits to this PR.

You can install this prerelease version with:

npm install inngest@pr-520

The last release was built and published from 7ed02b4658c007655d9a552da876807c6d1f0c33.

inngest-release-bot avatar Mar 18 '24 14:03 inngest-release-bot

@jpwilliams anything still waiting on? I think this PR can be merged as is.

darwin67 avatar Apr 20 '24 01:04 darwin67