docs: add self-signed certificate guide#2776
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
📝 WalkthroughWalkthroughAdds a new advanced docs page describing how to configure Seerr to trust upstream services using self-signed SSL certificates via Node.js CA configuration, and updates media-server and services settings docs to note self-signed certs are untrusted by default but can be configured. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
a9eecec to
38ee70e
Compare
Add a new page at docs/using-seerr/advanced/ covering NODE_EXTRA_CA_CERTS (recommended) and NODE_TLS_REJECT_UNAUTHORIZED (fallback) for self-signed certificate setups. Update the four "Use SSL" sections in services.md and mediaserver.mdx to cross-reference the new guide instead of saying "not supported." Closes #2578
- Link note to /getting-started instead of /getting-started/docker - Remove alternative disable-validation section per maintainer request
Since the alternative method section was removed, the "Recommended:" qualifier is unnecessary. Also removes the horizontal rule separator and adjusts the description text.
38ee70e to
4462a57
Compare
The alternative method section was removed per review feedback, so there is only one approach now. Drop the qualifier.
Description
Add a new docs page covering how to use self-signed certificates with Seerr. Documents the
NODE_EXTRA_CA_CERTSapproach with Docker CLI and Docker Compose examples. Updates the four "Use SSL" sections in services.md and mediaserver.mdx to link to the new guide instead of saying certificates are "not supported."How Has This Been Tested?
pnpm startin the docs directoryScreenshots / Logs (if applicable)
N/A — documentation-only change.
Checklist:
pnpm buildpnpm i18n:extractAI Disclosure
AI tools were used to assist with drafting and reviewing this documentation. All content was verified for technical accuracy against the Node.js documentation and the Seerr codebase.
Summary by CodeRabbit