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

Uncaught TypeError: Cannot create proxy with a non-object as target or handler #14077

Closed
richhapal opened this issue Sep 17, 2024 · 23 comments
Closed

Comments

@richhapal
Copy link

richhapal commented Sep 17, 2024

Environment

SaaS (https://sentry.io/)

What are you trying to accomplish?

I'm try to init sentry. Initially it was working fine but now getting error

How are you getting stuck?

I'm try to init sentry through firebase key but getting error now. Initially it was working fine and we caught many errors.

  1. if i comment Sentry.init then i'm not getting any error but comment out these code and throw these error

Image

Getting error in xhr.ts file of sentry pkg:
Image

Where in the product are you?

Issues

Link

No response

DSN

https://63ed691b2737c6fd97c017c3c79b4dc7@o1243174.ingest.us.sentry.io/4506735425290240

@getsantry
Copy link

getsantry bot commented Sep 17, 2024

Auto-routing to @getsentry/product-owners-issues for triage ⏲️

@snigdhas
Copy link
Member

Please file a support ticket and someone from the support team can help debug this futher.

@ReneGreen27
Copy link
Member

User has written in via internal ticket 132675, closing issue

@Roman-Simik
Copy link

Roman-Simik commented Sep 27, 2024

  • Hi, guys can you please share progress or resolution?
  • We encountered same problem after we upgraded @sentry/browser from 7.108.0 lib to 8.31.0 (on the previous version, everything worked correctly)
    - EDIT: - Last working version is 8.17.0 after this, Sentry starts showing this error
  • we encountered it only in native iOS app (running angular app wrapped in Capacitor)

Image
Image

@mdsaifurrahmann
Copy link

  • Hi, guys can you please share progress or resolution?
  • We encountered same problem after we upgraded @sentry/browser from 7.108.0 lib to 8.31.0 (on the previous version, everything worked correctly)
    - EDIT: - Last working version is 8.17.0 after this, Sentry starts showing this error
  • we encountered it only in native iOS app (running angular app wrapped in Capacitor)

Image Image

After downgrading to 8.17, the error is gone. thanks pal.

@kai-softr
Copy link

We've encountered the same issue after upgrading to 8.34.0. I rolled back to our old version 8.26.0 and it worked again.

@mdsaifurrahmann
Copy link

I'm currently using the same version, 8.26.0 and this version is moderately vulnerable, like 8.17
But I didn't find any option, rather using this version.

@richhapal
Copy link
Author

We've encountered the same issue after upgrading to 8.34.0. I rolled back to our old version 8.26.0 and it worked again.

exactly @kai-softr after downgrade , it's working fine

@JimJafar
Copy link

I have raised a new issue asking Sentry to update this thread with instructions on how to resolve this.
getsentry/sentry#79355

@Beauchaj
Copy link

Beauchaj commented Oct 22, 2024

Saw the issue on 8.33.0, however have got it working on 8.27.0

@lobsterkatie
Copy link
Member

According to the internal support ticket, the problem had to do with Firebase, specifically this: firebase/firebase-tools#1661.

I'm going to reopen this, but I'm also going to move it to the SDK repo, since that's where the incompatibility lies.

@lobsterkatie lobsterkatie reopened this Oct 24, 2024
@getsantry getsantry bot moved this to Waiting for: Product Owner in GitHub Issues with 👀 3 Oct 24, 2024
@lobsterkatie lobsterkatie transferred this issue from getsentry/sentry Oct 24, 2024
@chargome
Copy link
Member

@JimJafar @Beauchaj Could someone provide a minimal reproduction for this?

@Beauchaj
Copy link

@JimJafar @Beauchaj Could someone provide a minimal reproduction for this?

I don't believe its limited to firebase. I came across the problem on an Ionic react project, using sentry/capacitor.

Capacitor and Ionic versions

    "@capacitor/core": "^6.1.2",
    "@capacitor/ios": "^6.1.2",
    "@ionic/react": "^8.1.0",

This (excerpt from package.json) succeeds

"dependencies": {
    "@sentry/capacitor": "1.0.0",
    "@sentry/cli": "^2.38.0",
    "@sentry/react": "8.27.0",

While this fails with the error in the topic.

"dependencies": {
    "@sentry/capacitor": "1.0.1",
    "@sentry/cli": "^2.38.0",
    "@sentry/react": "8.33.0",

@getsantry getsantry bot moved this from Waiting for: Community to Waiting for: Product Owner in GitHub Issues with 👀 3 Oct 29, 2024
@chargome
Copy link
Member

@Beauchaj thanks for pointing that out, but we'll still need to wait for a reproduction to debug this issue.

@thomanil
Copy link

thomanil commented Nov 2, 2024

@chargome I am seeing the same thing inside a Capacitor app, when using the Capacitor http plugin - https://capacitorjs.com/docs/apis/http It works fine in the same capacitor project when I do not use that http plugin (capacitor/http patches or modifies xhr, and is very commonly in use for projects that use cookie-based auth/credentials.)

I can try to create a minimal reproduction skeleton project tomorrow, if that helps.

@getsantry getsantry bot moved this to Waiting for: Product Owner in GitHub Issues with 👀 3 Nov 2, 2024
@andreiborza
Copy link
Member

@thomanil thanks for chiming in. Yea a reproduction would help for sure, thanks!

@thomanil
Copy link

thomanil commented Nov 4, 2024

Did not get around to it yesterday but will be able to one of next few days :)

@thomanil
Copy link

thomanil commented Nov 5, 2024

@andreiborza @chargome @muuvmuuv Reproduction here: https://github.com/thomanil/sentry-on-capacitor-patched-xhr-crash-reproduce/tree/main

(Have tried to make it as easy as possible to set up, but let me know if you have any trouble!)

@getsantry getsantry bot moved this from Waiting for: Community to Waiting for: Product Owner in GitHub Issues with 👀 3 Nov 5, 2024
@andreiborza
Copy link
Member

@thomanil thanks for that. Taking a look.

@andreiborza
Copy link
Member

@thomanil this has been fixed with #14140. Please upgrade the sdk to the latest. I tested it out with your reproduction and got the successful init message with the latest browser sdk.

@thomanil
Copy link

thomanil commented Nov 6, 2024

So it does not just fail silently, error reports do make it out in latest release?

@andreiborza
Copy link
Member

@thomanil yep

Image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

No branches or pull requests