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

Adding new applications based on vendors docs: pluralsight #1501

Closed
elico opened this issue Mar 27, 2022 · 5 comments · Fixed by #1538
Closed

Adding new applications based on vendors docs: pluralsight #1501

elico opened this issue Mar 27, 2022 · 5 comments · Fixed by #1538
Labels

Comments

@elico
Copy link

elico commented Mar 27, 2022

I have seen couple uses of applications ID in production and nDPI is missing many of these.
One example is pluralsight SNI and IP ranges.
It's pretty simple to define this app based on the official docs at:
https://help.pluralsight.com/help/ip-allowlist

Basically any SNI and domain under the definition of:
*.pluralsight.com
ip-video-course-exercise-files-us-west-2.s3.us-west-2.amazonaws.com

is theirs and also their CDN domains are:
vid.pluralsight.com
vid5.pluralsight.com
vid20.pluralsight.com
vid21.pluralsight.com
vid30.pluralsight.com

They have a test page at:
https://app.pluralsight.com/video/test

I don't know who is responsible or what is the process of adding a new app but many can be added easily based on the vendors docs.

Thanks,
Eliezer

@elico elico added the question label Mar 27, 2022
@elico elico changed the title Adding new applications based on vendors docs Adding new applications based on vendors docs: pluralsight Mar 27, 2022
@IvanNardi
Copy link
Collaborator

I have seen couple uses of applications ID in production and nDPI is missing many of these.

Could you share the details or, better, push some PRs?

I added a basic support for Pluralsight in #1503.
If you have a pluralsight account, could you share a pcap with a real video stream in it, please?

@elico
Copy link
Author

elico commented Mar 27, 2022

@IvanNardi I will try to capture some traffic in the next couple days.
I cannot push a PR since I am really busy working on other things but I can try to provide a capture and other relevant details.
The next is the access.log from squid:
https://www.ngtech.co.il/squid/pluralsight-access-log.txt

@IvanNardi
Copy link
Collaborator

@elico, any updates?

@elico
Copy link
Author

elico commented Apr 25, 2022 via email

@elico
Copy link
Author

elico commented Apr 26, 2022

@IvanNardi The next should be enough to know what is needed.
There is a bit local traffic noise but I assume you will be able to filter the relevant packets.

https://www.ngtech.co.il/static/pluralsight_app/pluralsight_client_hello.pcapng
https://www.ngtech.co.il/static/pluralsight_app/pluralsight_dns.pcapng

utoni added a commit to utoni/nDPI that referenced this issue May 3, 2022
@utoni utoni linked a pull request May 3, 2022 that will close this issue
IvanNardi pushed a commit that referenced this issue May 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants