You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The problem with noreferrer is that it hides the referrer information to the site you link to, which can create problems (e.g. commercial traffic arrangements). But at the root, it's not the default that the HTML spec implemented, it makes eslint opinionated.
At one point in the future, this allowReferrer: false default will no longer make sense because Internet Explorer use would be so low that the cons outweigh the pros. Hence this issue is here to keep track of this.
Expected Behavior
eslint-plugin-react version
7.33.2
eslint version
8.56.0
node version
v18.18.2
The text was updated successfully, but these errors were encountered:
The safe default value of this option is still false, because that's the safest one, so I don't anticipate ever wanting to change it here. Certainly most downstream users will be able to set it to true if they want, but since the vast majority of users wouldn't want to expose user information because they wouldn't have commercial agreements, it's unlikely to be that valuable a change except at the application level.
Is there an existing issue for this?
Description Overview
Today
allowReferrer
default value isfalse
. This is because the rule is required for Internet Explorer (IE). https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/jsx-no-target-blank.md#rule-optionsThe problem with noreferrer is that it hides the referrer information to the site you link to, which can create problems (e.g. commercial traffic arrangements). But at the root, it's not the default that the HTML spec implemented, it makes eslint opinionated.
At one point in the future, this
allowReferrer: false
default will no longer make sense because Internet Explorer use would be so low that the cons outweigh the pros. Hence this issue is here to keep track of this.Expected Behavior
eslint-plugin-react version
7.33.2
eslint version
8.56.0
node version
v18.18.2
The text was updated successfully, but these errors were encountered: