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

Add support for mobile games: PUBG Mobile & Mobile Legends #205

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

mezzovide
Copy link

What CDN does this PR relate to

Add support for mobile games: PUBG Mobile & Mobile Legends

Does this require running via sniproxy

no

Testing Scenario

Live testing for network with 700+ clients for a week, zero case of complaint so far

Testing Configuration

<!-- Paste either your docker run command from the DNS container OR explain how you have setup DNS zone files etc to test this issue -->

Sniproxy output

Please paste the output from docker logs <sniproxy container name/id> | sed 's/.*\:443 \[//;s/\].*//' | sort | uniq -c below

<!-- If you are running sniproxy paste the output to the following command
docker logs <sniproxy container name/id> | sed 's/.*\:443 \[//;s/\].*//' | sort | uniq -c
-->

@MathewBurnett
Copy link
Member

i can believe that the two *.ml.youngjoygame.com are mobile legend specific but *.gjacky.com seems quite wide. I suspect there is a more specific domain for only pubg. As an example i found reference to this pubgmobile.com.cdn.ettdnsv.com elsewhere.

@mezzovide
Copy link
Author

i agree, let me recheck on that. I can be sure of k.gjacky.com being one of them since its the one i see, but after stumble upon this reference i decided to include wildcard. Maybe being conservative is better in this case.

revert wildcard and be conservative with only adding domain that can be confirmed seen and successfully cached
@mezzovide mezzovide marked this pull request as ready for review September 5, 2022 10:23
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