Trust the user when they say that their server name starts with 'https'

Review Request #5490 — Created Feb. 17, 2014 and submitted

Information

Review Board
master
52590c8...

Reviewers

Trust the user when they say that their server name starts with 'https'

We have a long-standing bug where someone would configure Review Board to use
HTTPS only, but due to proxies or other methods, the HttpRequest wouldn't think
it was https. This meant that any URLs created by the Django resolvers would
start with 'http' instead of 'https'.

Because we know what the user configured their server name as, we can choose in
load_site_config() whether to set the 'HTTPS' environment variable or not. We
also need to set wsgi.url_scheme to 'https' in request.META for wsgi-based
clients.

Configured my devserver with a server name starting with "https" and fetched
the API root. Saw that all the links returned started with 'https'.

Description From Last Updated

We can nuke this now.

chipx86chipx86
chipx86
  1. Awesome! This should cut down on some support issues.

    One thing while you're in there.

  2. reviewboard/admin/siteconfig.py (Diff revision 1)
     
     

    We can nuke this now.

  3. 
      
david
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-1.7.x (ac6f869).
Loading...