Skip to content

Commit 2ca4cee

Browse files
authored
Add nonces to script tags (#177)
1 parent 716e5f2 commit 2ca4cee

File tree

7 files changed

+15
-13
lines changed

7 files changed

+15
-13
lines changed

exampleassay/resources/assay/example/views/upload.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@
3838
</a>
3939
</p>
4040

41-
<script type="text/javascript">
41+
<script type="text/javascript" nonce="<%=scriptNonce%>">
4242

4343
function done()
4444
{

interactiveTutorial/resources/views/Lab Results.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
<div id='divLabResults'/>
2-
<script type="text/javascript">
2+
<script type="text/javascript" nonce="<%=scriptNonce%>">
33

44
// Ensure that page dependencies are loaded
55
LABKEY.requiresExt3ClientAPI(function() {

interactiveTutorial/resources/views/Physical Exam.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
<div id='divPhysicalExam'/>
2-
<script type="text/javascript">
2+
<script type="text/javascript" nonce="<%=scriptNonce%>">
33

44
// Ensure that page dependencies are loaded
55
LABKEY.requiresExt3ClientAPI(function() {

reactExamples/resources/views/demoWebpart.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
<script type="application/javascript">
1+
<script type="text/javascript" nonce="<%=scriptNonce%>">
22
+function() {
33
LABKEY.App.loadApp('demoWebpart', <%=webpartContext%>.wrapperDivId);
44
}();

sourdough/resources/views/_header.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
<script>
1+
<script type="text/javascript" nonce="<%=scriptNonce%>">
22
const elems = document.querySelectorAll("div[id^='ModuleHtmlView']");
33
elems[0].style.display = "inline";
44
</script>

sourdough/resources/views/login.html

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -6,15 +6,15 @@
66
<input id="email" name="email" type="text" class="input-block" tabindex="1" autocomplete="off">
77
<label for="password">Password</label>
88
<div class="forgot-password-link">
9-
<a href="login-resetPassword.view?">Forgot password</a>
9+
<a href="login-resetPassword.view">Forgot password</a>
1010
</div>
1111
<input id="password" name="password" type="password" class="input-block" tabindex="2" autocomplete="off">
1212
<input tabindex="3" type="checkbox" name="remember" id="remember" checked> Remember my email address
1313
<div class="termsOfUseSection" hidden>
1414
<div class="auth-header auth-item">Terms of Use</div>
1515
<div class="toucontent auth-item termsOfUseContent"></div>
1616
<div class="auth-item">
17-
<input type="checkbox" tabindex="4" name="approvedTermsOfUse" id="approvedTermsOfUse" class="auth-item" unchecked>
17+
<input type="checkbox" tabindex="4" name="approvedTermsOfUse" id="approvedTermsOfUse" class="auth-item">
1818
<label for="approvedTermsOfUse">I agree to these terms</label>
1919
</div>
2020
</div>
@@ -23,7 +23,7 @@
2323
<input type="submit" tabindex="-1" class="loginSubmitButton"/>
2424
<a tabindex="5" class="labkey-button primary signin-btn"><span>Sign In</span></a>
2525
<span class="registrationSection" hidden>
26-
<a class="labkey-button" id="registerButton" href="login-register.view?">Register</a>
26+
<a class="labkey-button" id="registerButton" href="login-register.view">Register</a>
2727
</span>
2828
</div>
2929
<div class="signing-in-msg" hidden>
@@ -36,8 +36,10 @@
3636
</div>
3737
</form>
3838

39-
<script type="application/javascript" nonce="<%= scriptNonce %>">
40-
if (LABKEY.ActionURL.getParameter('returnUrl')) {
41-
document.getElementById('registerButton').href += 'returnUrl=' + encodeURIComponent(LABKEY.ActionURL.getParameter('returnUrl'));
42-
}
39+
<script type="text/javascript" nonce="<%=scriptNonce%>">
40+
LABKEY.Utils.onReady(function() {
41+
if (LABKEY.ActionURL.getParameter('returnUrl')) {
42+
document.getElementById('registerButton').href += '?returnUrl=' + encodeURIComponent(LABKEY.ActionURL.getParameter('returnUrl'));
43+
}
44+
});
4345
</script>

sourdough/resources/views/styling.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88
<img class="loaf" src="https://media.tenor.com/TTTGbs0vtOEAAAAi/bread-spin.gif" alt="bouncing bread"/>
99

1010
<div>
11-
Click <a href="<%=contextPath%>/project<%=containerPath%>/sourdough-styling.view"> here </a> to see this panel as a full page!
11+
Click <a href="<%=contextPath%><%=containerPath%>/sourdough-styling.view"> here </a> to see this panel as a full page!
1212
</div>
1313
</div>
1414

0 commit comments

Comments
 (0)