Describe the bug
logOptions handled incorrectly when sync: true is used.
Expected behavior
logOptions are passed to the Logging.logSync function correctly.
Additional context
The log function accepts two arguments.
log(name: string, options?: LogOptions) {
Whereas the logSync function accepts three arguments, with options expected to be last.
logSync(name: string, transport?: Writable, options?: LogSyncOptions) {
In Cloud Pine, two arguments are passed to both the log and logSync functions. This results in no logs being sent to STDOUT when logSync is used because the options are overwriting the default transport.
this._log = this._sync
? this.logging.logSync(this.name, this._logOptions)
: this.logging.log(this.name, this._logOptions)
Describe the bug
logOptionshandled incorrectly whensync: trueis used.Expected behavior
logOptionsare passed to theLogging.logSyncfunction correctly.Additional context
The
logfunction accepts two arguments.Whereas the
logSyncfunction accepts three arguments, withoptionsexpected to be last.In Cloud Pine, two arguments are passed to both the
logandlogSyncfunctions. This results in no logs being sent to STDOUT whenlogSyncis used because the options are overwriting the default transport.