Skip to content

fix: create safeJoin util for console wrappers#1222

Merged
kamilogorek merged 1 commit intomasterfrom
safer-console-instrumentation
Feb 8, 2018
Merged

fix: create safeJoin util for console wrappers#1222
kamilogorek merged 1 commit intomasterfrom
safer-console-instrumentation

Conversation

@kamilogorek
Copy link
Contributor

Fixes #1219

This should fix the issue for now. I didn't pass values through JSON.serialize as some of them might be rather large, and we use msg only for logging in sentry, not in the console itself - there, you can still see the full value, as with regular console calls.

@kamilogorek kamilogorek requested a review from a team February 8, 2018 14:21
@kamilogorek
Copy link
Contributor Author

Before:
screen shot 2018-02-08 at 15 19 17

After:
screen shot 2018-02-08 at 15 20 14

@kamilogorek kamilogorek merged commit df8a9cd into master Feb 8, 2018
@kamilogorek kamilogorek deleted the safer-console-instrumentation branch February 8, 2018 14:34
chargome added a commit that referenced this pull request Mar 20, 2026
…ix h3 CVE

Fixes Dependabot alerts #1221 and #1222. Bumps nuxt from 3.17.7 to 3.21.2
and @nuxt/nitro-server from ^3.21.1 to ^3.21.2 to pull in h3 >=1.15.6,
fixing SSE injection (CVE-2026-33128) and path traversal vulnerabilities.

Co-Authored-By: Claude Sonnet 4.6 <[email protected]>
chargome added a commit that referenced this pull request Mar 20, 2026
…ix h3 CVE

Fixes Dependabot alerts #1221 and #1222. Bumps nuxt from 3.17.7 to 3.21.2
and @nuxt/nitro-server from ^3.21.1 to ^3.21.2 to pull in h3 >=1.15.6,
fixing SSE injection (CVE-2026-33128) and path traversal vulnerabilities.

Co-Authored-By: Claude Sonnet 4.6 <[email protected]>
chargome added a commit that referenced this pull request Mar 23, 2026
…ix h3 CVE

Fixes Dependabot alerts #1221 and #1222. Bumps nuxt from 3.17.7 to 3.21.2
and @nuxt/nitro-server from ^3.21.1 to ^3.21.2 to pull in h3 >=1.15.6,
fixing SSE injection (CVE-2026-33128) and path traversal vulnerabilities.

Co-Authored-By: Claude Sonnet 4.6 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Console Log Instrumentation breaks on foreign objects

2 participants