Opened 9 years ago

Last modified 3 years ago

#3556 new enhancement

[PATCH] Dedicated server — at Version 17

Reported by: elexis Owned by:
Priority: Nice to Have Milestone: Backlog
Component: Network Keywords: patch, beta
Cc: andy011973@…, Victor ADASCALITEI Patch:

Description (last modified by Victor ADASCALITEI)

Requirements

  • A dedicated host is a gameserver running from command-line
  • It has almost no performance requirements
    • no graphics are displayed
    • no local player/observer joins the game
  • could allow Wildfire Games to host multiple games on a single machine
    • can be patched independently from the release cycle (f.e. in case of bugs or abuse)
    • might satisfy the demand for servers (as only a minority of users can host/configure their router currently)
    • useful for rated games as we could make sure to not quit the server prematurely
    • ideally ensure not to kill all games when restarting the lobbybot

Notice that the source code is freely available, which means everyone will be able to use dedicated hosts. It will be the job of the programmers and lobby moderators to prevent and stop abuse.


Roadmap:

1. Implement & commit basic functions * Only one host per instance * Use chat commands instead of gui (similar to scythewirlers autohost implementation #3419) * No client-side changes required (bugs can't affect regular hosting/joining)

(Skipped to avoid adding code which would have to be removed later)

  1. Implement gamesetup UI changes
    • Allow either the first or everyone to control the gamesettings (#3806)
    • If there are serious bugs, we could revert to 1
  1. Implement hosting on different ports (done in #3575)
    • Requirement for Wildfire Games being able to host multiple games simultaneously on a single machine
    • To prevent abuse (in case of attacks on users IP-addresses), hosting games by unauthorized users could be prohibited.
    • Optional: Trusted hosts (authorized lobby accounts) could be able to advertize more than one game at a time.
  1. Lobby changes
    • New button "Wildfire-Games Host"
      • XPartaMupp will spawn (at most) 1 gameserver per user
      • The user will join this game and become a moderator of it

Change History (20)

by elexis, 8 years ago

Proof of concept. Server running and chat working. Still opens an unused window and has no features besides chat yet.

by elexis, 8 years ago

No more useless windows, only command line. Takes a fraction of a second to start. Changed to class structure.

comment:1 by elexis, 8 years ago

Description: modified (diff)

comment:2 by elexis, 8 years ago

Description: modified (diff)

in reply to:  2 comment:3 by Vladislav Belov, 8 years ago

Current patch still creates the graphic context, so app could be crashed when runs on the server w/o a graphic card. The rest of the patch looks good.

comment:4 by elexis, 8 years ago

Description: modified (diff)

comment:5 by elexis, 8 years ago

Component: Multiplayer lobbyNetwork

(set component to network)

comment:6 by elexis, 8 years ago

In 18322:

Major network cleanup. Patch by Imarok.

Access the server from the client only, not from the GUI (except for autostarted games).
Thereby lay the foundation for clients to setup the game (refs #3806) and dedicated hosting (refs #3556).
Doesn't transfer nor remove the SetTurnLength showcase from r7936.

comment:7 by Andy Alt, 8 years ago

Cc: andy011973@… added

comment:8 by Imarok, 8 years ago

Description: modified (diff)

comment:9 by elexis, 8 years ago

Keywords: patch added
Summary: Dedicated server[PATCH] Dedicated server

Update:

Besides the new controller authorization mechanism #3806, removing DedicatedServer_Gamesetup.cpp implies:

Problem: lobby stanzas

  • The server should send the stanza (so that it doesn't depend on some client registering the game which might disconnect and thus unregister the game)
  • but only the client has access to the simulation data (defeated state #3476) and the server should avoid any gameattribute-parsing logic ideally
  • Edge cases empty host: If the server doesn't construct the packets on its own, but relays a copy sent by a client: which packets should be sent if no client connected yet? what stanza to send if the game is running and all clients disconnected (the "player X is offline" state would be outdated with the last disconnect)

Proposed solution:

  • So the server has to construct an empty packet from scratch when starting a lobbied game. If the last client disconnected from a lobbied game, the server has to correct the playerlist entry. Therefore it would make sense if the server can produce the entire logic on its own.
  • Instead of requesting a lobby stanza from a client, the server could just request the player-defeated-state. But introducing a packet for either of these things seems very undesirable. It is probably better to just not send the defeated state if the game is hosted by a dedicated server. (There might be a dedicated server that also simulates the game optionally, for example to determine the winner of rated games. If that were enabled, it could also report the defeated state without asking a client).
  • The NetServerknows the current player assignments, game attributes and has a list of connected GUIDs / clients, thus could send these 3 arguments to a JS function that does the complex stanza construction in r18534. The affected functions would have to be moved to source/lobby/ if we want to distribute the dedicated server without requiring public/.

comment:10 by scythetwirler, 7 years ago

Keywords: beta added

comment:11 by scythetwirler, 7 years ago

Milestone: BacklogAlpha 22

comment:12 by Andy Alt, 7 years ago

Owner: set to Andy Alt
Status: newassigned

comment:13 by Andy Alt, 7 years ago

Status: assignednew

comment:14 by Andy Alt, 7 years ago

Owner: Andy Alt removed

comment:15 by Imarok, 7 years ago

refs #4364 a comment from scythetwirler at this ticket, that is probably better suited here: "If you could set the XMPP resource on the JID of the default dedicated server to something different (0ad is the default for the main client) for easy recognition, that'd be great. :)"

Last edited 7 years ago by Imarok (previous) (diff)

comment:16 by elexis, 7 years ago

Milestone: Alpha 22Backlog

Backlogging due to lack of progress

comment:17 by Victor ADASCALITEI, 6 years ago

Cc: Victor ADASCALITEI added
Description: modified (diff)

I'm interested in this.

Note: See TracTickets for help on using tickets.