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

Resolving fix deleting files #113

Closed
wants to merge 5 commits into from
Closed

Conversation

amcereijo
Copy link
Contributor

If a log file was versioned the versioned files were not be deleted.
Example:

A log file called mylog.log.
Each time the file reach 50mb it will be versioned.
It will create mylog.log.1, mylog.log.2, etc... (for example)
Until now the process only deleted mylog.log. From now on, the process also will delete mylog.log1, mylog.log.2, etc...

amcereijo and others added 5 commits August 4, 2017 19:23
…By now, only files with exact file name are removed. Copy files (file.log.1, file.log-21-09-2017,...) were not included
# Conflicts:
#	README.md
#	index.js
…By now, only files with exact file name are removed. Copy files (file.log.1, file.log-21-09-2017,...) were not included
@mattberther
Copy link
Member

@amcereijo Some long-overdue refactoring of the transport has occurred. A PR is available at #107. Also, a pre-release version has been published to npm. Please npm install [email protected]. This version should address the issue brought forward with this PR. Please set the maxFiles property to the maximum number of files to keep.

Closing for now. Please re-open if you continue to see old files remaining.

@amcereijo
Copy link
Contributor Author

@mattberther when are you going to publish a final version for 2.0?

@mattberther
Copy link
Member

@amcereijo I'm just waiting for word from @indexzero that the winstonjs transport architecture is finalized for 3.x. I'm expecting this to be soon-ish...

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

Successfully merging this pull request may close these issues.

2 participants