Skip to content

Commit 3082b98

Browse files
authored
Add message to the error consistent with one in proxy.js (#5518)
1 parent 22dfc65 commit 3082b98

2 files changed

Lines changed: 5 additions & 4 deletions

File tree

packages/dd-trace/src/profiler.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ module.exports = {
1414
debug: (message) => log.debug(message),
1515
info: (message) => log.info(message),
1616
warn: (message) => log.warn(message),
17-
error: (message) => log.error(message)
17+
error: (...args) => log.error(...args)
1818
}
1919

2020
const libraryInjected = injectionEnabled.length > 0

packages/dd-trace/src/profiling/profiler.js

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -18,9 +18,9 @@ function maybeSourceMap (sourceMap, SourceMapper, debug) {
1818
], debug)
1919
}
2020

21-
function logError (logger, err) {
21+
function logError (logger, ...args) {
2222
if (logger) {
23-
logger.error(err)
23+
logger.error(...args)
2424
}
2525
}
2626

@@ -52,7 +52,8 @@ class Profiler extends EventEmitter {
5252

5353
start (options) {
5454
return this._start(options).catch((err) => {
55-
logError(options.logger, err)
55+
logError(options.logger, 'Error starting profiler. For troubleshooting tips, see ' +
56+
'<https://dtdg.co/nodejs-profiler-troubleshooting>', err)
5657
return false
5758
})
5859
}

0 commit comments

Comments
 (0)