Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Race condition when replacing %DATE% placeholder #112

Open
R1D3R175 opened this issue Nov 2, 2024 · 0 comments
Open

Race condition when replacing %DATE% placeholder #112

R1D3R175 opened this issue Nov 2, 2024 · 0 comments

Comments

@R1D3R175
Copy link

R1D3R175 commented Nov 2, 2024

While using winston-daily-rotate-file I noticed that sometimes the %DATE% placeholder will get replaced and sometimes not, opened an issue on their repo but, thanks to @coreydaley, it turns out that the issue was caused by the library handling the rotation (which should be this).

For more details check the original issue winstonjs/winston-daily-rotate-file#396

@R1D3R175 R1D3R175 changed the title Rotation caused by maxSize writes '%DATE%' instead of writing date Race condition when replacing %DATE% placeholder Nov 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant