inngest / inngest-js

The developer platform for easily building reliable workflows with zero infrastructure for TypeScript & JavaScript
https://www.inngest.com/
GNU General Public License v3.0
414 stars 41 forks source link

INN-2882 Add selective header forwarding when sending events #520

Closed jpwilliams closed 5 months ago

jpwilliams commented 6 months ago

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

Related

linear[bot] commented 6 months ago

INN-2882 Header forwarding for TS SDK

changeset-bot[bot] commented 6 months ago

🦋 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

inngest-release-bot commented 6 months ago

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.

darwin67 commented 5 months ago

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