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.

Sign up for GitHub

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

Jump to bottom

Fix security flaw when linkTarget option contains _blank #398

Open

AntoineReneleau wants to merge 4 commits into jonschlinkert:master from AntoineReneleau:patch-1

AntoineReneleau commented Oct 19, 2020

https://web.dev/external-anchors-use-rel-noopener/

Or should I just add a linkRel option ?
It might be useful to make the links nofollow

AntoineReneleau added 4 commits

October 19, 2020 19:30


          Fix security flaw when linkTarget option contains _blank

c7b80e8

https://web.dev/external-anchors-use-rel-noopener/


          refacto

1b422de


          Add test

f00c32a


          Update misc.js

424225b

coveralls commented Oct 20, 2020

Coverage increased (+0.003%) to 98.548% when pulling 424225b on AntoineReneleau:patch-1 into 58b6945 on jonschlinkert:master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet