[Commotion-admin] [luci-commotion] Change HTTPS header check to look for port 443 instead of missing HTTPS environment variable (#448)

Josh King notifications at github.com
Thu Dec 11 13:33:33 EST 2014


This fix should act as a workaround for the fact that uhttpd2 has removed HTTPS as a part of the server environment in Barrier Breaker. In order to test:
* Build an image using the bb-test commotion-router branch and including this fix.
* Run through the setup wizard
* Ensure that you can login to the admin interface without issue after setup is complete
You can merge this Pull Request by running:

  git pull https://github.com/opentechinstitute/luci-commotion https-check

Or you can view, comment on it, or merge it online at:

  https://github.com/opentechinstitute/luci-commotion/pull/448

-- Commit Summary --

  * Change HTTPS header check to look for port 443 instead of missing HTTPS environment variable.

-- File Changes --

    M modules/crypto.lua (2)

-- Patch Links --

https://github.com/opentechinstitute/luci-commotion/pull/448.patch
https://github.com/opentechinstitute/luci-commotion/pull/448.diff

---
Reply to this email directly or view it on GitHub:
https://github.com/opentechinstitute/luci-commotion/pull/448
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.chambana.net/pipermail/commotion-admin/attachments/20141211/53353f97/attachment.html>


More information about the Commotion-admin mailing list