adds rel tags as suggested by security checklist

mozilla-rally · Feb 18, 2021 · 6b7a140 · 6b7a140
1 parent e6c83e5
commit 6b7a140
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/src/routes/Sidebar.svelte b/src/routes/Sidebar.svelte
@@ -113,11 +113,11 @@ li a {
   <ul slot="navigation" in:fly={{duration: 800, delay: 300, x: -10}}>
     <li><button class:active={currentView === 'current-studies'} on:click={send('current-studies')}>Current Studies</button></li>
     <li>
-      <a class="external" target="_blank" href="https://support.mozilla.org/en-US/kb/about-mozilla-rally">Support
+      <a class="external" target="_blank" rel="noopener noreferrer" href="https://support.mozilla.org/en-US/kb/about-mozilla-rally">Support
         <ExternalLink /></a>
     </li>
     <li>
-      <a class="external" target="_blank" href="__BASE_SITE__/how-rally-works/faqs/">FAQ
+      <a class="external" target="_blank" rel="noopener noreferrer" href="__BASE_SITE__/how-rally-works/faqs/">FAQ
         <ExternalLink /></a>
     </li>
   </ul>

diff --git a/src/routes/non-eligible-splashes/NonUSUser.svelte b/src/routes/non-eligible-splashes/NonUSUser.svelte
@@ -81,7 +81,7 @@ onMount(() => {
                 </Button>
                 </div>
                 <div class="waitlist">
-                <a target="_blank" href="__BASE_SITE__/waitlist/">Join the Waitlist
+                <a target="_blank" rel="noopener noreferrer" href="__BASE_SITE__/waitlist/">Join the Waitlist
                     <ExternalLink /></a>
                 </div>
             </div>

diff --git a/src/routes/welcome/Content.svelte b/src/routes/welcome/Content.svelte
@@ -106,7 +106,7 @@
       </Button>
     </div>
     <div class="how-it-works">
-      <a class="external-link" target="_blank" href="__BASE_SITE__/how-rally-works/">Wait – how does it work again?
+      <a class="external-link" target="_blank" rel="noopener noreferrer" href="__BASE_SITE__/how-rally-works/">Wait – how does it work again?
         <ExternalLink /></a>
     </div>
   </div>