eric6/Plugins/CheckerPlugins/CodeStyleChecker/Security/Checks/jinja2Templates.py

changeset 7615
ca2949b1a29a
child 7923
91e843545d9a
--- /dev/null	Thu Jan 01 00:00:00 1970 +0000
+++ b/eric6/Plugins/CheckerPlugins/CodeStyleChecker/Security/Checks/jinja2Templates.py	Wed Jun 10 17:52:53 2020 +0200
@@ -0,0 +1,105 @@
+# -*- coding: utf-8 -*-
+
+# Copyright (c) 2020 Detlev Offenbach <detlev@die-offenbachs.de>
+#
+
+"""
+Module implementing a check for not auto escaping in jinja2.
+"""
+
+#
+# This is a modified version of the one found in the bandit package.
+#
+# Original Copyright 2014 Hewlett-Packard Development Company, L.P.
+#
+# SPDX-License-Identifier: Apache-2.0
+#
+
+import ast
+
+
+def getChecks():
+    """
+    Public method to get a dictionary with checks handled by this module.
+    
+    @return dictionary containing checker lists containing checker function and
+        list of codes
+    @rtype dict
+    """
+    return {
+        "Call": [
+            (checkJinja2Autoescape, ("S701",)),
+        ],
+    }
+
+
+def checkJinja2Autoescape(reportError, context, config):
+    """
+    Function to check for not auto escaping in jinja2.
+    
+    @param reportError function to be used to report errors
+    @type func
+    @param context security context object
+    @type SecurityContext
+    @param config dictionary with configuration data
+    @type dict
+    """
+    if isinstance(context.callFunctionNameQual, str):
+        qualnameList = context.callFunctionNameQual.split('.')
+        func = qualnameList[-1]
+        if 'jinja2' in qualnameList and func == 'Environment':
+            for node in ast.walk(context.node):
+                if isinstance(node, ast.keyword):
+                    # definite autoescape = False
+                    if (
+                        getattr(node, 'arg', None) == 'autoescape' and
+                        (
+                            getattr(node.value, 'id', None) == 'False' or
+                            getattr(node.value, 'value', None) is False
+                        )
+                    ):
+                        reportError(
+                            context.node.lineno - 1,
+                            context.node.col_offset,
+                            "S701.1",
+                            "H",
+                            "H",
+                        )
+                        return
+                    
+                    # found autoescape
+                    if getattr(node, 'arg', None) == 'autoescape':
+                        value = getattr(node, 'value', None)
+                        if (
+                            getattr(value, 'id', None) == 'True' or
+                            getattr(value, 'value', None) is True
+                        ):
+                            return
+                        
+                        # Check if select_autoescape function is used.
+                        elif (
+                            isinstance(value, ast.Call) and
+                            (getattr(value.func, 'id', None) ==
+                             'select_autoescape')
+                        ):
+                            return
+                        
+                        else:
+                            reportError(
+                                context.node.lineno - 1,
+                                context.node.col_offset,
+                                "S701.1",
+                                "H",
+                                "M",
+                            )
+                            return
+            
+            # We haven't found a keyword named autoescape, indicating default
+            # behavior
+            reportError(
+                context.node.lineno - 1,
+                context.node.col_offset,
+                "S701.2",
+                "H",
+                "H",
+            )

eric ide

mercurial