-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Unhandled Promise rejection: toISOString is not a function ; Zone: <root> #1798
Comments
I did come across angular/angular#17772 which highlights why the change detection might not of triggered... but not why that error blew up int he first place |
@kevcjones can you give it a try with |
I gave that a try and the Date specific error is gone but now is replaced with
|
I also get the error @kevcjones mentioned in his last post. I tried switching vom raven-js to @sentry/browser@4.4.2 |
Any news on this? |
I get same error. |
Hey guys, any move on this? I need to bring up the confidence in the tool or we'll have to move away... |
I get same error. My stack: |
Sorry about this issue guys but are you able to provide a small repro case for this? Also please make sure to use the latest version |
I tried it with a clean angluar 7 project, not doing more than here locally: Works without any issues. Please provide a repro case. |
I can't upgrade to 7 just yet, but I'll give the latest version a drive and
see if it's resolved.
…On Tue, 15 Jan 2019, 14:42 Daniel Griesser ***@***.*** wrote:
I tried it with a clean angluar 7 project, not doing more than here
locally:
https://codesandbox.io/s/jpm71lq17w
Works without any issues. Please provide a repro case.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1798 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAqe8Lr7L5XA-pL2UgPe_BTJnaBlvpeCks5vDejagaJpZM4ZRsY8>
.
|
@HazAT follow an example reproducing the error: https://github.com/candidosales/sentry-error |
@candidosales Thanks for the repro case, confirmed, also crashes for me. |
|
@HazAT I made upgrade the version to 4.5.2, stopped the error looping but I don't receive the errors in my account. Can you verify? |
@candidosales Hey, was trying your example app and the error wasn't sent :| But I found the issue and fixed it, sorry about that, see: #1842 |
@HazAT When will publish the new version with the fix? |
Just to also update this here, version |
What about raven-js package ? our team got the same problem, should we switch to @sentry/browser ? |
@92hackers Since Keep in mind |
Package + Version
@sentry/browser
@sentry/node
raven-js
raven-node
(raven for node)Version:
Description
An API endpoint threw this
Inside the Error handler as per your docs, the following error message blow up my change detection
This prevented my modal for sign in from rendering to the screen.
I have found a work around where i simply don't give you this error message and handle the error locally, however i'd like some insights if you have any as to why the serializer inside @sentry has trouble parsing some XML?
some extra info
Browser witnessed in Chrome 70
Cheers
The text was updated successfully, but these errors were encountered: