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

Don't try to tail directories #11

Open
leonsodhi opened this issue Jun 6, 2014 · 3 comments
Open

Don't try to tail directories #11

leonsodhi opened this issue Jun 6, 2014 · 3 comments

Comments

@leonsodhi
Copy link

If I have a directory structure such as:

/var/log/test/test.log
/var/log/test/subtest

and specify /var/log/test/* in the config, the following error is produced:

2014-06-06 21:38:46 INFO  remote_syslog.go:86 Forwarding /var/log/test/test.log
2014-06-06 21:38:46 INFO  remote_syslog.go:86 Forwarding /var/log/test/subtest
2014-06-06 21:38:46 ERROR  remote_syslog.go:50 Tail worker executed abnormally
@coreybutler
Copy link

+1

@joelreymont joelreymont mentioned this issue Jul 6, 2015
@johlym
Copy link
Contributor

johlym commented Jun 1, 2016

This would require logic to determine if /var/log/test/subtest is a file with no extension or a directory. Not common but some log files don't have extensions.

@leonsodhi
Copy link
Author

There was some work done to solve this in the linked "Issue 11" PR. I think this would do it. That said, we can't risk merging it as so much other stuff has changed that we really need to add tests for all the basic functionality to make sure nothing breaks.

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

3 participants