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

scl: add mbox() source #703

Merged
merged 1 commit into from
Sep 22, 2015
Merged

scl: add mbox() source #703

merged 1 commit into from
Sep 22, 2015

Conversation

bazsi
Copy link
Collaborator

@bazsi bazsi commented Sep 16, 2015

This source can be used to fetch emails from local mbox files:

source { mbox("/var/spool/mail/root"); };

This will fetch root emails and parse them into a multiline $MSG.
Original implementation by Fabien Wernli, I only converted it into
an SCL.

Signed-off-by: Balazs Scheidler balazs.scheidler@balabit.com

This source can be used to fetch emails from local mbox files:

source { mbox("/var/spool/mail/root"); };

This will fetch root emails and parse them into a multiline $MSG.
Original implementation by Fabien Wernli, I only converted it into
an SCL.

Signed-off-by: Balazs Scheidler <balazs.scheidler@balabit.com>
@faxm0dem
Copy link
Contributor

Why not add the python part to the SCL?

@ihrwein
Copy link
Contributor

ihrwein commented Sep 21, 2015

That may define a dependency on the Python bindings. We are evaluating some ideas how to solve this issue.

@faxm0dem
Copy link
Contributor

okay, nevertheless I already made some changes to the python part so ping me when/if you need a new version

@lbudai
Copy link
Collaborator

lbudai commented Sep 22, 2015

I will merge this PR as it is, but we are working on a new feature for blocks which will help us to handle that Python dependency (or more accurately: use Python part only when syslog-ng compiled with Python support - and this is not restricted to Python).

lbudai added a commit that referenced this pull request Sep 22, 2015
@lbudai lbudai merged commit e5acdef into master Sep 22, 2015
@faxm0dem
Copy link
Contributor

Awesome!

@bazsi bazsi deleted the f/scl-add-mbox-source branch January 14, 2016 06:43
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.

4 participants