[BUG] empty messages for unhandled promise rejections

[JohnnyFun]

Description/Screenshot
message is passed as theEvent from Exception.CreateAutoException. Should evt been passed instead? And/or should you fall back to theEvent if it's a string?

Results in these empty error messages, so I don't see the actual error.

• SDK Version: 2.6.4

[MSNev]

Should evt been passed instead?

No

And/or should you fall back to theEvent if it's a string?

Yes

[MSNev]

The fix is going to look like this

   if (evtMessage && !isString(evtMessage)) {
       evtMessage = theEvent[strMessage] || theEvent[strDescription] || evtMessage;
   }

[JohnnyFun]

Sounds good, thanks for jumping on that. I'll just downgrade for now. When do you think that'll be released?

Fwiw, my stack trace also was empty. Looking at the code it looks like it would've used error.stack, so not sure why that happened. But I'll let you know if I find anything.

[MSNev]

At the moment it will be the next cycle so we will start this process in the first week of August

[MSNev]

v2.6.5 is now fully deployed

[JohnnyFun]

Looks like message is coming through now, but there appears to be no stack trace.

It used to show up in this "details" section from azure insights UI:

The user was using chrome 92, so shouldn't be a browser issue where it just provides a terrible error.stack value or something.

Let me know if you need any other information. Worst case, I might just add that as a custom dimension for now.

And fwiw, that's a custom error that we explicitly throw from a js file: throw new Error('Should not call this with student persona.')

[github-actions]

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

[MSNev]

#1679