No Console Logs Recorded in the Session Replay

getsentry / sentry-javascript

Official Sentry SDKs for JavaScript

https://sentry.io

MIT License

7.87k stars 1.55k forks source link

No Console Logs Recorded in the Session Replay #12244

Closed Kobby-Bawuah closed 3 months ago

Kobby-Bawuah commented 3 months ago

Is there an existing issue for this?

[X] I have checked for existing issues https://github.com/getsentry/sentry-javascript/issues
[X] I have reviewed the documentation https://docs.sentry.io/
[X] I am using the latest SDK release https://github.com/getsentry/sentry-javascript/releases

How do you use Sentry?

Sentry Saas (sentry.io)

Which SDK are you using?

@sentry/browser

SDK Version

8.4.0

Framework Version

No response

Link to Sentry event

No response

SDK Setup

Steps to Reproduce

Implement Sentry as described in the attached screenshots.
Trigger console.log() statements in the client.
Check the session replay for recorded console logs.

Expected Result

console.log() statements should be visible in the session replay.

Actual Result

No console.log() statements are visible, although errors are correctly reported.

my_console

mydea commented 3 months ago

Hey,

we have tests specifically ensuring that this works 🤔 do you have a link to a replay where this happens? Could you try with debug: true enabled, would be interesting to see where the logs go in terms of timeline.

billyvg commented 3 months ago

@mydea do we have any size limits on console logs? The log message seems to be quite big.

For the error, it looks like it's an uncaught rejection which is capture as an exception to Sentry. We do not display these in the Console tab of Replay unless it was explicitly called with console.error().

Hugo-Dz commented 3 months ago

@mydea do we have any size limits on console logs? The log message seems to be quite big.

For the error, it looks like it's an uncaught rejection which is capture as an exception to Sentry. We do not display these in the Console tab of Replay unless it was explicitly called with console.error().

Do you mean it doesn't record console.log() but only console.error()?

mydea commented 3 months ago

@mydea do we have any size limits on console logs? The log message seems to be quite big.

We truncate logs when we add them as breadcrumbs to replay - we have specifically a test for this 😅 https://github.com/getsentry/sentry-javascript/blob/develop/dev-packages/browser-integration-tests/suites/replay/captureConsoleLog/test.ts#L59

billyvg commented 3 months ago

@Hugo-Dz no, we capture both, I was more remarking that the Error message you see in the console is not called with console.error and so it's expected that you won't see it in the Replay's Console tab (this is because it's captured as an exception and displayed in Breadcrumbs).

Can you give us a repro for this? Or a publicly accessible URL where we can debug?

Hugo-Dz commented 3 months ago

@billyvg For some reasons, logs now shows up in my Replays, not sure why it was not the case before 🤔

You can close this :)

mydea commented 3 months ago

If somebody encounters this again, please re-open the issue!