1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
|
# Copyright (c) 2018 Alexander Todorov <atodorov@MrSenko.com>
# Licensed under the GPL 2.0: https://www.gnu.org/licenses/old-licenses/gpl-2.0.html
# For details: https://github.com/pylint-dev/pylint-django/blob/master/LICENSE
"""
Various suggestions about JSON http responses
"""
import astroid
from pylint import checkers
from pylint_django.__pkginfo__ import BASE_ID
from pylint_django.compat import check_messages
class JsonResponseChecker(checkers.BaseChecker):
"""
Looks for some common patterns when returning http responses containing
JSON data!
"""
# configuration section name
name = "json-response-checker"
msgs = {
f"R{BASE_ID}01": (
"Instead of HttpResponse(json.dumps(data)) use JsonResponse(data)",
"http-response-with-json-dumps",
"Used when json.dumps() is used as an argument to HttpResponse().",
),
f"R{BASE_ID}02": (
"Instead of HttpResponse(content_type='application/json') use JsonResponse()",
"http-response-with-content-type-json",
"Used when HttpResponse() is returning application/json.",
),
f"R{BASE_ID}03": (
"Redundant content_type parameter for JsonResponse()",
"redundant-content-type-for-json-response",
"Used when JsonResponse() contains content_type parameter. "
"This is either redundant or the content_type is not JSON "
"which is probably an error.",
),
}
@check_messages(
"http-response-with-json-dumps",
"http-response-with-content-type-json",
"redundant-content-type-for-json-response",
)
def visit_call(self, node):
if (
node.func.as_string().endswith("HttpResponse")
and node.args
and isinstance(node.args[0], astroid.Call)
and node.args[0].func.as_string() == "json.dumps"
):
self.add_message("http-response-with-json-dumps", node=node)
if node.func.as_string().endswith("HttpResponse") and node.keywords:
for keyword in node.keywords:
if keyword.arg == "content_type" and keyword.value.as_string().lower().find("application/json") > -1:
self.add_message("http-response-with-content-type-json", node=node)
break
if node.func.as_string().endswith("JsonResponse") and node.keywords:
for keyword in node.keywords:
if keyword.arg == "content_type":
self.add_message("redundant-content-type-for-json-response", node=node)
break
|
