Skip to content

Stream destroy#87

Open
joshstubble wants to merge 2 commits intoatomheartother:masterfrom
joshstubble:Stream-Destroy
Open

Stream destroy#87
joshstubble wants to merge 2 commits intoatomheartother:masterfrom
joshstubble:Stream-Destroy

Conversation

@joshstubble
Copy link
Copy Markdown

I have changed a few lines to cause the stream to shut itself down since I get stuck in a exit loop on errors with current code. More of a hack than an upgrade, but have been using it for about 4 months and has been working great. Only downtime has been affiliated with ISP or server errors. Bot is running amazing.

I have changed a few lines to cause the stream to shut itself down since I get stuck in a exit loop on errors with current code. More of a hack than an upgrade.
Causes bot to continuously try and reboot itself on a `process.exit()` error.
@atomheartother
Copy link
Copy Markdown
Owner

atomheartother commented Dec 13, 2020

Huh... I'm not sure how much I like this fix, especially since production QTweet hasn't ever been stuck on a destroy loop... But I have seen users report issues with this. Tell you what, I'll leave this PR up and wait for feat/typescript (#86) to be merged, then I'll give this a shot and ask the members who are running into stream destruction errors to try it out.

@joshstubble
Copy link
Copy Markdown
Author

Sounds good. Figured it wasn't the greatest fix, but it has worked quite well for me so thought i would pass it along 😀

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