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

add master and content server address to openttd.cfg #2914

Closed
DorpsGek opened this issue May 24, 2009 · 4 comments
Closed

add master and content server address to openttd.cfg #2914

DorpsGek opened this issue May 24, 2009 · 4 comments
Labels
flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) patch from FlySpray This issue is in fact a Patch, but imported from FlySrpay

Comments

@DorpsGek
Copy link
Member

yorick opened the ticket and wrote:

I made a patch that adds a master_server_host, master_server_port, content_server_host and content_server_port setting to openttd.cfg.
This way, users can configure what master server to use, instead of having to patch openttd.

Patch against 22edbd05c877(svn r16409).

Attachments

Reported version: trunk
Operating system: All


This issue was imported from FlySpray: https://bugs.openttd.org/task/2914
@DorpsGek
Copy link
Member Author

DorpsGek commented Sep 7, 2009

Rubidium wrote:

The major problem I see with this patch is that it adds yet another point-of-failure for the 'noobs' to get their servers registered. So I rather not see this as a setting.

Furthermore there is no real use for another master server/content server as it only duplicates stuff and splits the community, but if you really want to you can always instruct people to update their hosts file to point master.openttd.org to whatever IP you like.


This comment was imported from FlySpray: https://bugs.openttd.org/task/2914#comment6602

@DorpsGek
Copy link
Member Author

muxy wrote:

I think it's up to the server admin to decide to which master server he wants his openttd server to be registered. Instead of changing the master server, give the server a list of master server where the openttd server can be registered. The client side should also be able to set up a list of master server to request openttd servers list and remove duplicates entry (unique_id?) (it means that 1 openttd server can register itself to many master servers)

To avoid spliting community, add a function to the master server to register secondary master server. The master server will provide to the client a list of master server where he will be able to have a openttd server list.

This will permit local (national - French, Sweedish, Oz, or functionnal - OpenttdCoop )communities to be created. Then they can choose if they want to be linked to THE primary community (openttd.org) or not. Usefull to publish openttd servers to the local community only and to avoid people outside the community.


This comment was imported from FlySpray: https://bugs.openttd.org/task/2914#comment6630

@DorpsGek
Copy link
Member Author

Rubidium wrote:

So you want to add an extra layer of indirection (failure) to finding servers? First get a list of "master" servers from the "master master" server so the player can choose which of the "master" servers to search on? That only makes things more complicated for users.

Also I don't see the need to split communities into their own unique islands; most communities thrive on the fact that they can be found via the master server, i.e. it's a way they get into contact with them.


This comment was imported from FlySpray: https://bugs.openttd.org/task/2914#comment7955

@DorpsGek
Copy link
Member Author

Rubidium closed the ticket.

Reason for closing: Won't implement


This comment was imported from FlySpray: https://bugs.openttd.org/task/2914

@DorpsGek DorpsGek added Core flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) patch from FlySpray This issue is in fact a Patch, but imported from FlySrpay labels Apr 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) patch from FlySpray This issue is in fact a Patch, but imported from FlySrpay
Projects
None yet
Development

No branches or pull requests

1 participant