Skip to content

Commit

Permalink
fix: toucan-js in access-api does not use RewriteFrame integration (#729
Browse files Browse the repository at this point in the history
)

Motivation:
*
#623 (comment)
* this was suggested by toucan-js author as no longer being needed in
3.0.0.
  • Loading branch information
gobengo authored Apr 7, 2023
1 parent 48d9c89 commit eeb90e6
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 44 deletions.
1 change: 0 additions & 1 deletion packages/access-api/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,6 @@
"license": "(Apache-2.0 OR MIT)",
"dependencies": {
"@ipld/dag-ucan": "^3.2.0",
"@sentry/integrations": "^7.28.0",
"@ucanto/core": "^5.2.0",
"@ucanto/interface": "^6.2.0",
"@ucanto/principal": "^5.1.0",
Expand Down
2 changes: 0 additions & 2 deletions packages/access-api/src/utils/context.js
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@ import {
} from '../models/delegations.js'
import { createD1Database } from './d1.js'
import { DbProvisions } from '../models/provisions.js'
import { RewriteFrames } from '@sentry/integrations'

/**
* Obtains a route context object.
Expand Down Expand Up @@ -48,7 +47,6 @@ export function getContext(request, env, ctx) {
const sentry = new Toucan({
context: ctx,
integrations: [
new RewriteFrames({ root: '/' }),
new RequestData({
allowedHeaders: ['user-agent', 'x-client'],
allowedSearchParams: /(.*)/,
Expand Down
41 changes: 0 additions & 41 deletions pnpm-lock.yaml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit eeb90e6

Please sign in to comment.