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.

Sign up for GitHub

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

Jump to bottom

fix: all files ending by a timestamp were validated #61

Merged
merged 1 commit into from
Apr 19, 2020
Merged

fix: all files ending by a timestamp were validated #61

merged 1 commit into from
Apr 19, 2020

Conversation

rnd-debug
Copy link
Contributor

Related to issue #60 . Files that :

  • have a "base name" that is not matching the template,
  • but were ending by a suitable timestamp
    were taken into consideration for the log rolling.

@nomiddlename nomiddlename added this to the 2.2.4 milestone Apr 19, 2020
@nomiddlename nomiddlename merged commit a4e5da1 into log4js-node:master Apr 19, 2020
@nomiddlename
Copy link
Collaborator

Thanks for this, really appreciate it.

@nomiddlename
Copy link
Collaborator

Published to npm in [email protected]

@lamweili lamweili linked an issue Jan 6, 2022 that may be closed by this pull request
daysToKeep affects files with different names in the same folder #60
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
2.2.4
Development

Successfully merging this pull request may close these issues.

daysToKeep affects files with different names in the same folder
2 participants
@rnd-debug @nomiddlename