Skip to content

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