Add Sentry integration for Hono

[FaureAlexis]

Problem Statement

Add tracing integration for Hono (https://hono.dev/) to @sentry/node like other frameworks like Express, fastify, nest etc.

Solution Brainstorm

If it's okay for you, I can create a PR with the integration within a week

[s1gr1d]

Thanks for suggesting this! We were already talking about a hono integration. Hono already provides their own Sentry middleware. @yusukebe, you are the core maintainer of the middleware. What do you think of adding a hono integration to our Sentry repository? We could also mention the hono middleware in our docs.

[yusukebe]

Hi.

@s1gr1d

What do you think of adding a hono integration to our Sentry repository? We could also mention the hono middleware in our docs.

As you said, we have the middleware @hono/sentry, so creating a new one is unnecessary. I think the best approach is adding a description for the middleware in the sentry docs. Thanks!

[timfish]

@hono/sentry currently uses toucan-js so that it can fully support all platforms including Cloudflare Workers.

toucan-js is awaiting an update to the v8 SDK robertcepa/toucan-js#234

[s1gr1d]

cc @MathurAditya724

[vKongv]

does @hono/sentry support performance monitoring?

[wesleyakio]

@s1gr1d @yusukebe I believe the initial ticket was about supporting tracing, but as far as I could understand @hono/sentry only works for exceptions.

Are there plans to support tracing on @hono/sentry, adding hono to @sentry/node or neither?

[yusukebe]

Hi, @sam-lippert. What do you think of the above things?

[sam-lippert]

@wesleyakio @yusukebe The Sentry middleware for Hono is designed to be as lightweight as possible of a wrapper for toucan-js, so if a feature is supported by Toucan, it should be supported by @hono/sentry. Regarding performance monitoring specifically, I am unsure if Toucan supports it, as there has been an open issue on it since 2021. It is a feature that people have been asking about, and so I've been periodically trying to figure that out through the documentation over the past few months, and if it is supported by Toucan, I suspect that monitoring by default could be supported by adding performance spans to the middleware.