Merge "Disable http_proxy_to_wsgi middleware by default"

1 files changed, 31 insertions, 0 deletions

diff --git a/oslo_middleware/tests/test_http_proxy_to_wsgi.py b/oslo_middleware/tests/test_http_proxy_to_wsgi.py
index 8db1190..26baa77 100644
--- a/oslo_middleware/tests/test_http_proxy_to_wsgi.py
+++ b/oslo_middleware/tests/test_http_proxy_to_wsgi.py
@@ -29,6 +29,10 @@ class TestHTTPProxyToWSGI(test_base.BaseTestCase):
             return util.application_uri(req.environ)
 
         self.middleware = http_proxy_to_wsgi.HTTPProxyToWSGI(fake_app)
+        self.middleware.oslo_conf.set_override('enable_proxy_headers_parsing',
+                                               True,
+                                               group='oslo_middleware',
+                                               enforce_type=True)
         self.request = webob.Request.blank('/foo/bar', method='POST')
 
     def test_backward_compat(self):
@@ -98,3 +102,30 @@ class TestHTTPProxyToWSGI(test_base.BaseTestCase):
         self.request.headers['X-Forwarded-Prefix'] = "/bla"
         response = self.request.get_response(self.middleware)
         self.assertEqual(b"https://example.com:8043/bla", response.body)
+
+
+class TestHTTPProxyToWSGIDisabled(test_base.BaseTestCase):
+
+    def setUp(self):
+        super(TestHTTPProxyToWSGIDisabled, self).setUp()
+
+        @webob.dec.wsgify()
+        def fake_app(req):
+            return util.application_uri(req.environ)
+
+        self.middleware = http_proxy_to_wsgi.HTTPProxyToWSGI(fake_app)
+        self.middleware.oslo_conf.set_override('enable_proxy_headers_parsing',
+                                               False,
+                                               group='oslo_middleware',
+                                               enforce_type=True)
+        self.request = webob.Request.blank('/foo/bar', method='POST')
+
+    def test_no_headers(self):
+        response = self.request.get_response(self.middleware)
+        self.assertEqual(b"http://localhost:80/", response.body)
+
+    def test_url_translate_ssl_has_no_effect(self):
+        self.request.headers['X-Forwarded-Proto'] = "https"
+        self.request.headers['X-Forwarded-Host'] = "example.com:123"
+        response = self.request.get_response(self.middleware)
+        self.assertEqual(b"http://localhost:80/", response.body)