src/eric7/Plugins/CheckerPlugins/CodeStyleChecker/Security/Checks/requestWithoutTimeout.py

branch
eric7
changeset 9325
8157eb19aba5
child 9653
e67609152c5e
--- /dev/null	Thu Jan 01 00:00:00 1970 +0000
+++ b/src/eric7/Plugins/CheckerPlugins/CodeStyleChecker/Security/Checks/requestWithoutTimeout.py	Tue Sep 13 20:00:55 2022 +0200
@@ -0,0 +1,66 @@
+# -*- coding: utf-8 -*-
+
+# Copyright (c) 2022 Detlev Offenbach <detlev@die-offenbachs.de>
+#
+
+"""
+Module implementing checks for using requests without timeout.
+"""
+
+#
+# This is a modified version of the one found in the bandit package.
+#
+# SPDX-License-Identifier: Apache-2.0
+#
+
+
+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": [
+            (checkRequestWithouTimeout, ("S114",)),
+        ],
+    }
+
+
+def checkRequestWithouTimeout(reportError, context, config):
+    """
+    Function to check for use of requests without timeout.
+
+    @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
+    """
+    httpVerbs = ("get", "options", "head", "post", "put", "patch", "delete")
+    if (
+        "requests" in context.callFunctionNameQual
+        and context.callFunctionName in httpVerbs
+    ):
+        # check for missing timeout
+        if context.checkCallArgValue("timeout") is None:
+            reportError(
+                context.node.lineno - 1,
+                context.node.col_offset,
+                "S114.1",
+                "M",
+                "L",
+            )
+
+        # check for timeout=None
+        if context.checkCallArgValue("timeout", "None"):
+            reportError(
+                context.node.lineno - 1,
+                context.node.col_offset,
+                "S114.2",
+                "M",
+                "L",
+            )

eric ide

mercurial