Skip to content

Comments

implemented cli / environment option for disabling HTTP to HTTPS redirect#242

Merged
umputun merged 1 commit intoumputun:masterfrom
2128506:master
Feb 13, 2026
Merged

implemented cli / environment option for disabling HTTP to HTTPS redirect#242
umputun merged 1 commit intoumputun:masterfrom
2128506:master

Conversation

@2128506
Copy link
Contributor

@2128506 2128506 commented Feb 13, 2026

This PR adds CLI option --ssl.no-redirect and env option NO_REDIRECT - if set, HTTP to HTTPS redirect service is not started.
Rationale: when reproxy is used for terminating HTTPS and forwarding to HTTP server, HTTP to HTTPS redirect on reproxy level is not needed. This option provide way for cleaner setup.

@2128506 2128506 requested a review from umputun as a code owner February 13, 2026 13:41
Copy link
Owner

@umputun umputun left a comment

Choose a reason for hiding this comment

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

looks good overall, the three-way logic for SSLAuto (DNS-01 vs HTTP-01 with no-redirect) is well thought out.

one minor thing - the PR description mentions NO_REDIRECT env var, but since it's inside the SSL group it's actually SSL_NO_REDIRECT. would be good to fix the description to avoid confusion.

lgtm otherwise, thx

@umputun umputun merged commit 2204ac9 into umputun:master Feb 13, 2026
1 check failed
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