[FIX] make dbfilter_from_header compliant with its documentation

yelizariev · Nov 18, 2015 · 5737ebc · 5737ebc
1 parent 35f26dc
commit 5737ebc
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 5 deletions.
diff --git a/dbfilter_from_header/__init__.py b/dbfilter_from_header/__init__.py
@@ -28,9 +28,12 @@
 def db_filter(dbs, httprequest=None):
     dbs = db_filter_org(dbs, httprequest)
     httprequest = httprequest or http.request.httprequest
-    db_filter_hdr = \
-        httprequest.environ.get('HTTP_X_ODOO_DBFILTER') or \
-        httprequest.environ.get('HTTP_X_OPENERP_DBFILTER')
+    db_filter_hdr_odoo = httprequest.environ.get('HTTP_X_ODOO_DBFILTER')
+    db_filter_hdr_openerp = httprequest.environ.get('HTTP_X_OPENERP_DBFILTER')
+    if db_filter_hdr_odoo and db_filter_hdr_openerp:
+        raise RuntimeError("x-odoo-dbfilter and x-openerp-dbfiter "
+                           "are both set")
+    db_filter_hdr = db_filter_hdr_odoo or db_filter_hdr_openerp
     if db_filter_hdr:
         dbs = [db for db in dbs if re.match(db_filter_hdr, db)]
     return dbs

diff --git a/dbfilter_from_header/__openerp__.py b/dbfilter_from_header/__openerp__.py
@@ -20,7 +20,7 @@
 ##############################################################################
 {
     "name": "dbfilter_from_header",
-    "version": "8.0.1.0.0",
+    "version": "8.0.1.0.1",
     "author": "Therp BV,Odoo Community Association (OCA)",
     "license": "AGPL-3",
     "complexity": "normal",
@@ -30,8 +30,9 @@
     This is interesting for setups where database names can't be mapped to
     proxied host names.
 
-    In nginx, use
+    In nginx, use one of
     proxy_set_header X-OpenERP-dbfilter [your filter];
+    proxy_set_header X-Odoo-dbfilter [your filter];
 
     The addon has to be loaded as server-wide module.
     """,