Add rel tag to links

[juancarlosqr]

Hi,

For security reasons external links should be able to have a rel tag (.e.g. "noopener noreferrer"). More info here

I created a linkRel option to add the tag, similar to linkTarget

This is the first year I participate in the Hacktoberfest, so I'm open to any comments or feedback regarding my contribution 😉

Happy Hacktoberfest!

[blaiz]

This looks great! I also have a need for this prop. Hopefully this can be merged soon.

[benliger1987]

Any progress on merging this? Seems like a pretty crucial commit security wise

[fivetwelve]

Would be nice to be able to add optionally nofollow value in addition to those base ones.

[wchargin]

@rexxars: Friendly ping :-) if you get a chance to look at this, that’d
be lovely, since it enables fixing a security hole. It looks like the PR
still merges cleanly into origin/master, and the proposed patch looks
nice and clean.

[madhums]

Can anyone merge this and do a release? Seems like an important fix!

[fcsonline]

Can this be merged? Is something blocking it?

[juancarlosqr]

@fcsonline no blocker, the repo looks like is not maintained anymore :( any comment @rexxars?

[wooorm]

You can use a plugin for that: https://github.com/remarkjs/remark-external-links

[2manyvcos]

@wooorm That would be a preferable solution.
However, like stated in this comment, the suggested plugin sadly does not seem to work with react-markdown.

I just tried it but without success.

EDIT: NVM I just saw your other comment here

[wooorm]

there’s a PR merged that solves that which will be released soon

[adarnon]

Has anyone managed to get remark-external-links to work? The new version has been released, but it looks like the plugin just isn't run.

[MorelSerge]

@adarnon I haven't been able to on 5.0.3

[wassim]

remark-external-links still doesn't work... @wooorm please reopen the issue!

[statico]

As of August, 2021, remark-external-links seems to work fine.