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 support for PubSub #20

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Add support for PubSub #20

wants to merge 3 commits into from

Conversation

Dunedan
Copy link
Collaborator

@Dunedan Dunedan commented Apr 22, 2023

This adds support for publishing game lists, rating lists and the leader board via PubSub. With matching support in pyrogenesis this will improve the performance and efficiency of the lobby, as the bots then don't have to manually send updates to each connected client, but the clients subscribe to the PubSub nodes and the XMPP server takes care of deliverying updates instead.

The previous behavior of sending stanzas with updates to every connected client is still available, but can be disabled with the --disable-legacy-lists command line flag. The performance benefits of PubSub do only materialize when the legacy behavior is disabled.

This adds support for publishing game lists, rating lists and the
leader board via PubSub. With matching support in pyrogenesis this will
improve the performance and efficiency of the lobby, as the bots then
don't have to manually send updates to each connected client, but the
clients subscribe to the PubSub nodes and the XMPP server takes care of
deliverying updates instead.

The previous behavior of sending stanzas with updates to every connected
client is still available, but can be disabled with the
`--disable-legacy-lists` command line flag. The performance benefits
of PubSub do only materialize when the legacy behavior is disabled.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant