Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Cannot decapsulate GRE tunnels. #53

Open
systemxcom opened this issue Nov 19, 2024 · 8 comments
Open

Cannot decapsulate GRE tunnels. #53

systemxcom opened this issue Nov 19, 2024 · 8 comments
Labels
enhancement New feature or request
Milestone

Comments

@systemxcom
Copy link

Is there a flag in nDPId to enable tunnels, like GRE, to be decapsulated? Or have we missed this in the documentation.

@utoni
Copy link
Owner

utoni commented Nov 19, 2024

Nope, GRE tunnel dissection is currently not supported. But I'll add this feature ASAP.

@utoni utoni added the enhancement New feature or request label Nov 19, 2024
@utoni utoni added this to the 1.8 milestone Nov 19, 2024
@systemxcom
Copy link
Author

Thank you for a great capability.

utoni added a commit that referenced this issue Nov 22, 2024
 * make finally use of the thread distribution seed

Signed-off-by: Toni Uhlig <matzeton@googlemail.com>
@utoni
Copy link
Owner

utoni commented Nov 22, 2024

Hey @systemxcom, please try PR #55 and give me some feedback.
I won't merge it now, because some additional tests (as PCAP files) are needed.
Those will most likely get merged in upstream nDPI as the code is pretty similar related to GRE decoding.

@systemxcom
Copy link
Author

Thank you Utoni! Will test right away and provide feedback.

@systemxcom
Copy link
Author

Hi @utoni, tested in a lab on live network traffic, (no pcap). Working as expected. Will test at scale on this coming Monday. I see there is no -flag necessary and it decapsulates all identified GRE tunnels. At some point may want to add a -tunnel flag so operators can have the option not to decapsulate when monitoring complex environments. Great work and I will get you additional feedback if anything material crops up. Thank you for your hard work.

@utoni
Copy link
Owner

utoni commented Nov 23, 2024

Hey @systemxcom,
there will be some additions to this PR i.e. adding -t to enable tunnel decoding.
Appreciated & You're welcome! 😸

@utoni
Copy link
Owner

utoni commented Nov 24, 2024

-t as command line parameter added, decode-tunnel= as configuration option

PR #55 is now complete and will only get bug fixes before merging.
I'll start with long-term tests. Please do the same on your side. 😃

@systemxcom
Copy link
Author

Will have testing feedback Thursday, this week.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants