-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
TypeError: Invalid value used as weak map key during destroy() #14512
Comments
Is this with an empty app? |
Yes,
Let me know if I can provide debugging help somehow. I’m not sure where to look since I don’t see any application code in the stacktrace. |
@Serabe nope, this was from one of my production apps |
Can any of you try to provide a reproduction? I'll try to take a look at travis-web later this week otherwise |
unfortunately I haven't been able to reproduce this yet |
While I get the error with every load on a local |
Seems related to #14499, but I'd need to dig in a bit more. |
The common error message here does not make these all the same issue. We need stack traces in each case like @Turbo87 and @Serabe added, those are separate issues. @rwjblue #14499 is about node 0.12 in which get() with a non object key fails. This does seem to match the stack that @Turbo87 gave and is fixed in #14499 but is not the same issue as @Serabe @Turbo87 can you confirm if your error is when running in an old v8? |
@krisselden I haven't been able to trigger it with the current Chrome versions that are installed on my MacBook and mobile devices. The only user that was able to trigger it was reportedly using Chrome 38 as stated above. |
If I understand you correctly, @krisselden, are you suggesting that I open another issue? Is there more detail I can give apart from pasting the stacktraces and including the browser versions? |
@backspace yes, the issue that @Turbo87 reported is fixed with #14499 please don't open issues with just a TypeError. It should say TypeError during some operation, the error with old v8 and destroy() has been fixed. The error with ref get() bailing out is a different. |
I’ll open a separate issue, sorry for my lack of understand. Can you see that it’s difficult for someone not familiar with the internals to know what portion of the stacktrace is important and should be extracted? You’re saying the |
@backspace I'm sorry. |
I've just been reported the following error running v2.10.0-beta.1:
Stack trace without source maps
This was reported from a production build with source maps, so don't trust the stack trace too much... according to sentry.io the browser that hit this was Chrome Mobile 38 on Android 6.0.
@backspace reported a similar issue in #dev-ember (feel free to paste your call stack here as well)
The text was updated successfully, but these errors were encountered: