This commit is to include the line number of a given occurrence when …

…using HTML output format. (#683) Resolves: #672
PyCQA · Dec 27, 2020 · 8eee173 · 8eee173
1 parent 1956d64
commit 8eee173
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/bandit/formatters/html.py b/bandit/formatters/html.py
@@ -263,6 +263,7 @@ def report(manager, fileobj, sev_level, conf_level, lines=-1):
     <b>Severity: </b>{severity}<br>
     <b>Confidence: </b>{confidence}<br>
     <b>File: </b><a href="{path}" target="_blank">{path}</a> <br>
+    <b>Line number: </b>{line_number}<br>
     <b>More info: </b><a href="{url}" target="_blank">{url}</a><br>
 {code}
 {candidates}
@@ -358,7 +359,8 @@ def report(manager, fileobj, sev_level, conf_level, lines=-1):
                                           confidence=issue.confidence,
                                           path=issue.fname, code=code,
                                           candidates=candidates,
-                                          url=url)
+                                          url=url,
+                                          line_number=issue.lineno)
 
     # build the metrics string to insert in the report
     metrics_summary = metrics_block.format(

diff --git a/tests/unit/formatters/test_html.py b/tests/unit/formatters/test_html.py
@@ -120,6 +120,7 @@ def test_report_contents(self, get_issue_list, get_code):
             self.assertIn('BBBBBBB', issue1.text)
             self.assertIn('CCCCCCC', issue1.text)
             self.assertIn('abc.py', issue1.text)
+            self.assertIn('Line number: 1', issue1.text)
 
     @mock.patch('bandit.core.issue.Issue.get_code')
     @mock.patch('bandit.core.manager.BanditManager.get_issue_list')