feat(deno): add Deno.serve integration#12460
feat(deno): add Deno.serve integration#12460brc-dd wants to merge 6 commits intogetsentry:developfrom
Conversation
| uses: denoland/setup-deno@v1.1.4 | ||
| with: | ||
| deno-version: v1.38.5 | ||
| deno-version: v1.x |
There was a problem hiding this comment.
jsr wasn't supported on deno 1.38. deno.land/std/... won't receive non-critical updates and jsr is the recommended way
if needed we can lock it to some minor
re-running CI should pass I guess 👀
| return new Proxy(handler, { | ||
| apply(handlerTarget, handlerThisArg, handlerArgs: Parameters<RawHandler>) { | ||
| return withIsolationScope(isolationScope => { | ||
| isolationScope.clear(); |
There was a problem hiding this comment.
also this line should be removed when #12450 (comment) is resolved
|
sorry, this has been open for a very long time - is this still relevant to the state of deno today (where we are basically node-compatible)? This has run quite out of sync and would need rebasing on develop. |
Yes.
Node.js compatibility here means you will be able to use Sentry's Node.js Also, even then the Node.js compatibility mode and Deno globals are different (see #12400 and denoland/deno#20826) and since Sentry v9 is no longer published to deno.land and the JSR version is pretty much unusable, there is no official way for stuff like console integration to work with Deno.
Yeah, I can rebase it, but I don't think it's gonna be unblocked anytime soon, so it will get stale again 🤷♂️ However, if the OLTP endpoint lands, then it will fix both these issues. See #85902 and denoland/deno#28330 (comment) On a side note, we have been using a patched version in prod for almost an year now - https://github.com/globalbrain/hado/blob/main/src/sentry.ts - so if anyone using Deno wants to use Sentry with fixes for console and fetch capturing and Deno.serve integration, they can use it. |
|
closing in favor of #19230 |
closes #12450
the code is mostly taken from the bun sdk
Before submitting a pull request, please take a look at our
Contributing guidelines and verify:
yarn lint) & (yarn test).