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

bug: server crashes when logging out after changing the maximum spell slot #2122

Closed
1 task done
dhmello opened this issue Nov 29, 2023 · 6 comments · Fixed by #2426
Closed
1 task done

bug: server crashes when logging out after changing the maximum spell slot #2122

dhmello opened this issue Nov 29, 2023 · 6 comments · Fixed by #2426
Labels
bug Something isn't working needs verification Pending confirmation that the bug exists by another user.

Comments

@dhmello
Copy link
Contributor

dhmello commented Nov 29, 2023

Description

I increased the maximum number of spells that the player can have on my server, now when entering the character's screen the equipment does not load and also when the player logs out, the server crashes.

Steps to Reproduce

  1. Have an old database, where the maximum spells in the server settings are the default value.
    2 - Change the maximum spells to 100 (in my case it was this value)
    3 - Log in with the existing character, notice that he will appear without clothes (paperdoll), but when logged in he will render the clothes normally.
    4 - disconnect the character and watch the bug crash the server.

Version with bug

0.8.0.0

Last version that worked well

Unknown

Affected platforms

Windows

Did you find any workaround?

errors-Intersect Server.log

Relevant log output

No response

Duplicate Bug Check

  • This bug report is not a duplicate to the best of my knowledge.
@dhmello dhmello added bug Something isn't working needs verification Pending confirmation that the bug exists by another user. labels Nov 29, 2023
@dhmello
Copy link
Contributor Author

dhmello commented Nov 29, 2023

Update: If i back to 35 spells slots, the bug dont happen

@Thomasfds
Copy link

Thomasfds commented Jan 24, 2024

Encounter this issue too.

Server can crash or client can disconnect user, my version is 0.8.0-beta.127

Intersect Server-2024_01_24-14_09_35_896.log
errors-Intersect Server.log

@cydyn
Copy link

cydyn commented Jan 26, 2024

@Thomasfds However, there is no problem for me. You need to change Initialbankslots in the config, not MaxSlots.
These are two different things

@cydyn
Copy link

cydyn commented Jan 26, 2024

@theceltos At the moment, this error can be solved in one way:

  1. Start the server.
  2. How to create playerdata.db files, change maxspells in the config to higher than 35
  3. Turn off the server.
  4. Delete playerdata.db
  5. Start the server to recreate playerdata.db, thanks to which the database will load more slots for the player and the concurrency error will not appear

Apparently, after creating playerdata.db and changing maxspells, new slots are not added to the database, which causes an error and a server crash.

It is best to make more slots and if you think about changes later, you can reduce them, because you can reduce the slots without any error, but currently you cannot increase them.

@dhmello
Copy link
Contributor Author

dhmello commented Jan 26, 2024

OK regarding the method to avoid bugs, but the bug will not cease to exist, what will we do with a project where there are several accounts, characters, items in the player database?
Ahh it bugged, am I going to delete it and do it again?

I believe this is a BUG, and should not be resolved as a workaround.

If it is not resolved, some other project will face the same problem down the road. We're just hiding the elephant in a room where no one can see...

@cydyn
Copy link

cydyn commented Jan 26, 2024

Yes, this is just a workaround, but I think that the number of available spells is actually set at the beginning, before we start the server, even for visual reasons. We make a menu and we probably don't increase it

This is a bug that needs to be fixed, but you can set the value to 100 spells and only use 50

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs verification Pending confirmation that the bug exists by another user.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants