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

ALL: changed another file. to fileinfo. #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

decanio
Copy link
Contributor

@decanio decanio commented Apr 2, 2014

I found another file that had to be changed to fileinfo in the ALL dashboard.

@pevma
Copy link
Owner

pevma commented Apr 2, 2014

I saw and patched that up last night :)
Thanks Tom !

Regards,
Peter Manev

On 2 apr 2014, at 07:21, Tom DeCanio [email protected] wrote:

I found another file that had to be changed to fileinfo in the ALL dashboard.

You can merge this Pull Request by running

git pull https://github.com/decanio/Suricata-Logstash-Templates ALL-patch
Or view, comment on, or merge it at:

#1

Commit Summary

ALL: changed another file. to fileinfo.
File Changes

M Templates/ALL (4)
Patch Links:

https://github.com/pevma/Suricata-Logstash-Templates/pull/1.patch
https://github.com/pevma/Suricata-Logstash-Templates/pull/1.diff

Reply to this email directly or view it on GitHub.

@decanio
Copy link
Contributor Author

decanio commented Apr 2, 2014

OK. I guess it hadn't hit your github repo yet.

Tom

On Wed, Apr 2, 2014 at 5:48 AM, Peter Manev [email protected]:

I saw and patched that up last night :)
Thanks Tom !

Regards,
Peter Manev

On 2 apr 2014, at 07:21, Tom DeCanio [email protected] wrote:

I found another file that had to be changed to fileinfo in the ALL
dashboard.

You can merge this Pull Request by running

git pull https://github.com/decanio/Suricata-Logstash-TemplatesALL-patch
Or view, comment on, or merge it at:

#1

Commit Summary

ALL: changed another file. to fileinfo.
File Changes

M Templates/ALL (4)
Patch Links:

https://github.com/pevma/Suricata-Logstash-Templates/pull/1.patch

https://github.com/pevma/Suricata-Logstash-Templates/pull/1.diff

Reply to this email directly or view it on GitHub.

Reply to this email directly or view it on GitHubhttps://github.com//pull/1#issuecomment-39290962
.

@pevma
Copy link
Owner

pevma commented Apr 2, 2014

No , I meant - I found that out by myself accidentally last night and fixed it :)

Regards,
Peter Manev

On 2 apr 2014, at 08:17, Tom DeCanio [email protected] wrote:

OK. I guess it hadn't hit your github repo yet.

Tom

On Wed, Apr 2, 2014 at 5:48 AM, Peter Manev [email protected]:

I saw and patched that up last night :)
Thanks Tom !

Regards,
Peter Manev

On 2 apr 2014, at 07:21, Tom DeCanio [email protected] wrote:

I found another file that had to be changed to fileinfo in the ALL
dashboard.

You can merge this Pull Request by running

git pull https://github.com/decanio/Suricata-Logstash-TemplatesALL-patch
Or view, comment on, or merge it at:

#1

Commit Summary

ALL: changed another file. to fileinfo.
File Changes

M Templates/ALL (4)
Patch Links:

https://github.com/pevma/Suricata-Logstash-Templates/pull/1.patch

https://github.com/pevma/Suricata-Logstash-Templates/pull/1.diff

Reply to this email directly or view it on GitHub.

Reply to this email directly or view it on GitHubhttps://github.com//pull/1#issuecomment-39290962
.


Reply to this email directly or view it on GitHub.

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