lib: fix module print timing when specifier includes `"`

[aduh95]

Currently, we generate invalid JSON if the specifier contains ", this can be addressed by escaping all double quotes like this PR is doing.

[codecov[bot]]

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 88.18%. Comparing base (f5d454a) to head (bda9e62). Report is 917 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #55150      +/-   ##
==========================================
- Coverage   88.23%   88.18%   -0.05%     
==========================================
  Files         651      649       -2     
  Lines      183902   183327     -575     
  Branches    35855    35686     -169     
==========================================
- Hits       162269   161672     -597     
- Misses      14926    15024      +98     
+ Partials     6707     6631      -76     

Files with missing lines	Coverage Δ
lib/internal/util/debuglog.js	95.20% <100.00%> (ø)

... and 56 files with indirect coverage changes

[H4ad]

From what I remember, almost all places that use/expose trace labels are susceptible to this issue, I don't remember exactly the places but I think could be something worthy to take a double look.

Fixing this implementation also fixes the console.time implementation, so if you want to include a test there too, could be useful.

[aduh95]

Fixing this implementation also fixes the console.time implementation, so if you want to include a test there too, could be useful.

Do you have an example? Naive testing (node -e 'console.time(`test"`);console.timeEnd(`test"`)') doesn't show the issue.

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/62840/

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/62842/

[H4ad]

@aduh95 node --trace-event-categories -e 'console.time(`test"`);console.timeEnd(`test"`)'

The generated logs are in json, and the json file is broken.

Edit2: I'm okay with adding tests in another PR (since the CI of this PR is already green)

[nodejs-github-bot]

Landed in 78f421de886b80be099e597845d8b3235200fdfa

[joyeecheung]

Tentatively backport in https://github.com/nodejs/node/pull/56927 to reduce conflicts. Though this is not very essential and might get dropped.

[joyeecheung]

Backing it out from https://github.com/nodejs/node/pull/56927 to avoid introducing surface for regressions.