Sentry 6.17.7 changes logged message from String to Array

[erikt9]

• Review the documentation: https://docs.sentry.io/
• Search for existing issues: https://github.com/getsentry/sentry-javascript/issues
• Use the latest release: https://github.com/getsentry/sentry-javascript/releases
• Provide a link to the affected event from your Sentry account

Package + Version

• @sentry/browser
• @sentry/node
• raven-js
• raven-node (raven for node)
• other:

Version:

6.17.7

Description

I believe this is related to #4533.

When opening the dev console on previous sentry version and writing a simple log, the message is written as a string:

console.log('hello world')
[Log] hello world
< undefined

With 6.17.7, message is written as an Array.

console.log('hello world');
[Log] ["hello world"]
< undefined

I see this behavior in latest Safari, Firefox, and Chrome.

[lobsterkatie]

Hi, @erikt9. Sorry you're having trouble!

I've tried to replicate this behavior and haven't been able to. Might there be something else in your app also touching the console methods?

Chrome:

FF:

Safari:

[erikt9]

@lobsterkatie
We do our own instrumentation of console.error for our own logging, but none of the others (console.log, debug, info, warn etc). The behavior in on the others as well.

We are loading the latest sentry via the lazy load method:
https://docs.sentry.io/platforms/javascript/install/lazy-load-sentry/
and our version is set to latest in the javascript loader section of our preferences.

The above lines for finding the version don't seem to work when loaded this way, so perhaps it is also a consequence of lazy loading? Removing sentry from our page does fix the problem.

[ramosbugs]

I can also confirm that this behavior started with the 6.17.7 release.

it reproduces for me in the Chrome console:

[ramosbugs]

Might there be something else in your app also touching the console methods?

I was curious about this too, so I put in a breakpoint, but it looks like the underlying log function is native code, so I don't think anything else is wrapping it, at least for my project:

[erikt9]

Same for us regarding the console methods. console.error does reflect our instrumentation, but console.log shows native code.

[princemaple]

Hi, getting the same issue. And on a different app without sentry:

[dbendaou]

Same issue here and the console.log function seems to be inherited from instrument.ts
I can even reproduce the issue on sentry.io

This could be the issue someone found:
https://github.com/getsentry/sentry-javascript/pull/4533/files#r805670094

Same problem here.