Skip to content

Commit a887ff4

Browse files
committed
JS: Add test cases to include results with column-zero end locations
1 parent 577f1a5 commit a887ff4

File tree

3 files changed

+19
-1
lines changed

3 files changed

+19
-1
lines changed

javascript/ql/test/query-tests/Security/CWE-094/ExpressionInjection/.github/workflows/comment_issue.yml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,12 @@ on: issue_comment
22

33
jobs:
44
echo-chamber:
5+
runs-on: ubuntu-latest
6+
steps:
7+
- run: |
8+
echo '${{ github.event.comment.body }}'
9+
10+
echo-chamber2:
511
runs-on: ubuntu-latest
612
steps:
713
- run: |
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
on: issue_comment
2+
3+
# same as comment_issue but this file ends with a line break
4+
5+
jobs:
6+
echo-chamber:
7+
runs-on: ubuntu-latest
8+
steps:
9+
- run: |
10+
echo '${{ github.event.comment.body }}'
Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1,3 @@
1-
| .github/workflows/comment_issue.yml:7:12:8:47 | \| | Potential injection from the github.event.comment.body context, which may be controlled by an external user. |
1+
| .github/workflows/comment_issue.yml:7:12:10:0 | \| | Potential injection from the github.event.comment.body context, which may be controlled by an external user. |
2+
| .github/workflows/comment_issue.yml:13:12:14:47 | \| | Potential injection from the github.event.comment.body context, which may be controlled by an external user. |
3+
| .github/workflows/comment_issue_newline.yml:9:12:11:0 | \| | Potential injection from the github.event.comment.body context, which may be controlled by an external user. |

0 commit comments

Comments
 (0)