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
JSX Support (Was: Cannot read property 'innerHTML' of undefined) #86
Comments
@Yserz Do you know which version introduced this? Was it 3.0.2 or earlier? I'm guessing it is 3.0.2 from the looks of the error. I think it is potentially an issue with the configuration, please can you post the no-unsanitized parts of your configuration(s)? |
I'm seeing the same error on 3.0.2. No issues on 3.0.1. I'm using the following configuration type in my eslintrc file: |
We have the same configuration as @audishos. Can't really say with which version it appeared, sorry. |
I've just been trying to reproduce this with a simple test directory, with just an .eslintrc file, and a couple of js files, plus another in a sub-directory. Unfortunately I can't reproduce it. Can someone try and track this down, or see what might be triggering it? |
@Standard8 Unfortunately I can't reproduce this anymore. But I would like to add that we are using react and react-intl where react-intl is making use of |
I'm really out of touch with the react ecosystem, so it seems like I can't help you reproduce your error. If you can't or won't give us better steps to reproduce, I'm afraid I'll have to close this issue. |
The error message and debug info don't appear to give much information. I'm using VS Code with the eslint plugin and the error pops up on every file with a react component that uses dangerouslySetInnerHTML. |
As I said before, I'm really, really out of touch with the react ecosystem and everything that is JSX. Could you come up with a minimal example and instructions on how to reproduce? I'd still be happy to take a look. |
I was able to trigger the error with the following file.
|
Thanks @audishos. I think we're getting closer. |
Sure thing. Here's the full file.
|
Sorry, I couldn't follow up sooner. "extends": [
"eslint:recommended",
"plugin:react/recommended",
"plugin:jsx-a11y/recommended",
"plugin:no-unsanitized/DOM", // <-- here
"plugin:security/recommended",
], What is Err, I'm wrong. it's been a while since I looked at this code 😓 |
Hey,
just wanted to let you know that we observed the following error while using this plugin in version
3.0.2
. I'm not sure how to get more information about this issue.The text was updated successfully, but these errors were encountered: