27

Currently I have an ELB serving both http://www.example.org and https://www.example.org.

I would like to set it up so any request pointing to http://www.example.org is redirect to https://www.example.org.

The ELB sends the https requests as http requests, so using:

server {
      listen         80;
       server_name    www.example.org;
       rewrite        ^ https://$server_name$request_uri? permanent;
}

will not work because requests made to https://www.example.org will still be made to port 80 on nginx.

I know it's possible to rewrite it as 

server {
      listen         80;
      server_name    www.example.org;
      if ($http_x_forwarded_proto != "https") {
          rewrite ^(.*)$ https://$server_name$1 permanent;
      }
}

But everything I've read said that if should be avoided at all costs within nginx configuration, and this would be for every single request. Also, it means I have to set up a special separate configuration for the health check (as described here: "…when you are behind an ELB, where the ELB is acting as the HTTPS endpoint and only sending HTTP traffic to your server, you break the ability to respond with an HTTP 200 OK response for the health check that the ELB needs").

I'm considering putting the login in the code of the web application rather than the nginx configuration (and for the purposes of this question let's assume it's a Django-based application), but I'm not certain whether that would be more overhead than the if in configuration.

Jordan Reiter
  • 1,260
  • 4
  • 17
  • 38
  • Hi can you please tell me where do you put these code? – YuAn Shaolin Maculelê Lai Jun 29 '17 at 10:46
  • @YuAnShaolinMaculelêLai Sure. These are configuration files for nginx, so I just put the code in a file in /etc/nginx/conf.d/. I usually name the file domainname.conf where "domainname" is the domain of the website in question. You can name the file whatever you want so long as it ends with .conf. – Jordan Reiter Jun 29 '17 at 18:49
  • Thank you very much. I tried to create a new file following by .conf. But it didn't work for me. Then I put the code in the file that generated from AWS in /etc/nginx/conf.d/. It works now. – YuAn Shaolin Maculelê Lai Jul 06 '17 at 06:52

4 Answers4

15
  1. Setup your AWS ELB mapping ELB:80 to instance:80 and ELB:443 to instance:1443.
  2. Bind nginx to listen on port 80 and 1443.
  3. Forward requests arriving at port 80 to port 443.
  4. Health check should be HTTP:1443. It rejects the HTTP:80 because the 301 redirect.

aws elb setup

NGINX Setup

    server {
       listen         80;
       server_name    www.example.org;
       rewrite        ^ https://$server_name$request_uri? permanent;
    }

    server {
       listen         1443;
       server_name    www.example.org;
   }
nu everest
  • 907
  • 3
  • 13
  • 27
  • what about the health check settings? could you please elaborate on that as well. – samkhan13 Apr 22 '15 at 17:25
  • this did not work with the helth check set to http:80, the health check fails. – samkhan13 Apr 22 '15 at 19:41
  • 2
    Health check should be `HTTP:1443`. It rejects the `HTTP:80` because the 301 redirect. – cbron Jun 19 '15 at 22:47
  • this mostly worked for me but the rewrite line did not work with my wildcard domain. This other post fixed that part: http://serverfault.com/questions/447258/redirect-wildcard-subdomains-to-https-nginx/518385#518385 – Ron Sep 08 '16 at 05:30
9

If it's working correctly like that, don't be scared of it. http://wiki.nginx.org/IfIsEvil

It is important to note that the behaviour of if is not inconsistent, given two identical requests it will not randomly fail on one and work on the other, with proper testing and understanding ifs can be used. The advice to use other directives where available still very much apply, though.

ceejayoz
  • 32,469
  • 7
  • 81
  • 105
  • This page also says that "if has problems when used in location context". It seems to me that you can do what you're needing to do outside of `location {}`, in `server {}` instead. (But please let me know if this is incorrect!) – Excalibur May 24 '18 at 19:51
9

This solution uses conditional logic, but as the accepted answer suggests, I also think this is ok. Ref: https://stackoverflow.com/questions/4833238/nginx-conf-redirect-multiple-conditions

Also, this doesn't require opening any additional ports in the aws security settings for the image. You can terminate ssl in the AWS LB, and route https traffic to http port 80 on your instance.

In this example the LB health check hits /health on port 80 which routes to the app server, so the health check validates both nginx and your app are breathing.

server {
  listen 80 default deferred;

  set $redirect_to_https 0;
  if ($http_x_forwarded_proto != 'https') {
    set $redirect_to_https 1;
  }
  if ($request_uri = '/health') {
    set $redirect_to_https 0;
  }
  if ($redirect_to_https = 1) {
    rewrite ^ https://www.example.com$request_uri? permanent;
  }
  ...
}
Community
  • 1
Jonny Mac
  • 91
  • 1
  • 2
0

You can now create a new Listener in AWS Load Balancer Settings which redirects HTTP Port 80 to HTTPS Port 443. So you don't need to touch the nginx/apache config anymore.

Kenny Greulich
  • 1