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

Party ID pool runs out of IDs over time #110

Open
alborrajo opened this issue Dec 17, 2022 · 3 comments
Open

Party ID pool runs out of IDs over time #110

alborrajo opened this issue Dec 17, 2022 · 3 comments
Labels
bug Something isn't working

Comments

@alborrajo
Copy link
Collaborator

Describe the bug
There may be some unhandled case in which a party disbands without the server knowing. The ID pool runs out and no more players can join the game anymore.

To Reproduce
Run the server and have players join and leave over time. It took about 3 months for this issue to happen.

Expected behavior
IDs should return to the pool in all cases the party no longer exists.

@alborrajo alborrajo added the bug Something isn't working label Dec 17, 2022
@alborrajo
Copy link
Collaborator Author

None of the logs over these months have shown the "Failed to remove partyId" error message.

@alborrajo
Copy link
Collaborator Author

A workaround has been pushed with PR #177 that prevents this issue from requiring a server restart for new players to join

@pacampbell
Copy link
Collaborator

pacampbell commented Jun 1, 2024

I attempted to address this in my recent quest PR. I found there was an issue which happened where a player was in one party and then was invited to another party. This would fail, then when the client asked to create a solo party for the player this would also fail because they were already invited to another party. The client would then panic and keep requesting to make a party over and over, causing the party IDs to get exhausted. This would happen until the player logs out.

I am not sure if there exists more issues like this, but this was probably at least one reason for this bug which is hopefully fixed now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants