Skip to content

security: Adds rel="noopener" to all target="_blank" #494

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 9 commits into from
Jan 24, 2021
Merged
Show file tree
Hide file tree
Changes from 3 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion app/views/manage/messages/_templating.haml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

%p
Message bodies can make use of template variables to help personalize and streamline emails.
Templating is powered by <a target="_blank" href="https://mustache.github.io/mustache.5.html">mustache</a>.
Templating is powered by <a target="_blank" rel="noopener" href="https://mustache.github.io/mustache.5.html">mustache</a>.

%table.table.table-striped
%thead
Expand Down
2 changes: 1 addition & 1 deletion app/views/manage/messages/template.haml
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@
.form-inputs
%h5.mb-3 Customize template
%p.text-secondary Must save to update preview. CSS will be converted to inline styles when messages are sent.
%p.text-secondary Be sure to test with as many email providers as possible! <a href="https://putsmail.com" target="blank">Litmus PutsMail</a> is one great resource.
%p.text-secondary Be sure to test with as many email providers as possible! <a href="https://putsmail.com" target="_blank" rel="noopener">Litmus PutsMail</a> is one great resource.
= f.input :html, input_html: { 'data-code-mirror-textarea' => '1' }, label: false, wrapper: :bootstrap_inline_form

.form-actions.mt-3.mb-3
Expand Down
6 changes: 3 additions & 3 deletions db/seed_messages/questionnaire--accepted.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,8 @@
You have been accepted to attend {{hackathon_name}}! **Please RSVP:**

<p>
<a href="{{accept_rsvp_url}}" class="button" target="_blank">Yes, I will Attend &raquo;</a>
<a href="{{deny_rsvp_url}}" class="button" target="_blank">No, I Can't Attend &raquo;</a>
<a href="{{accept_rsvp_url}}" class="button" target="_blank" rel="noopener">Yes, I will Attend &raquo;</a>
<a href="{{deny_rsvp_url}}" class="button" target="_blank" rel="noopener">No, I Can't Attend &raquo;</a>
<br>
<small><i>Link not working? Go to <a href="{{rsvp_url}}">{{rsvp_url}}</a></i></small>
<small><i>Link not working? Go to <a href="{{rsvp_url}}" target="_blank" rel="noopener">{{rsvp_url}}</a></i></small>
</p>
2 changes: 1 addition & 1 deletion db/seed_messages/questionnaire--denied.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

It is with our sincerest regret to inform you that our admissions committee has chosen to not accept your application to {{hackathon_name}} at this time. We were overjoyed with the number of applicants we received, but unfortunately we can not accept everyone.

We invite you to apply again next year. There are plenty of other hackathons this season, and it may not be too late to apply for those. Checkout <a href="https://mlh.io" target="_blank">https://mlh.io</a> to find out more information.
We invite you to apply again next year. There are plenty of other hackathons this season, and it may not be too late to apply for those. Checkout <a href="https://mlh.io" target="_blank" rel="noopener">https://mlh.io</a> to find out more information.

Thank you for applying,<br>
- The {{hackathon_name}} Team
6 changes: 3 additions & 3 deletions db/seed_messages/questionnaire--rsvp_reminder.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,9 @@
<h1>Are you coming to {{hackathon_name}}?</h1>
<h3>Let us know if we should expect you there!</h3>
<p>
<a href="{{accept_rsvp_url}}" class="button" target="_blank">Yes, I will Attend &raquo;</a>
<a href="{{deny_rsvp_url}}" class="button" target="_blank">No, I Can't Attend &raquo;</a>
<a href="{{accept_rsvp_url}}" class="button" target="_blank" rel="noopener">Yes, I will Attend &raquo;</a>
<a href="{{deny_rsvp_url}}" class="button" target="_blank" rel="noopener">No, I Can't Attend &raquo;</a>
<br>
<small><i>Link not working? Go to <a href="{{rsvp_url}}">{{rsvp_url}}</a></i></small>
<small><i>Link not working? Go to <a href="{{rsvp_url}}" target="_blank" rel="noopener">{{rsvp_url}}</a></i></small>
</p>
</div>
2 changes: 1 addition & 1 deletion website/core/Footer.js
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ class Footer extends React.Component {
<div />
<div>
<h5>More</h5>
<a href={this.props.config.repoUrl} target="_blank">
<a href={this.props.config.repoUrl} target="_blank" rel="noopener">
GitHub Repo
</a>
<a
Expand Down
3 changes: 2 additions & 1 deletion website/pages/en/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,8 @@ class HomeSplash extends React.Component {
<p style={{ marginTop: '2em' }}>
<a
href="https://heroku.com/deploy?template=https://github.com/codeRIT/hackathon-manager"
target="blank"
target="_blank"
rel="noopener"
>
<img
src="https://www.herokucdn.com/deploy/button.svg"
Expand Down