Skip to content

Ship default tunnel_cache for users blocked from the master list#902

Open
11EJDE11 wants to merge 2 commits into
CnCNet:developfrom
11EJDE11:add-tunnelcache-file
Open

Ship default tunnel_cache for users blocked from the master list#902
11EJDE11 wants to merge 2 commits into
CnCNet:developfrom
11EJDE11:add-tunnelcache-file

Conversation

@11EJDE11
Copy link
Copy Markdown
Member

@11EJDE11 11EJDE11 commented May 4, 2026

Summary

  • Ship package/Client/tunnel_cache (seeded from the current master list) so users who can't reach https://cncnet.org/master-list (e.g. Cloudflare-blocked regions like Russia) have a working tunnel list out of the box.
  • Add Client/ to versionconfig.ini [Include] so the updater tracks and refreshes the seed file on every package update.
  • Adjust .gitignore to keep ignoring local Client/ state while tracking only tunnel_cache.

Why

The client (TunnelHandler.cs) already falls back to Client/tunnel_cache when the master list URL fails. Until now the only way to deliver that file to affected users was DM'ing it on Discord, which goes stale quickly. Bundling it via the package means blocked users get a refreshed snapshot on each release.

For unblocked users this is a no-op: the file is overwritten on the first successful master list fetch.

Maintenance

Refresh the seed file shortly before each release:
curl https://cncnet.org/master-list -o package/Client/tunnel_cache

11EJDE11 and others added 2 commits May 4, 2026 16:12
@11EJDE11 11EJDE11 requested a review from SadPencil May 4, 2026 09:08
Copy link
Copy Markdown
Member

@SadPencil SadPencil left a comment

Choose a reason for hiding this comment

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

LGTM. Is there a 2nd review?

@brichardson1991 brichardson1991 marked this pull request as ready for review May 4, 2026 11:18
@SadPencil
Copy link
Copy Markdown
Member

@11EJDE11 This PR was a draft somehow. Is it ready for merge?

@11EJDE11
Copy link
Copy Markdown
Member Author

11EJDE11 commented May 4, 2026

@11EJDE11 This PR was a draft somehow. Is it ready for merge?

Yeah, I haven't tested it and don't want to mess up the updater if it's wrong. Can we put it in Pre Release?

@SadPencil
Copy link
Copy Markdown
Member

@11EJDE11 This PR was a draft somehow. Is it ready for merge?

Yeah, I haven't tested it and don't want to mess up the updater if it's wrong. Can we put it in Pre Release?

yeah. ideally this should be named canary instead of pre-release

alternatively any branch starts with feature/ can also be automatically pushed to update server

@11EJDE11
Copy link
Copy Markdown
Member Author

I've pushed to pre-release, @SadPencil. What happens now?

@SadPencil
Copy link
Copy Markdown
Member

I've pushed to pre-release, @SadPencil. What happens now?

You can then ask some Russian players to test by switching to pre-release channel in client option window

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.

3 participants