archive-gh-me/log4js-node
1
0
Fork 0
mirror of https://github.com/log4js-node/log4js-node.git synced 2025-12-08 19:26:01 +00:00
Code Issues Projects Releases Wiki Activity

chore(refactor): fileSyncAppender to have same internal code ordering as fileAppender

Browse Source
This commit is contained in:
Lam Wei Li 2022-03-06 21:22:08 +08:00
parent 3dfa03e36b
commit 8d2a211a9f
No known key found for this signature in database
GPG Key ID: 90F6ABECF080D7BF
1 changed files with 14 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
lib/appenders/fileSync.js
View File

@ -175,15 +175,22 @@ class RollingFileSync {
* if not provided then logs won't be rotated.
* @param numBackups - the number of log files to keep after logSize
* has been reached (default 5)
* @param timezoneOffset - optional timezone offset in minutes
* (default system local)
* @param options - passed as is to fs options
* @param options - options to be passed to the underlying stream
* @param timezoneOffset - optional timezone offset in minutes (default system local)
*/
function fileAppender(file, layout, logSize, numBackups, timezoneOffset, options) {
debug('fileSync appender created');
function fileAppender(file, layout, logSize, numBackups, options, timezoneOffset) {
file = path.normalize(file);
numBackups = (!numBackups && numBackups !== 0) ? 5 : numBackups;
debug(
'Creating fileSync appender (',
file, ', ',
logSize, ', ',
numBackups, ', ',
options, ', ',
timezoneOffset, ')'
);
function openTheStream(filePath, fileSize, numFiles) {
let stream;
@ -234,8 +241,8 @@ function configure(config, layouts) {
layout,
config.maxLogSize,
config.backups,
config.timezoneOffset,
options
options,
config.timezoneOffset
);
}