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

Dnstap support added #215

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

Conversation

gartline
Copy link

No description provided.

@astrolox
Copy link

@blblack This pull request is DNSTAP support that was developed internally in Tiggee for our own use. As we're now ready to contribute this back to the open source, please let us know if there are any changes that you need from us before it can be merged.

@astrolox
Copy link

astrolox commented Nov 1, 2021

Fixes #212

@blblack
Copy link
Member

blblack commented Jan 7, 2022

Sorry for the lack of a reply for such a long time. Life and work has been busy, and I've been ignoring the project a bit and hadn't even noticed this!

At a quick glance, this looks pretty good - thank you so much for sending the pull req! As soon as I get some time, I'll dig into it a bit deeper and see what we can do about getting this work into the master branch and future feature releases.

@astrolox
Copy link

@blblack
Sorry for the nudge, just wondering if we could get this merged?
We'd love to see it included in future releases.

If there are any changes needed; I'm not sure if @gartline is still available to do work to help with this as he's no longer working with us, and we all know about the situation in Ukraine at the moment.

@bachlx19992
Copy link

Sorry for the lack of a reply for such a long time. Life and work has been busy, and I've been ignoring the project a bit and hadn't even noticed this!

At a quick glance, this looks pretty good - thank you so much for sending the pull req! As soon as I get some time, I'll dig into it a bit deeper and see what we can do about getting this work into the master branch and future feature releases.

have you done this yet?

This was referenced Jan 3, 2024
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.

None yet

4 participants