Skip to content
This repository has been archived by the owner on Jan 24, 2019. It is now read-only.

HTTP to HTTPS redirect listener #324

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bshelton229
Copy link

An attempt to solve #308. I apologize in advance, I'm very new to go. This pr listens on a separate port that will redirect to the host and scheme of the configured redirect-url. The thinking is the redirect-url will be the publicly accessible https URL when wanting to re-direct http -> https.

Thanks!

@bshelton229 bshelton229 reopened this Nov 23, 2016
@danbf
Copy link

danbf commented Dec 1, 2016

@jehiah this is a patch for #308

i'm interested in this since right now I'm running nginx in addition to the oauth_proxy just to get http->https and would like to remove that.

@mgoldsborough
Copy link

Thanks @bshelton229. I've been looking for a solution for this for a while.

@jehiah I'm intersted as well. Any thoughts on if/when this would be merged and released?

@chopj
Copy link

chopj commented Jan 20, 2017

+1

@idntfy
Copy link
Contributor

idntfy commented Dec 20, 2017

hey guys, any updates regarding this very useful feature? thank you

cc @jehiah

@jehiah jehiah added this to the v2.4 milestone Dec 20, 2017
@jehiah
Copy link
Member

jehiah commented Dec 20, 2017

@idntfy We are trying to slowly work through the open PR's in the v2.3 milestone. I've queued this (along with letsencrypt support) into the v2.4 milestone. Sorry for the delay.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

Successfully merging this pull request may close these issues.

6 participants