Skip to content
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

Replace eslint-plugin-jam3 #74

Open
ben-polinsky opened this issue Jun 20, 2024 · 0 comments
Open

Replace eslint-plugin-jam3 #74

ben-polinsky opened this issue Jun 20, 2024 · 0 comments
Labels
good first issue Good for newcomers

Comments

@ben-polinsky
Copy link
Collaborator

ben-polinsky commented Jun 20, 2024

Aside from having a terrible package name, this library is quite outdated. We've run into an issue with its use of requireindex which also is quite outdated. I'm sure we can find some new rules (or god forbid catch these issues in PR) which do the same as:

  • forbid-methods: By default, this rule aims to limit the use of React un-prefixed deprecated lifecycle methods:
  • no-sanitizer-window-location: This rule aims to avoid Address DOM XSS attack. Using window.location improperly can cause javascript to trigger DOM based XSS attacks. Since window.location is readable and writable, it requires paying attention to what data is being assigned. Please find examples below as bad and best practices.
  • no-sanitizer-with-danger: This rule aims to prevent Cross-Site Scripting (XSS) attacks from hackers (using dangerousSetInnerHTML)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant