Skip to content

Commit

Permalink
added sanitize middleware class for django
Browse files Browse the repository at this point in the history
  • Loading branch information
deronsmith committed Oct 31, 2024
1 parent ae6c254 commit 789aea5
Show file tree
Hide file tree
Showing 4 changed files with 55 additions and 0 deletions.
Binary file removed hms_comments.sqlite3
Binary file not shown.
53 changes: 53 additions & 0 deletions request_sanitize.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,53 @@
import logging
import json
import requests
from json import JSONDecodeError
from django.http import HttpResponseBadRequest
from urllib.parse import unquote
import bleach

logger = logging.getLogger(__name__)


class SanitizeMiddleware:
def __init__(self, get_response):
self.get_response = get_response

def clean_dict(self, data):
for key, value in data.items():
if isinstance(value, dict):
data[key] = self.clean_dict(value)
else:
data[key] = bleach.clean(value)
return data

def __call__(self, request):
request.url = unquote(request.get_full_path())
if request.method == 'POST':
if request.content_type == 'application/json':
if len(request.POST) == 0:
try:
request_body = (request.body.decode("utf-8")).replace('\t', '').replace('\r\n', '')
if type(request_body) == str:
data = json.loads(request_body)
else:
data = request_body
except JSONDecodeError as e:
logger.warn("Unable to load JSON data in POST requests")
return HttpResponseBadRequest(content=f"Unable to load JSON data in POST requests".encode("utf-8"))
else:
data = request.POST
else:
logger.warn("Only JSON data is supported for POST requests")
return HttpResponseBadRequest(content=f"Only JSON data is supported for POST requests".encode("utf-8"))
for key, value in data.items():
data[key] = bleach.clean(value)
request.POST = data
elif request.method == 'GET':
params = request.GET.copy()
for key in params:
params[key] = bleach.clean(params[key])
request.GET = params

response = self.get_response(request)
return response
1 change: 1 addition & 0 deletions requirements.txt
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
bleach==6.2.0
cryptography==43.0.1
dask==2022.1.1
distributed==2022.1.1
Expand Down
1 change: 1 addition & 0 deletions settings.py
Original file line number Diff line number Diff line change
Expand Up @@ -86,6 +86,7 @@
'django.contrib.auth.middleware.AuthenticationMiddleware',
'django.contrib.messages.middleware.MessageMiddleware',
'django.middleware.clickjacking.XFrameOptionsMiddleware',
'hms_app.request_sanitize.SanitizeMiddleware'
]

ROOT_URLCONF = 'urls'
Expand Down

0 comments on commit 789aea5

Please # to comment.