-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
"Non-Error promise rejection captured with value:" #3440
Comments
Unfortunately, there's nothing we can extract from promise rejections that were thrown with an empty string. Even trying to "fake" an error doesn't provide a valid stack trace, as global error handlers are not keeping track of the callers. This event is produced in one of three ways: async function foo() {
throw '';
}
// or
new Promise((resolve, reject) => {
throw '';
});
// or
new Promise((resolve, reject) => {
reject('');
}); In order for us to extract the stack trace, rejections/throws inside async functions have to be done through an |
This started happening to us on April 16th, on a day where we didn't change any code at all. Since them we've had 98k of these errors. We've added some code to filter this out in our js as we can't figure out a way to silence these at all. I'm guessing it's related to some password manager like lastpass as it only happens on our login page, with chrome, on windows 10. We've added code like:
If there's a better way to deal with this, I'd love to know about it. On one hand, I can see how you don't want to editorialize here and prevent these errors, but when there's no info at all there's nothing actionable and we can't do anything about it other than be annoyed. |
Thanks for the insight here! Really helpful. We are having the same affected browser and device, but really can't tell where this is coming from. @kamilogorek said it great where it is just returning an empty error object so not much Sentry can do here. Would be cool to know the origin of the error maybe? Not sure how this works with extensions. Either way appreciate the insight! |
Adding an update for anyone else, this problem has evolved to be a "Non-Error promise rejection captured with value: Object Not Found Matching Id:5" error, so we had to update our hack to deal with that. We decided to just put the hack on the login page, as it appears this is from some password manager browser extension. |
We are also getting this error, both empty and now with the "Object Not Found Matching Id:" message. I want to point out that while we do not have a login page we do have forms all over the site and there's two characteristics that stick out:
It may be a completely different issue from yours @wynnw but who knows. |
This is most likely happening due to the safe links being scanned by Outlook If you have an Office 365 subscription and your emails contain links to your site and these links use We were able to reproduce this and get the error in Sentry For details see this thread https://forum.sentry.io/t/unhandledrejection-non-error-promise-rejection-captured-with-value/14062 |
In the next version - 6.9.0 (releasing as we speak), |
See getsentry/sentry-javascript#3440 for more information
See getsentry/sentry-javascript#3440 for more information
Is there any way to disable sending those events to sentry? We are on |
Sentry.init({
ignoreErrors: ['Non-Error promise rejection captured']
}); or through |
It's been a while, but this has come up again so just going to chime in here: All of the above hacks work, but are broader than they need to be, and therefore risk filtering out unrelated events. As Kamil said above, Since this can happen in any number of different circumstances, if you want to filter the events mentioned above (empty rejections and rejections with the As for the Ultimately, the ideal solution would be to work with the Outlook folks to get them to not throw the error in the first place. |
In our case the error says now:
The Id seems to change from time to time, though. MethodName and ParamCount seem to have been added in cefsharp/CefSharp#4209 ? |
we can also see the following for example:
|
I don't think there is much we can do about this, I feel like we give a reasonable amount of detail in this case now? This means that something like this is happening: Promise.reject({ value: 'Object Not Found Matching Id:5, MethodName:update, ParamCount:4' }) In which case I'd say this is as reasonable a thing to capture as any? You should be able to filter this via |
Hi Sentry team, This error and ticket is already closed but I am facing this error many times.
How can I do? How did everyone solve? |
it still exist |
As mentioned before, we simply try to build error messages from objects that we receive that are not actual |
This is happening for us all of a sudden too. hiroshinishio or anyone have an example of a configuration that works? My issue adding an exception to the error handle is how can we be sure this doesn't happen in some other situation which is more critical than outlook crawling it. I'd hate to ignore the error now, and that have it actually impact users in the future and have no visibility into that. |
Hi all!
I didn't want to create a bug as I don't believe there is a bug here considering this error has been talked about a few times here. TL:DR - We are receiving a copious amount i these errors with no values provided. I can't see anything that sticks out in our code for why this is happening, but I don't want to dismiss this as an external factor. Some information about our Sentry Setup:
Was curious if anyone has any advice/thoughts on how to to begin debugging this further? Thanks in advance for your time!
The text was updated successfully, but these errors were encountered: