Skip to content

Commit c54c9d7

Browse files
committed
Added CWE information for Sonar and some pixee codemod
1 parent 7b2d930 commit c54c9d7

10 files changed

+29
-0
lines changed

src/core_codemods/disable_graphql_introspection.py

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -127,6 +127,12 @@ def _is_introspection_rule_or_starred(
127127
Reference(
128128
url="https://owasp.org/www-project-web-security-testing-guide/v42/4-Web_Application_Security_Testing/12-API_Testing/01-Testing_GraphQL#introspection-queries",
129129
),
130+
Reference(
131+
url="https://cwe.mitre.org/data/definitions/200.html",
132+
),
133+
Reference(
134+
url="https://cwe.mitre.org/data/definitions/669.html",
135+
),
130136
],
131137
),
132138
transformer=LibcstTransformerPipeline(DisableGraphQLIntrospectionTransform),

src/core_codemods/django_json_response_type.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,7 @@ def on_result_found(self, _, updated_node):
5353
Reference(
5454
url="https://cheatsheetseries.owasp.org/cheatsheets/Cross_Site_Scripting_Prevention_Cheat_Sheet.html#output-encoding-for-javascript-contexts"
5555
),
56+
Reference(url="https://cwe.mitre.org/data/definitions/79"),
5657
],
5758
),
5859
transformer=LibcstTransformerPipeline(DjangoJsonResponseTypeTransformer),

src/core_codemods/enable_jinja2_autoescape.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@ def on_result_found(self, original_node, updated_node):
3030
Reference(
3131
url="https://jinja.palletsprojects.com/en/3.1.x/api/#autoescaping"
3232
),
33+
Reference(url="https://cwe.mitre.org/data/definitions/79"),
3334
],
3435
),
3536
detector=SemgrepRuleDetector(

src/core_codemods/flask_json_response_type.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -281,6 +281,7 @@ def _fix_json_dumps(self, node: cst.BaseExpression) -> cst.Tuple:
281281
Reference(
282282
url="https://cheatsheetseries.owasp.org/cheatsheets/Cross_Site_Scripting_Prevention_Cheat_Sheet.html#output-encoding-for-javascript-contexts"
283283
),
284+
Reference(url="https://cwe.mitre.org/data/definitions/79"),
284285
],
285286
),
286287
transformer=LibcstTransformerPipeline(FlaskJsonResponseTypeTransformer),

src/core_codemods/jwt_decode_verify.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -109,6 +109,7 @@ def is_verify_keyword(element: cst.DictElement) -> bool:
109109
Reference(
110110
url="https://owasp.org/www-project-web-security-testing-guide/latest/4-Web_Application_Security_Testing/06-Session_Management_Testing/10-Testing_JSON_Web_Tokens"
111111
),
112+
Reference(url="https://cwe.mitre.org/data/definitions/347"),
112113
],
113114
),
114115
transformer=LibcstTransformerPipeline(JwtDecodeVerifyTransformer),

src/core_codemods/process_creation_sandbox.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,8 @@ class ProcessSandbox(SimpleCodemod):
1616
Reference(
1717
url="https://cheatsheetseries.owasp.org/cheatsheets/OS_Command_Injection_Defense_Cheat_Sheet.html"
1818
),
19+
Reference(url="https://cwe.mitre.org/data/definitions/20"),
20+
Reference(url="https://cwe.mitre.org/data/definitions/78"),
1921
],
2022
)
2123
change_description = (

src/core_codemods/secure_random.py

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,18 @@ def on_result_found(self, original_node, updated_node):
3333
Reference(
3434
url="https://docs.python.org/3/library/random.html",
3535
),
36+
Reference(
37+
url="https://cwe.mitre.org/data/definitions/338",
38+
),
39+
Reference(
40+
url="https://cwe.mitre.org/data/definitions/330",
41+
),
42+
Reference(
43+
url="https://cwe.mitre.org/data/definitions/326",
44+
),
45+
Reference(
46+
url="https://cwe.mitre.org/data/definitions/1241",
47+
),
3648
],
3749
),
3850
detector=SemgrepRuleDetector(

src/core_codemods/sql_parameterization.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -369,6 +369,7 @@ def _remove_literal_and_gather_extra(
369369
summary="Parameterize SQL Queries",
370370
review_guidance=ReviewGuidance.MERGE_AFTER_CURSORY_REVIEW,
371371
references=[
372+
Reference(url="https://cwe.mitre.org/data/definitions/20"),
372373
Reference(url="https://cwe.mitre.org/data/definitions/89.html"),
373374
Reference(url="https://owasp.org/www-community/attacks/SQL_Injection"),
374375
],

src/core_codemods/tempfile_mktemp.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -169,6 +169,8 @@ def _mktemp_is_sink(
169169
Reference(
170170
url="https://docs.python.org/3/library/tempfile.html#tempfile.mktemp"
171171
),
172+
Reference(url="https://cwe.mitre.org/data/definitions/377"),
173+
Reference(url="https://cwe.mitre.org/data/definitions/379"),
172174
],
173175
),
174176
transformer=LibcstTransformerPipeline(TempfileMktempTransformer),

src/core_codemods/url_sandbox.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,8 @@ def dependency(self) -> Dependency:
4040
url="https://www.rapid7.com/blog/post/2021/11/23/owasp-top-10-deep-dive-defending-against-server-side-request-forgery/"
4141
),
4242
Reference(url="https://blog.assetnote.io/2021/01/13/blind-ssrf-chains/"),
43+
Reference(url="https://cwe.mitre.org/data/definitions/20"),
44+
Reference(url="https://cwe.mitre.org/data/definitions/918"),
4345
],
4446
),
4547
detector=SemgrepRuleDetector(

0 commit comments

Comments
 (0)