Skip to content

Add support for virtual host redirection #1563

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Add support for virtual host redirection #1563

wants to merge 2 commits into from

Conversation

propheth
Copy link

@propheth propheth commented Feb 25, 2021 •
edited
Loading

Added the REDIRECT environment variable to solve issues #180, #960, #958, #444, #1095 and an alternative to #1369.
Should work for both HTTP and HTTPS, depending on what is configured and perform a single redirect.

The config below does non-www to www,

environment:
    - VIRTUAL_HOST: test.com,www.test.com,a.test.com
    - REDIRECT: www.test.com
    - LETSENCRYPT_HOST: test.com,www.test.com,a.test.com

will create the 301 redirects below

test.com -> www.test.com
a.test.com -> www.test.com

The config below does www to non-www,

environment:
    - VIRTUAL_HOST: test.com,www.test.com,a.test.com
    - REDIRECT: test.com
    - LETSENCRYPT_HOST: test.com,www.test.com,a.test.com

will create the 301 redirects below

www.test.com -> test.com
a.test.com -> test.com

@buchdag buchdag added kind/feature-request Issue requesting a new feature status/pr-needs-tests This PR needs new or additional test(s) labels Mar 17, 2021
@alekna
Copy link
Contributor

alekna commented Apr 3, 2021

How about:

  • making redirect code configurable
  • if not configured, making HTTP/302 a default redirect code

@buchdag buchdag added type/feat PR for a new feature and removed kind/feature-request Issue requesting a new feature labels Apr 29, 2021
@coaperator
Copy link

When is the merger of this PR, #1836 and #1927 planned?

@buchdag buchdag added the scope/host-aliases Issue or PR related to alias redirection label Dec 26, 2022
@R0Wi
Copy link
Contributor

R0Wi commented Jul 13, 2023

@buchdag would you like to see such a feature in this product? If yes, I can provide a conflict resolution for this PR and add appropriate tests. Just let me know 👍

@buchdag
Copy link
Member

buchdag commented Jul 13, 2023

@R0Wi if documented and tested, yes I'd like to see this feature merged 👍

@buchdag buchdag mentioned this pull request Jul 13, 2023
Closed
@Langsdorf
Copy link

@R0Wi hey, any updates on this pr?

@CarlosTorrecillas
Copy link

Thanks for this! I am using it right now and it works like a charm. I had to update it a little bit because the template itself has been updated but the essence is great. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
scope/host-aliases Issue or PR related to alias redirection status/pr-needs-tests This PR needs new or additional test(s) type/feat PR for a new feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@propheth @alekna @coaperator @R0Wi @buchdag @Langsdorf @CarlosTorrecillas
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy