Changing filewrite behavior

This commit is contained in:
Fabian Stamm 2021-05-08 22:43:46 +02:00
parent c58d75129d
commit 0742490527
2 changed files with 12 additions and 14 deletions

View File

@ -1,6 +1,6 @@
{ {
"name": "@hibas123/nodelogging", "name": "@hibas123/nodelogging",
"version": "3.0.4", "version": "3.0.5",
"description": "", "description": "",
"main": "out/index.js", "main": "out/index.js",
"types": "out/index.d.ts", "types": "out/index.d.ts",
@ -32,7 +32,7 @@
"typescript": "^4.2.4" "typescript": "^4.2.4"
}, },
"dependencies": { "dependencies": {
"@hibas123/logging": "^3.0.3", "@hibas123/logging": "^3.0.5",
"@hibas123/utils": "^2.2.18" "@hibas123/utils": "^2.2.18"
} }
} }

View File

@ -7,12 +7,7 @@ const MAX_FILE_SIZE = 500000000;
export class LoggingFiles implements Adapter { export class LoggingFiles implements Adapter {
file: Files; file: Files;
constructor( constructor(private filename: string, private maxFileSize = MAX_FILE_SIZE) {}
private filename: string,
private error = false,
private maxFileSize = MAX_FILE_SIZE,
private noPrefix = false
) {}
init() { init() {
if (!this.file) { if (!this.file) {
@ -58,20 +53,22 @@ export class Files {
private stream: fs.WriteStream = undefined; private stream: fs.WriteStream = undefined;
private lock = new Lock(); private lock = new Lock();
private $initialized = false; #initialized = false;
#iv: any;
public get initlialized() { public get initlialized() {
return this.$initialized; return this.#initialized;
} }
private constructor(private file: string) {} private constructor(private file: string) {}
public async init(maxFileSize: number) { public async init(maxFileSize: number) {
if (this.$initialized) return; if (this.#initialized) return;
let lock = await this.lock.getLock(); let lock = await this.lock.getLock();
this.maxFileSize == maxFileSize; this.maxFileSize == maxFileSize;
await this.initializeFile(); await this.initializeFile();
this.$initialized = true; this.#initialized = true;
this.#iv = setInterval(() => this.checkQueue(), 500);
lock.release(); lock.release();
this.checkQueue(); this.checkQueue();
} }
@ -104,7 +101,7 @@ export class Files {
this.size = size; this.size = size;
} catch (e) { } catch (e) {
console.log(e); console.log(e);
//ToDo is this the right behavior? //TODO: is this the right behavior?
process.exit(1); process.exit(1);
} }
} }
@ -125,10 +122,12 @@ export class Files {
await this.flush(false); await this.flush(false);
this.open--; this.open--;
if (this.open <= 0) { if (this.open <= 0) {
clearInterval(this.#iv);
this.stream.close(); this.stream.close();
Files.files.delete(this.file); Files.files.delete(this.file);
} }
} }
public flush(sync: boolean) { public flush(sync: boolean) {
if (sync) { if (sync) {
// if sync flush, the process most likely is in failstate, so checkQueue stopped its work. // if sync flush, the process most likely is in failstate, so checkQueue stopped its work.
@ -165,7 +164,6 @@ export class Files {
public write(data: Buffer) { public write(data: Buffer) {
this.queue.push(data); this.queue.push(data);
this.checkQueue();
} }
public dispose() {} public dispose() {}