Skip to content

docs: improve nginx example with HTTPS configuration#2606

Merged
vytas7 merged 3 commits intofalconry:masterfrom
toroleapinc:docs/issue-1731-improve-nginx-config
Mar 6, 2026
Merged

docs: improve nginx example with HTTPS configuration#2606
vytas7 merged 3 commits intofalconry:masterfrom
toroleapinc:docs/issue-1731-improve-nginx-config

Conversation

@toroleapinc
Copy link
Contributor

Summary

Update the NGINX example configuration in the deployment guide to include HTTPS support with HTTP-to-HTTPS redirect.

Closes #1731

Changes

  • Added HTTP → HTTPS redirect server block
  • Added SSL/TLS configuration with Let's Encrypt certificate paths
  • Referenced Mozilla SSL Configuration Generator for production tuning
  • Added note about simplifying to HTTP-only for development
  • Changed code block language from ini to nginx for proper syntax highlighting
  • Fixed missing semicolons in uwsgi_pass directive

Update the NGINX example configuration in the deployment guide to
include HTTPS support with HTTP-to-HTTPS redirect, which is the
standard configuration for production deployments.

Added:
- HTTP to HTTPS redirect server block
- SSL/TLS configuration with Let's Encrypt certificate paths
- Mozilla Intermediate TLS configuration reference
- Note about Mozilla SSL Configuration Generator
- Note about simplifying for development (HTTP-only)

Closes falconry#1731

Signed-off-by: edvatar <88481784+toroleapinc@users.noreply.github.com>
@codecov
Copy link

codecov bot commented Mar 4, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (42321c9) to head (b66168b).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##            master     #2606   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           64        64           
  Lines         7911      7911           
  Branches      1086      1086           
=========================================
  Hits          7911      7911           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@vytas7 vytas7 requested review from CaselIT and vytas7 March 5, 2026 12:12
Copy link
Member

@vytas7 vytas7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I checked the rendered docs and this looks good, thanks 👍

However, a general question (since you removed the PR template): which proportion of this PR was generated by LLM ("AI")?
We'd like contributors to state this when opening a PR (if that was the case): Use of LLMs (“AI”).

@toroleapinc
Copy link
Contributor Author

Thanks for the approval! To be transparent — the initial draft of this PR was LLM-assisted (mostly for identifying which RFCs had been superseded and generating the nginx config snippets). I reviewed and tested everything manually, and the RST fix for the Let's Encrypt reference was done by hand after seeing the CI failure.

@vytas7 vytas7 merged commit 002fa6c into falconry:master Mar 6, 2026
33 checks passed
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.

[docs] Improve nginx example configuration in the deploy FAQ

2 participants