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

Only use lancache.steamcontent.com for steam #124

Merged
merged 2 commits into from
Jan 14, 2021
Merged

Conversation

VibroAxe
Copy link
Member

What CDN does this PR relate to

Remove all non lancache domains from steam. This PR requires lancachenet/monolithic#87 to be implemented before merging (and perhaps some time for people to adopt as well!)

Does this require running via sniproxy

No

Capture method

Steam change logs and direct discussions with valve

Testing Scenario

lancachenet/monolithic#87

Testing Configuration

lancachenet/monolithic#87 (comment)

Sniproxy output

N/A

@VibroAxe
Copy link
Member Author

Added WIP tag to prevent merging until lancachenet/monolithic#87 is merged

@unspec
Copy link
Member

unspec commented Feb 13, 2020

@VibroAxe - lancachenet/monolithic#87 has been merged. Worth waiting for the next batch of LANs next week to further test the new steam approach or happy to take this one out of WIP now?

@unspec unspec added the wip label Feb 13, 2020
@unspec unspec mentioned this pull request Feb 13, 2020
@VibroAxe
Copy link
Member Author

I think we should leave this be for the moment, without the ua caching this merge will break the cache for anyone not running latest. I'm thinking give it 2-3 months then merge in?

@Lochnair
Copy link

Ran into an issue where creating workshop collections won't work because it tries to reach https://ufs0101.steamcontent.com:8699/ugcupload. That obviously can't work when one only have a SNI proxy on port 443.

I assume Valve expects that anything else than lancache.steamcontent.com aren't messed with now. So I ask that you consider merging this PR again to allieviate issues like this.

@VibroAxe
Copy link
Member Author

@Lepidopterist @mintopia @MathewBurnett anyone have any thoughts on merging this?

@mintopia
Copy link
Collaborator

I think we should try merging it and keep an eye on support requests and feedback. It should be OK.

@Lepidopterist
Copy link
Collaborator

Lepidopterist commented Jan 14, 2021 via email

@VibroAxe VibroAxe changed the title WIP: Only use lancache.steamcontent.com for steam Only use lancache.steamcontent.com for steam Jan 14, 2021
@VibroAxe VibroAxe merged commit 3f712d7 into uklans:master Jan 14, 2021
@miketweaver
Copy link

You update http://diagnostics.lancache.net/ to reflect this change.

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 this pull request may close these issues.

6 participants