Skip to content
This repository has been archived by the owner on Dec 17, 2022. It is now read-only.

Implement summon request protocol #2

Open
vswarte opened this issue Aug 28, 2022 · 1 comment
Open

Implement summon request protocol #2

vswarte opened this issue Aug 28, 2022 · 1 comment
Assignees

Comments

@vswarte
Copy link
Owner

vswarte commented Aug 28, 2022

Rework the spawn procedure a bit:

  • In case of a client we should request the host to spawn a summon for us.
  • In case of a host we should listen for these requests and make the decision on whether or not this is a valid request.
  • In case of a host we should spawn the summons and broadcast SummonSpawned to all clients.
  • In case of a host we should never handle SummonSpawned messages as these are meant to inform clients of a new summon.
  • In case of a client we should never handle SummonSpawned from any other lobby member than the host.
@vswarte vswarte self-assigned this Aug 28, 2022
@vswarte
Copy link
Owner Author

vswarte commented Sep 5, 2022

Most of the work here has been done except for some anti-abuse items. I still need to:

  • Implement role validation for incoming messages (host/client validation)
  • Gracefully handle incorrect statemachine transitions (just drop them? stop handling messages for users that seem abusive?)

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

No branches or pull requests

1 participant