pinojs / sonic-boom

Extremely fast utf8 only stream implementation
MIT License
266 stars 41 forks source link

prevent memory leak when writing before ready #183

Closed rluvaton closed 1 year ago

rluvaton commented 1 year ago

while trying to add a callback function to flush in pino I got into a situation where I logged and flushed before the file was ready so the cb never got called

rluvaton commented 1 year ago

I just noticed that the branch name has nothing to do with the fix 😅

rluvaton commented 1 year ago

let's merge this so I can continue working on pino flush?

(don't forget hacktoberfest 😅 I need some shirts 😂 )

rluvaton commented 1 year ago

Can you please release that so pinojs/pino#1827 can be merged?