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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

[The reconnect bug is maybe fixed] Queue stat panel commands, ideally fixing another source of the reconnect bug #59228

Merged
merged 2 commits into from May 22, 2021

Conversation

Mothblocks
Copy link
Member

About The Pull Request

Fixes #59226, ideally.

It's hard to test this, but this queues any ?command until init_verbs, in which case we have confirmed to be connected. The only one it does not is the one in onload, as that didn't seem to appear in my logs and I suspect is only called once we're connected to the server, which helps.

Changelog

馃啈
fix: Fixed another source to the bug preventing reconnect.
/:cl:

@tgstation-server tgstation-server added the Fix Rewrites a bug so it appears in different circumstances label May 21, 2021
@Mothblocks Mothblocks added the Test Merge Candidate You're our unpaid test team label May 21, 2021
@Mothblocks
Copy link
Member Author

TM'd on Terry.

@Mothblocks
Copy link
Member Author

It seems to have completely fixed the reconnect bug for me on Terry.

@Mothblocks Mothblocks changed the title Queue stat panel commands, ideally fixing another source of the reconnect bug [The reconnect bug is fixed] Queue stat panel commands, ideally fixing another source of the reconnect bug May 21, 2021
@tgstation-server tgstation-server removed the Test Merge Candidate You're our unpaid test team label May 21, 2021
@ghost
Copy link

ghost commented May 21, 2021

ive heard this one before....

@Mothblocks Mothblocks changed the title [The reconnect bug is fixed] Queue stat panel commands, ideally fixing another source of the reconnect bug [The reconnect bug is maybe fixed] Queue stat panel commands, ideally fixing another source of the reconnect bug May 21, 2021
@Mothblocks
Copy link
Member Author

my confidence has waned

@Mothblocks Mothblocks marked this pull request as draft May 21, 2021 23:19
@Mothblocks Mothblocks marked this pull request as ready for review May 22, 2021 00:36
@Mothblocks
Copy link
Member Author

Someone with the bug confirmed to me it worked fine after they cleared cache.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix Rewrites a bug so it appears in different circumstances
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Send-Tabs is blocking reconnect
3 participants