Skip to content

Commit 40d6de2

Browse files
authored
chore: escape raw html (#2395)
* chore: escape raw html * fix: don't escape includes directives * chore: don't escape long url
1 parent 80942e7 commit 40d6de2

File tree

2 files changed

+12
-12
lines changed

2 files changed

+12
-12
lines changed

src/server/views/404.error.ejs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6,10 +6,10 @@
66
<meta http-equiv="X-UA-Compatible" content="ie=edge">
77
<meta http-equiv="Content-Language" content="en">
88
<meta charset="UTF-8">
9-
<title><%- displayHostname %>: Page not found</title>
9+
<title><%= displayHostname %>: Page not found</title>
1010
<base href="~/" />
1111
<link href="/assets/styles/shared/shared-styles.css" rel="stylesheet">
12-
<link href="/assets/<%- assetVariant %>/styles/colours.css" rel="stylesheet">
12+
<link href="/assets/<%= assetVariant %>/styles/colours.css" rel="stylesheet">
1313
<link href="https://fonts.googleapis.com/css2?family=IBM+Plex+Sans:wght@400;500;700&display=swap" rel="stylesheet">
1414
<link href="/assets/styles/not-found-page/not-found-page.css" rel="stylesheet">
1515
<link rel="shortcut icon" href="/favicon.ico" type="image/x-icon">
@@ -29,11 +29,11 @@
2929
<div id="divider" class="divider"></div>
3030
<div id="graphics-container">
3131
<div id="graphic-div">
32-
<img src="/assets/<%- assetVariant %>/not-found-page/images/not-found-graphic.svg" draggable="false" />
32+
<img src="/assets/<%= assetVariant %>/not-found-page/images/not-found-graphic.svg" draggable="false" />
3333
</div>
3434
<div id="go-logo-div">
3535
<!-- go logo -->
36-
<img src="/assets/<%- assetVariant %>/not-found-page/icons/go-logo.svg" draggable="false" />
36+
<img src="/assets/<%= assetVariant %>/not-found-page/icons/go-logo.svg" draggable="false" />
3737
</div>
3838
</div>
3939
</main>

src/server/views/transition-page.ejs

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -6,9 +6,9 @@
66
<meta http-equiv="X-UA-Compatible" content="ie=edge">
77
<meta http-equiv="Content-Language" content="en">
88
<meta charset="UTF-8">
9-
<title><%- displayHostname %></title>
9+
<title><%= displayHostname %></title>
1010
<link href="/assets/styles/shared/shared-styles.css" rel="stylesheet">
11-
<link href="/assets/<%- assetVariant %>/styles/colours.css" rel="stylesheet">
11+
<link href="/assets/<%= assetVariant %>/styles/colours.css" rel="stylesheet">
1212
<link href="https://fonts.googleapis.com/css2?family=IBM+Plex+Sans:wght@400;500;700&display=swap" rel="stylesheet">
1313
<link href="/assets/styles/transition-page/transition-page.css" rel="stylesheet">
1414
<link rel="apple-touch-icon" sizes="180x180" href="/assets/apple-touch-icon.png">
@@ -22,23 +22,23 @@
2222
<div class="top-half">
2323
<div class="top-half-content">
2424
<h3>Check your address bar</h3>
25-
<a href="https://go.gov.sg/go-antiphishing" target="_blank" rel="noreferrer noopener">Beware of phishing! <br class="mobile-break">Make sure your link starts with <%- displayHostname.toLowerCase() %></a>
25+
<a href="https://go.gov.sg/go-antiphishing" target="_blank" rel="noreferrer noopener">Beware of phishing! <br class="mobile-break">Make sure your link starts with <%= displayHostname.toLowerCase() %></a>
2626
<p id="url" data-href="<%- escapedLongUrl %>">You will be redirected in <span id="countdown-seconds">6</span> second<span id="s">s</span></p>
27-
<img id="spinner" src="/assets/<%- assetVariant %>/transition-page/images/spinner.gif" alt="loading" />
28-
<div class="browser-image" style="background-image: url('/assets/<%- assetVariant %>/transition-page/images/browser.svg');">
29-
<div class="links links-<%- assetVariant %>" ><%- displayHostname.toLowerCase() %>/</div>
27+
<img id="spinner" src="/assets/<%= assetVariant %>/transition-page/images/spinner.gif" alt="loading" />
28+
<div class="browser-image" style="background-image: url('/assets/<%= assetVariant %>/transition-page/images/browser.svg');">
29+
<div class="links links-<%= assetVariant %>" ><%= displayHostname.toLowerCase() %>/</div>
3030
</div>
3131
</div>
3232
</div>
3333
<div class="bottom-half">
3434

3535
<div id="skip">
3636
<span>I've checked the link. Skip ahead</span>
37-
<img src="/assets/<%- assetVariant %>/transition-page/icons/icon-arrow-right.svg" alt="right arrow" />
37+
<img src="/assets/<%= assetVariant %>/transition-page/icons/icon-arrow-right.svg" alt="right arrow" />
3838
</div>
3939
<div class="footer">
4040
<div id="logo">
41-
<img src="/assets/<%- assetVariant %>/transition-page/icons/ogp-logo.svg" alt="ogp logo" />
41+
<img src="/assets/<%= assetVariant %>/transition-page/icons/ogp-logo.svg" alt="ogp logo" />
4242
</div>
4343
<p>You will only be shown this page the first time you access this short link.</p>
4444
</div>

0 commit comments

Comments
 (0)