Skip to content

Commit c8ea292

Browse files
committed
Bump version to 0.2.6 and enhance proxy_pass_normalized plugin to handle variable proxy_pass destinations
1 parent 264a781 commit c8ea292

File tree

3 files changed

+9
-1
lines changed

3 files changed

+9
-1
lines changed

gixy/__init__.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,4 +2,4 @@
22

33
from gixy.core import severity
44

5-
version = "0.2.5"
5+
version = "0.2.6"

gixy/plugins/proxy_pass_normalized.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,10 @@ def audit(self, directive):
3232
if not proxy_pass_args:
3333
return
3434

35+
if proxy_pass_args[0].startswith("$"):
36+
# If proxy pass destination is defined by a variable, it is not possible to check for path normalization issues
37+
return
38+
3539
parsed = urlparse(proxy_pass_args[0])
3640

3741
if not parsed:
Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
# False positive because we can't reliably check whether variable has path or not
2+
location @__auth__event_proxying {
3+
proxy_pass $sm_auth_event_url;
4+
}

0 commit comments

Comments
 (0)