jdb/personal-measure
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix bug in API logger (used setInterval instead of setTimeout).

Browse Source
This commit is contained in:
Jonathan Bernard 2019-05-18 12:20:16 -05:00
parent 2380bf8d1e
commit 0ce1581a87
1 changed files with 14 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
web/src/services/logging/api-log-appender.ts
View File

@ -22,7 +22,7 @@ export class ApiLogAppender implements LogAppender {
private lastSent = 0;
constructor(public readonly apiEndpoint: string, public authToken?: string, threshold?: LogLevel) {
setInterval(this.checkPost, 1000);
setTimeout(this.checkPost, 1000);
if (threshold) { this.threshold = threshold; }
}
@ -38,18 +38,6 @@ export class ApiLogAppender implements LogAppender {
});
}
private checkPost = () => {
const now = Date.now();
const min = this.lastSent + (this.minimumTimePassedInSec * 1000);
const max = this.lastSent + (this.maximumTimePassedInSec * 1000);
if ( (this.msgBuffer.length >= this.batchSize && min < now) ||
(this.msgBuffer.length > 0 && max < now) ) {
this.doPost();
}
setInterval(this.checkPost, Math.max(10000, this.minimumTimePassedInSec * 1000));
}
private doPost() {
if (this.msgBuffer.length > 0 && this.authToken) {
@ -63,6 +51,19 @@ export class ApiLogAppender implements LogAppender {
this.msgBuffer = [];
}
}
private checkPost = () => {
const now = Date.now();
const min = this.lastSent + (this.minimumTimePassedInSec * 1000);
const max = this.lastSent + (this.maximumTimePassedInSec * 1000);
if ( (this.msgBuffer.length >= this.batchSize && min < now) ||
(this.msgBuffer.length > 0 && max < now) ) {
this.doPost();
}
setTimeout(this.checkPost, Math.max(10000, this.minimumTimePassedInSec * 1000));
}
}
export default ApiLogAppender;