Hercules
Hercules copied to clipboard
HerculesWS
Fix server sync reading and wrong disconnection from API Server
CAREFUL! UNTESTED CODE AHEAD
I have not done any thoroughly testing/validation of this code. I am opening this as a draft in case people who are experiencing the issue wants to check / try. I can not guarantee this won't cause any unexpected side effects at this time.
Pull Request Prelude
- [x] I have followed proper Hercules code styling.
- [x] I have read and understood the contribution guidelines before making this PR.
- [x] I am aware that this PR may be closed if the above-mentioned criteria are not fulfilled.
Changes Proposed
This fixes how packet 0x272D (CharServer -> LoginServer online accounts sync) is read by Login Server. Thus, preventing accounts from being wrongly marked as offline.
When login-server marks an account as offline, it will eventually tell API Server to disconnect them, thus also invalidating users' API Tokens while they are in game.
Huge thanks to AcidMarco in Herc's discord that provided a quite good explanation of what he has observed and this helped me a lot in finding the root cause (or at least I think it did...)
I am still having a hard time believing this is the fix, because this code has been untouched for almost 10 years, but I will perform a deeper investigation when I have some time (and before making this PR ready to review)
Issues addressed: None, I think.
Probably unrelated to this PR, but there's one more issue wherein API server is called for autotraders on restart, and we get bunch of Account not online errors
Probably unrelated to this PR, but there's one more issue wherein API server is called for autotraders on restart, and we get bunch of Account not online errors
Yeah, I think this is a separate issue. I will take a look into it and open a new PR if I find something (maybe we should create an issue about it?)
Hello, I applied this on my server and it fixes some of recent problem but after few hours of server being online the character-login gets disconnected.
I don't know how it happen or reproduce its just gets disconnected. Then i had to restart the Login and Character server to be able to login again.
@dastgirp , @guilherme-gm any updates?
@vBrenth can you try using the first commit only? this one: https://github.com/HerculesWS/Hercules/pull/3304/commits/29d858c70589fdb8a3279d930103a43b8cc7d6c7
I am wondering if I may have made a mistake when converting to struct.
@guilherme-gm I've been running the first commit only and its still having some saving problem but less and no disconnection between char and login so far.
@guilherme-gm I've been running the first commit only and its still having some saving problem but less and no disconnection between char and login so far.
Thanks for the update, I made a fix on #3312 so the 2nd commit properly works.
Are the saving problems giving any logs? is there any reproduction steps for it? I think we need to figure out whether this is a server issue or a client one, since I noticed that there are some cases where the client doesn't send the changes to be saved
I cant find a way to reproduce it. Its just happen after some time with players are online.