@bnfinet This is fixing a goof I made in #263 , sorry about the thrashing.

The 302/redirect does need to be absolute -- for the use cases where request initially comes in on a url other than /. For example, https://myserver.example.com/my/cool/stuff/, the redirect should go to https://myserver.example.com/login, not https://myserver.example.com/my/cool/stuff/login

redirecting to the same $scheme and $http_host, is what is needed. I was originally trying to avoid having to hardcode a hostname here, and this is the fix for that. I have tested this working standalone, and behind a downstream http proxy (a Kubernetes Ingress).

Again, sorry for the thrashing :/

No problem @bgehman glad you caught it. Thanks for the update.

Nobody bats a 1000 :)

Appreciate it 👍