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

Stream destroy #87

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

joshstubble
Copy link

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
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
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