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

Steam: /server-status #30

Open
Crabbey opened this issue Apr 16, 2018 · 6 comments
Open

Steam: /server-status #30

Crabbey opened this issue Apr 16, 2018 · 6 comments

Comments

@Crabbey
Copy link
Contributor

Crabbey commented Apr 16, 2018

For steam, /server-status shouldn't be cached.
Contains load information about that CDN server, and may result in lower performance if it remains cached.
example content:
"status" { "csid" "3" "load" "22" "cell" "31" }

@mohsinskynet
Copy link

where to find nginx file to change the CACHE_MEM_SIZE, CACHE_DISK_SIZE, CACHE_MAX_AGE
and how to run Game Content Cache Hostnames please post a guide or tell here ty

@ScholliYT
Copy link
Contributor

ScholliYT commented Aug 3, 2018

@mohsinskynet I think you can edit these in the dockerfile. Or by just passing the env variables to the docker run command. Have a look at the readme

@mskafi
Copy link

mskafi commented Sep 2, 2018

@Crabbey I went down this line too, and changed the generic docker container's source code for nginx to avoid caching it. You might also want to avoid caching /server-list (or close, not sure of the exact URL.) I believe that is why you should actually opt for the steam specific docker image, since it only caches url's under the /depot url, which seems to be independent from the host.

@unspec
Copy link
Member

unspec commented Feb 23, 2020

Would be worth re-investigating this as a result of lancachenet/monolithic#85

@VibroAxe
Copy link
Member

@Lepidopterist what happened to #111 why did we decide to not merge that?

@Lepidopterist
Copy link
Member

Honestly, no idea.

Looks like it wont merge cleanly as that branch is polluted. Can just re-branch, merge and commit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants