Skip to content

Expand Coverage of SSL Verification Settings #530

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

Open
wants to merge 47 commits into
base: dev
Choose a base branch
from

Conversation

jharder01
Copy link
Contributor

This pull request includes changes to improve SSL verification handling across multiple APIs and refactors some request logic for better maintainability. The most significant updates include centralizing SSL verification settings, adding support for query parameters in API requests, and simplifying redundant code.

Admin9705 and others added 30 commits May 20, 2025 18:49
feat: expand SSL verification handling to API requests in Radarr, Rea…
feat: refactor test_connection to use arr_request for API status check
feat: add verify_ssl parameter to get_missing_episodes and update API…
feat: refactor API request handling to use arr_request for multiple e…
feat: update arr_request to ignore verify_ssl parameter in favor of g…
feat: update arr_request and check_connection to use global SSL verif…
feat: enhance connection check and add get_system_status function for…
feat: add debug logging for missing episodes count request in get_mis…
feat: reintroduce retry logic and enhance SSL verification logging in…
…nhance logging; adjust page size and reintroduce retry logic in get_cutoff_unmet_episodes_random_page function
feat: update arr_request to use global SSL verification setting and e…
…s_manager; enhance logging for better traceability
…_episodes_random_page for clarity; remove redundant code and enhance logging
jharder01 added 17 commits May 25, 2025 18:17
@jharder01
Copy link
Contributor Author

@Admin9705 I cleared the conflicts between branches. Is there anything else I should be doing prior to merging to dev?

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