Enhanced Sabnzbd - API timeout and always display stats #62

Closed
opened 2026-02-20 10:23:22 -05:00 by deekerman · 2 comments
Owner

Originally created by @MikeSterry on GitHub (Feb 14, 2018).

When configuring the API, I wouldn't receive a status result on test.

This was my fault, for not adding a trailing slash. That said, would it be possible to add 'read_timeout' to the request? I think the test just hung on my end, so it appeared to not be working.

Also, could we remove the if statement that prevents stats from displaying all the time? I would love to always see the stats.

From:
if($size > 0 || $rate > 0) { $output = ' <ul class="livestats"> <li><span class="title">Queue</span><strong>'.$queue_size.'</strong></li> <li><span class="title">Speed</span><strong>'.$current_speed.'/s</span></strong></li> </ul> '; }

To:
$output = ' <ul class="livestats"> <li><span class="title">Queue</span><strong>'.$queue_size.'</strong></li> <li><span class="title">Speed</span><strong>'.$current_speed.'/s</span></strong></li> </ul> ';

Originally created by @MikeSterry on GitHub (Feb 14, 2018). When configuring the API, I wouldn't receive a status result on test. This was my fault, for not adding a trailing slash. That said, would it be possible to add 'read_timeout' to the request? I think the test just hung on my end, so it appeared to not be working. Also, could we remove the if statement that prevents stats from displaying all the time? I would love to always see the stats. From: ` if($size > 0 || $rate > 0) { $output = ' <ul class="livestats"> <li><span class="title">Queue</span><strong>'.$queue_size.'</strong></li> <li><span class="title">Speed</span><strong>'.$current_speed.'/s</span></strong></li> </ul> '; }` To: ` $output = ' <ul class="livestats"> <li><span class="title">Queue</span><strong>'.$queue_size.'</strong></li> <li><span class="title">Speed</span><strong>'.$current_speed.'/s</span></strong></li> </ul> ';`
Author
Owner

@MikeSterry commented on GitHub (Feb 14, 2018):

The code block removed spaces, so it's a bit tough to read. Hopefully, it still makes sense to everyone.

@MikeSterry commented on GitHub (Feb 14, 2018): The code block removed spaces, so it's a bit tough to read. Hopefully, it still makes sense to everyone.
Author
Owner

@KodeStar commented on GitHub (Feb 14, 2018):

The master version has a fix in place for the trailing slash thing. I'll look into adding a timeout.

I'll look at adding an option as to whether to display stats all the time or only when something is happening.

Thanks for the feedback.

@KodeStar commented on GitHub (Feb 14, 2018): The master version has a fix in place for the trailing slash thing. I'll look into adding a timeout. I'll look at adding an option as to whether to display stats all the time or only when something is happening. Thanks for the feedback.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/Heimdall#62
No description provided.