-
Notifications
You must be signed in to change notification settings - Fork 1
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
Bot crashes when updating the status #3
Comments
When you run |
You can add it unfortunately still haven't got it working, it just updates status and then get a connection error it's a bit or miss if it even tells me how many players are online.
Sent from Outlook for Android<https://aka.ms/AAb9ysg>
…________________________________
From: Emil Krebs ***@***.***>
Sent: Sunday, May 21, 2023 6:30:19 PM
To: emilkrebs/Nitrado-Bot ***@***.***>
Cc: Conorjames98 ***@***.***>; Author ***@***.***>
Subject: Re: [emilkrebs/Nitrado-Bot] typescript required (Issue #3)
Yeah, forgot to add that in the README.md file. Thanks! Should I add it or do you want to add it yourself?
—
Reply to this email directly, view it on GitHub<#3 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AVDWCGXKYY6SGTWNNYYUMU3XHJGKXANCNFSM6AAAAAAYJQSDOM>.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
And yeah npm I didn't install anything I had to install Discord.js colors. And also had to run --save-dev @types/node-fetch for type declaration. |
When I follow the README everything works fine. What node version are you using and how can I replicate this issue? |
I sadly can't test the bot currently because I don't have any Nitrado service running |
[18:42:24] Bot is Ready. and it continues to do this until it fails to connect I was wondering is there a need for the repeated updating status? |
It repeats to update the status so you can view the number of players playing on the server on the bots status. |
Line 43 in f886ce5
this line controls the updating delay (currently 10s) |
Oh for some reason after three or four attempts it sadly fails and the error is fatal
Sent from Outlook for Android<https://aka.ms/AAb9ysg>
…________________________________
From: Emil Krebs ***@***.***>
Sent: Sunday, May 21, 2023 6:53:11 PM
To: emilkrebs/Nitrado-Bot ***@***.***>
Cc: Conorjames98 ***@***.***>; Author ***@***.***>
Subject: Re: [emilkrebs/Nitrado-Bot] typescript required (Issue #3)
It repeats to update the status so you can view the number of players playing on the server on the bots status.
—
Reply to this email directly, view it on GitHub<#3 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AVDWCGQTYG7XK53U2RM55P3XHJJAPANCNFSM6AAAAAAYJQSDOM>.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
What error message do you get? |
ETIMEDOUT Fetch error
Sent from Outlook for Android<https://aka.ms/AAb9ysg>
…________________________________
From: Emil Krebs ***@***.***>
Sent: Sunday, May 21, 2023 7:01:34 PM
To: emilkrebs/Nitrado-Bot ***@***.***>
Cc: Conorjames98 ***@***.***>; Author ***@***.***>
Subject: Re: [emilkrebs/Nitrado-Bot] typescript required (Issue #3)
What error message do you get?
—
Reply to this email directly, view it on GitHub<#3 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AVDWCGTDDKZVHUQR4OO6RG3XHJJ75ANCNFSM6AAAAAAYJQSDOM>.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
Does it provide any error code like |
Now I'm getting ECONRESET and no there isn't.
Sent from Outlook for Android<https://aka.ms/AAb9ysg>
…________________________________
From: Emil Krebs ***@***.***>
Sent: Sunday, May 21, 2023 7:08:55 PM
To: emilkrebs/Nitrado-Bot ***@***.***>
Cc: Conorjames98 ***@***.***>; Author ***@***.***>
Subject: Re: [emilkrebs/Nitrado-Bot] typescript required (Issue #3)
Does it provide any error code like 504?
—
Reply to this email directly, view it on GitHub<#3 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AVDWCGSTQ2GBYQC2P2GSE43XHJK3PANCNFSM6AAAAAAYJQSDOM>.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
It seems like NitrAPI has some limitations which don't get handled. |
It's annoying to say the least.
Sent from Outlook for Android<https://aka.ms/AAb9ysg>
…________________________________
From: Emil Krebs ***@***.***>
Sent: Sunday, May 21, 2023 7:13:59 PM
To: emilkrebs/Nitrado-Bot ***@***.***>
Cc: Conorjames98 ***@***.***>; Author ***@***.***>
Subject: Re: [emilkrebs/Nitrado-Bot] typescript required (Issue #3)
It seems like NitrAPI has some limitations which don't get handled.
—
Reply to this email directly, view it on GitHub<#3 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AVDWCGREWHK5GJHZ2OS3KEDXHJLOPANCNFSM6AAAAAAYJQSDOM>.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
Yes, maybe turn off the status updates for now. I can't fix it currently but I am open if you have any suggestions for fixing this problem. |
I'll take a look, I've just added error handling so it doesn't terminate the application for now
Sent from Outlook for Android<https://aka.ms/AAb9ysg>
…________________________________
From: Emil Krebs ***@***.***>
Sent: Sunday, May 21, 2023 7:17:51 PM
To: emilkrebs/Nitrado-Bot ***@***.***>
Cc: Conorjames98 ***@***.***>; Author ***@***.***>
Subject: Re: [emilkrebs/Nitrado-Bot] typescript required (Issue #3)
Yes, maybe turn off the status updates for now. I can't fix it currently but I am open if you have any suggestions for fixing this problem.
—
Reply to this email directly, view it on GitHub<#3 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AVDWCGWNP2OLM56NN7SPLGLXHJL47ANCNFSM6AAAAAAYJQSDOM>.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
Thats nice! You could push your changes into a separate branch and create a Pull Request. |
Yeah it's probably an issue with the API I've never had any issues like this with other tokens it's even failing the health check sometimes now.
Sent from Outlook for Android<https://aka.ms/AAb9ysg>
…________________________________
From: Emil Krebs ***@***.***>
Sent: Sunday, May 21, 2023 7:25:34 PM
To: emilkrebs/Nitrado-Bot ***@***.***>
Cc: Conorjames98 ***@***.***>; Author ***@***.***>
Subject: Re: [emilkrebs/Nitrado-Bot] Bot crashes when updating the status (Issue #3)
Thats nice! You could push your changes into a separate branch and create a Pull Request.
—
Reply to this email directly, view it on GitHub<#3 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AVDWCGUMQ3MPGCKNLIAAXOTXHJMZ5ANCNFSM6AAAAAAYJQSDOM>.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
Hit or miss really lol
Sent from Outlook for Android<https://aka.ms/AAb9ysg>
…________________________________
From: Conor Norris ***@***.***>
Sent: Sunday, May 21, 2023 7:29:45 PM
To: emilkrebs/Nitrado-Bot ***@***.***>; emilkrebs/Nitrado-Bot ***@***.***>
Cc: Author ***@***.***>
Subject: Re: [emilkrebs/Nitrado-Bot] Bot crashes when updating the status (Issue #3)
Yeah it's probably an issue with the API I've never had any issues like this with other tokens it's even failing the health check sometimes now.
Sent from Outlook for Android<https://aka.ms/AAb9ysg>
________________________________
From: Emil Krebs ***@***.***>
Sent: Sunday, May 21, 2023 7:25:34 PM
To: emilkrebs/Nitrado-Bot ***@***.***>
Cc: Conorjames98 ***@***.***>; Author ***@***.***>
Subject: Re: [emilkrebs/Nitrado-Bot] Bot crashes when updating the status (Issue #3)
Thats nice! You could push your changes into a separate branch and create a Pull Request.
—
Reply to this email directly, view it on GitHub<#3 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AVDWCGUMQ3MPGCKNLIAAXOTXHJMZ5ANCNFSM6AAAAAAYJQSDOM>.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
But got it working pretty much with error handling and changing the interval of the player status to 10 mins it's not restarted one of my ark servers.
Sent from Outlook for Android<https://aka.ms/AAb9ysg>
…________________________________
From: Conor Norris ***@***.***>
Sent: Sunday, May 21, 2023 7:30:18 PM
To: emilkrebs/Nitrado-Bot ***@***.***>; emilkrebs/Nitrado-Bot ***@***.***>
Cc: Author ***@***.***>
Subject: Re: [emilkrebs/Nitrado-Bot] Bot crashes when updating the status (Issue #3)
Hit or miss really lol
Sent from Outlook for Android<https://aka.ms/AAb9ysg>
________________________________
From: Conor Norris ***@***.***>
Sent: Sunday, May 21, 2023 7:29:45 PM
To: emilkrebs/Nitrado-Bot ***@***.***>; emilkrebs/Nitrado-Bot ***@***.***>
Cc: Author ***@***.***>
Subject: Re: [emilkrebs/Nitrado-Bot] Bot crashes when updating the status (Issue #3)
Yeah it's probably an issue with the API I've never had any issues like this with other tokens it's even failing the health check sometimes now.
Sent from Outlook for Android<https://aka.ms/AAb9ysg>
________________________________
From: Emil Krebs ***@***.***>
Sent: Sunday, May 21, 2023 7:25:34 PM
To: emilkrebs/Nitrado-Bot ***@***.***>
Cc: Conorjames98 ***@***.***>; Author ***@***.***>
Subject: Re: [emilkrebs/Nitrado-Bot] Bot crashes when updating the status (Issue #3)
Thats nice! You could push your changes into a separate branch and create a Pull Request.
—
Reply to this email directly, view it on GitHub<#3 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AVDWCGUMQ3MPGCKNLIAAXOTXHJMZ5ANCNFSM6AAAAAAYJQSDOM>.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
if anyone else has the same issue when the api fails to connect every 10 seconds change line 43 to setTimeout(scheduleAsync, 200000); this will only update players every 10 minutes. (alternatively add error handling so the bot doesn't terminate the program.) it's important to note that my connection might not be perfect and this could be an issue unique to me but nonetheless the solution worked for me. |
Be sure to run "npm install -g typescript" in the root
The text was updated successfully, but these errors were encountered: