Skip to content
This repository has been archived by the owner on Mar 31, 2021. It is now read-only.

Update servers.js #443

Closed
wants to merge 1 commit into from
Closed

Update servers.js #443

wants to merge 1 commit into from

Conversation

muleyo
Copy link

@muleyo muleyo commented Sep 29, 2018

New Steam servers.

@muleyo muleyo mentioned this pull request Oct 5, 2018
@seishun
Copy link
Owner

seishun commented Oct 8, 2018

Where is this list from?

@muleyo
Copy link
Author

muleyo commented Oct 8, 2018

From the official API.

@muleyo
Copy link
Author

muleyo commented Oct 8, 2018

@seishun
Copy link
Owner

seishun commented Oct 9, 2018

SteamKit2 is considering reintroducing a bootstrapped list (SteamRE/SteamKit#597 (comment)). If they do, I'll use their list. Otherwise I think I'll get rid of servers.js and use cm0.steampowered.com (which works most of the time) by default and try to make it clearer that one should really handle the 'servers' event (as if the code in example.js weren't enough).

@divslinger divslinger mentioned this pull request Oct 17, 2018
@seishun
Copy link
Owner

seishun commented Dec 1, 2018

Thank you for your contribution. I chose a different approach in 8f0f68b. Instead of hard-coding a server list, it is downloaded automatically when installing from git and is (will be) bundled only in the npm package.

@seishun seishun closed this Dec 1, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants