search

MI-DPLA / combine

Combine /kämˌbīn/ - Metadata Aggregator Platform
MIT License
26 stars 11 forks source link

consider https for Combine Django app #386

Closed ghukill closed 4 years ago

ghukill commented 5 years ago

pointed out that Combine runs almost entirely on http, but hopefully would not be terribly difficult to move Django app to optional https as well

antmoth commented 4 years ago

We may actually be able to do this just by configuring nginx properly. Which is good, since we aren't in direct control of most deployments and you do need a valid certificate to run https. But I can write up a how-to and link to Let's Encrypt, probably.

richardcadler commented 4 years ago

nginx can do this for us. Just waiting on an SSL certificate. @estelendur

antmoth commented 4 years ago

Fixed in docker deployment