Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Release v8.5.0 #948

Merged
merged 3 commits into from
Oct 12, 2021
Merged

Release v8.5.0 #948

merged 3 commits into from
Oct 12, 2021

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Oct 12, 2021

Proposed Release Notes

  • Added full support for Fastify v2 and v3. Fastify instrumentation is now GA.

    • Removed fastify feature flag.
    • Instrumented Fastify routes by wrapping addHook.
    • Added middleware mounting for fastify v3.
    • Fixed capturing of mount point for middleware naming.
    • Fixed the WebFramework spec definitions for Fastify middleware and route handlers to properly retrieve the IncomingMessage from a request object.
    • Added proper definition to middleware handlers so that the relationship to consecutive middleware and route handler are siblings and not direct children.
  • Added experimental instrumentation for the undici http client behind a feature flag.

    To enable undici support, add the following into your config: { feature_flag: { undici_instrumentation: true } }. The support for undici client is Node.js 16.x as it takes advantage of the diagnostics_channel. Lastly, you must be using v4.7.0+ of the undici client for any of the instrumentation to work.

    Note: There are currently some state issues if requests to an app are made with keep alive and you have multiple undici requests being made in parallel. In this case, set feature_flag: { undici_async_tracking: false } which avoids these state issues at the cost of some broken segment nesting.

Links

Details

@michaelgoin michaelgoin added semver: minor New features added. risk: low This change does not impact critical areas of the code and is expected to have minimal risk. labels Oct 12, 2021
@michaelgoin michaelgoin marked this pull request as ready for review October 12, 2021 20:00
Copy link
Member

@bizob2828 bizob2828 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fastify GA 🚀

Undici feature flag 🙌🏻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
risk: low This change does not impact critical areas of the code and is expected to have minimal risk. semver: minor New features added.
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants