-
-
Notifications
You must be signed in to change notification settings - Fork 82
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
Enriched dot
report
#797
Comments
Coming back from #875
|
|
That's great! I should be able to emulate something akin to that. One challenge here is the treatment of "unknown" nodes, Trippy currently conflates all unknown nodes into a single node as part of the
The 1st option is what Trippy does today which isn't great. It appear from the example you have shared that Scapy chooses the 3rd option which leads to a lot of "duplicate" unknown nodes (58 in your example). Perhaps the 2nd option is a good compromise? Again, this would be something that would be nice to expose to users as an option as there seems to be no clear "right" answer. |
Enrich the Graphviz
dot
report:The text was updated successfully, but these errors were encountered: