Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@snicket2100
Copy link
Contributor

No description provided.

@snicket2100 snicket2100 force-pushed the mullvad-protocol-added branch 2 times, most recently from b36de0e to 2d79f14 Compare July 28, 2023 15:30
Copy link
Collaborator

@IvanNardi IvanNardi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great and complete work.
Just a minor issue: could you add a simple pcap with mulvad traffic? One TLS or DNS flow matching "mullvad.net" domain and one flow matching one of the ips; just a few pkts for each flow...

@snicket2100
Copy link
Contributor Author

Ah yes, of course. I will add.

@snicket2100 snicket2100 force-pushed the mullvad-protocol-added branch from 2d79f14 to 86bd059 Compare August 2, 2023 15:31
@snicket2100
Copy link
Contributor Author

Still need to rebase it.

@snicket2100 snicket2100 force-pushed the mullvad-protocol-added branch from 86bd059 to 1078d56 Compare August 2, 2023 15:59
@sonarqubecloud
Copy link

sonarqubecloud bot commented Aug 2, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

No Coverage information No Coverage information
0.0% 0.0% Duplication

@IvanNardi IvanNardi merged commit 1fbe8a2 into ntop:dev Aug 2, 2023
@IvanNardi
Copy link
Collaborator

Thanks!

@snicket2100 snicket2100 deleted the mullvad-protocol-added branch August 3, 2023 07:37
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