Closed
Description
Is there an existing issue for this?
- I have checked for existing issues https://github.com/getsentry/sentry-javascript/issues
- I have reviewed the documentation https://docs.sentry.io/
- I am using the latest SDK release https://github.com/getsentry/sentry-javascript/releases
How do you use Sentry?
Self-hosted/on-premise
Which package are you using?
@sentry/vue
SDK Version
7.12.1
Framework Version
Vue 3.2.23
Link to Sentry event
No response
Steps to Reproduce
- add a route to a new page
- the new page's template add one undeclared var
- visit the new page. and the warning message from instruments.js will go on and on until the page dead.
The reason I found is that, when rendering gets a warning, will call console.warn
to console the error stack. But the stack includes a Proxy variable. When sentry do safeJoin
with String(value)
, will visit the proxy variable, and visit the get
handler, got undefined, so vue will call console.warn
again. So the warning loop happends.
I tried to fix but I can not deal with how to test a var is Proxied. The way I test will trigger the get
handler, so I just disable the console option of Breadcrumbs to solve the problem temporary.
Even though I should clean my code without undeclared var, I think it is still a bug.
Expected Result
do not loop warning
Actual Result
do loop waning.