facebookarchive / CommunityCellularManager

Tool for deploying, managing and controlling your Community Cellular Networks
Other
87 stars 36 forks source link

nginx config incorrect #17

Closed kheimerl closed 7 years ago

kheimerl commented 7 years ago

As per https://www.nginx.com/resources/wiki/start/topics/tutorials/config_pitfalls/#taxing-rewrites

nginx config ssl redirect should be:

# Force HTTPS
if ($http_x_forwarded_proto != "https") {
  return 301 https://$host$request_uri;
}
9muir commented 7 years ago

do you want to just make that a PR? Was that the cause of the double parameter in the query string?

kheimerl commented 7 years ago

I can make one eventually but i'm in the midst of the whole thing. This was the cause of the double param.

9muir commented 7 years ago

Closing, will accept PR once it's available.