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 an API to return load on each server i.e. number of local sessions. #34

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

danesh-d
Copy link

The main intention of this patch was to make it possible to get the local load of each instance of Openfire in a clustered environment so the load balancer would know where to route the traffic.

There might be better solutions but this is our way to solve the problem so far. Hope this patch is useful and will be accepted.

@Redor
Copy link
Contributor

Redor commented Jan 27, 2020

Sry for a super late response. Thy for the PR.
I think it is a better way, to create an own "Entity" class instead of using SessionEntity for that.
Otherwise we have the "load" parameter also by getting the Session Information.
And maybe we should consider to use another name, like "clients", instead of load.

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.

2 participants