ring-clojure / ring-defaults

A library to provide sensible Ring middleware defaults
MIT License
345 stars 32 forks source link

Redirect loop using `secure-site-defaults` #5

Closed metasoarous closed 10 years ago

metasoarous commented 10 years ago

Running on heroku. Any idea what might be cause this or what I should look at?

weavejester commented 10 years ago

Have you set :proxy to true in the configuration? e.g.

(assoc secure-site-defaults :proxy true)
metasoarous commented 10 years ago

Ah... Brilliant! Thanks so much.

Might I suggest putting a note in the docs somewhere warning that may be necessary for installations with proxies or load balances? I see from the description of proxy that if I had read that I would have put two and two together, but since I was only thinking about the SSL I didn't connect.

Again; thanks so much :-)

weavejester commented 10 years ago

Yep, I'll add a note to the docs warning people about that. I agree it's not immediately obvious.

metasoarous commented 10 years ago

Awesome! Thanks again!