Skip to content

Commit 362a4a3

Browse files
authored
Merge pull request #10 from github-samples/alert-autofix-2
Potential fix for code scanning alert no. 2: Reflected cross-site scripting
2 parents 015d819 + 3cb91a8 commit 362a4a3

File tree

1 file changed

+10
-1
lines changed

1 file changed

+10
-1
lines changed

pages/api/display-message.js

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,15 @@
22
// DO NOT USE IN PRODUCTION
33
// This API endpoint demonstrates XSS vulnerabilities for CodeQL detection
44

5+
function escapeHtml(str) {
6+
return String(str)
7+
.replace(/&/g, '&')
8+
.replace(/</g, '&lt;')
9+
.replace(/>/g, '&gt;')
10+
.replace(/"/g, '&quot;')
11+
.replace(/'/g, '&#39;');
12+
}
13+
514
export default function handler(req, res) {
615
const { message } = req.query;
716

@@ -20,7 +29,7 @@ export default function handler(req, res) {
2029
</head>
2130
<body>
2231
<h1>Your Message:</h1>
23-
<div>${message}</div>
32+
<div>${escapeHtml(message)}</div>
2433
</body>
2534
</html>
2635
`;

0 commit comments

Comments
 (0)