Open
Description
Description:
The current logging behavior for HTTP timeouts and 429 (Too Many Requests) responses results in excessive stack trace spam in the logs. This makes troubleshooting difficult and consumes unnecessary storage.
Problem Examples:
-
Timeout Errors
When a request to a subtitle provider (e.g., Podnapisi.NET) times out, the full stack trace is logged:[ERR] subbuzz.Providers.PodnapisiNet: GET: [URL] Search error: System.Threading.Tasks.TaskCanceledException... [Full stack trace] -
Rate Limit Errors
When receiving 429 responses, the plugin logs the full exception:[ERR] subbuzz.Providers.PodnapisiNet: GET: [URL] Search error: MediaBrowser.Common.Extensions.RateLimitExceededException... [Full stack trace]
Suggested Improvements:
-
For timeout exceptions:
- Log a concise warning message instead of full stack trace
- Example:
[WRN] Request to Podnapisi.NET timed out after 20s (URL: [URL])
-
For 429 responses:
- Log as warning instead of error
- Include
Retry-Afterperiod if available in response headers - Example:
[WRN] Podnapisi.NET rate limit exceeded. Retry after 120s
-
General improvements:
- Add cooldown periods between repeated warnings for the same endpoint
- Include helper text like "This may indicate network issues or server-side throttling" in timeout messages
Benefits:
- Reduces log noise while preserving important diagnostic information
- Makes rate limiting behavior more explicit
- Helps users distinguish between temporary issues vs actual errors
Metadata
Metadata
Assignees
Labels
No labels