Skip to content

Commit 0f2b81e

Browse files
committed
Polish tests
1 parent 48c3c3d commit 0f2b81e

4 files changed

Lines changed: 14 additions & 16 deletions

File tree

python/ql/test/experimental/query-tests/Security/CWE-614/django_bad.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,11 +3,13 @@
33

44
def django_response(request):
55
resp = django.http.HttpResponse()
6-
resp.set_cookie("name", "value", secure=None)
6+
resp.set_cookie("name", "value", secure=False,
7+
httponly=False, samesite='None')
78
return resp
89

910

1011
def django_response(request):
1112
resp = django.http.HttpResponse()
12-
resp.set_cookie("name", "value", secure=False)
13+
resp.set_cookie("name", "value", secure=False,
14+
httponly=False, samesite='None')
1315
return resp

python/ql/test/experimental/query-tests/Security/CWE-614/django_good.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,13 +3,14 @@
33

44
def django_response(request):
55
resp = django.http.HttpResponse()
6-
resp['Set-Cookie'] = "name=value; Secure;"
6+
resp['Set-Cookie'] = "name=value; Secure; HttpOnly; SameSite=Lax;"
77
return resp
88

99

1010
def django_response(request):
1111
resp = django.http.HttpResponse()
12-
resp.set_cookie("name", "value", secure=True)
12+
resp.set_cookie("name", "value", secure=True,
13+
httponly=True, samesite='Lax')
1314
return resp
1415

1516

python/ql/test/experimental/query-tests/Security/CWE-614/flask_bad.py

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -6,14 +6,8 @@
66
@app.route("/false")
77
def false():
88
resp = make_response()
9-
resp.set_cookie("name", value="value", secure=False)
10-
return resp
11-
12-
13-
@app.route("/none")
14-
def none():
15-
resp = make_response()
16-
resp.set_cookie("name", value="value", secure=None)
9+
resp.set_cookie("name", value="value", secure=False,
10+
httponly=False, samesite='None')
1711
return resp
1812

1913

@@ -27,7 +21,7 @@ def flask_Response():
2721
@app.route("/flask_make_response")
2822
def flask_make_response():
2923
resp = make_response("hello")
30-
resp.headers['Set-Cookie'] = "name=value;"
24+
resp.headers['Set-Cookie'] = "name=value; SameSite=None;"
3125
return resp
3226

3327
# if __name__ == "__main__":

python/ql/test/experimental/query-tests/Security/CWE-614/flask_good.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,21 +6,22 @@
66
@app.route("/true")
77
def true():
88
resp = make_response()
9-
resp.set_cookie("name", value="value", secure=True)
9+
resp.set_cookie("name", value="value", secure=True,
10+
httponly=True, samesite='Lax')
1011
return resp
1112

1213

1314
@app.route("/flask_Response")
1415
def flask_Response():
1516
resp = Response()
16-
resp.headers['Set-Cookie'] = "name=value; Secure;"
17+
resp.headers['Set-Cookie'] = "name=value; Secure; HttpOnly; SameSite=Lax;"
1718
return resp
1819

1920

2021
@app.route("/flask_make_response")
2122
def flask_make_response():
2223
resp = make_response("hello")
23-
resp.headers['Set-Cookie'] = "name=value; Secure;"
24+
resp.headers['Set-Cookie'] = "name=value; Secure; HttpOnly; SameSite=Lax;"
2425
return resp
2526

2627

0 commit comments

Comments
 (0)