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

Cannont read property 'outerHTML' of undefined on #465

Closed
molant opened this issue Aug 29, 2017 · 1 comment
Closed

Cannont read property 'outerHTML' of undefined on #465

molant opened this issue Aug 29, 2017 · 1 comment

Comments

@molant
Copy link
Member

molant commented Aug 29, 2017

URL: https://sonarwhal.com
Connector: jsdom
Formatter: codeframe

Downloading https://cdn.jsdelivr.ne … ch.js/2/docsearch.min.jsTypeError: Cannot read property 'outerHTML' of undefined
    at JSDOMAsyncHTMLDocument.pageHTML (C:\Users\antonmo\AppData\Local\nvs\node\8.4.0\x64\node_modules\@sonarwhal\sonar\dist\src\lib\connectors\jsdom\jsdom-async-html.js:19:58)
    at Object.exports.findElementLocation (C:\Users\antonmo\AppData\Local\nvs\node\8.4.0\x64\node_modules\@sonarwhal\sonar\dist\src\lib\utils\location-helpers.js:44:46)
    at Object.exports.findProblemLocation (C:\Users\antonmo\AppData\Local\nvs\node\8.4.0\x64\node_modules\@sonarwhal\sonar\dist\src\lib\utils\location-helpers.js:105:43)
    at RuleContext.report (C:\Users\antonmo\AppData\Local\nvs\node\8.4.0\x64\node_modules\@sonarwhal\sonar\dist\src\lib\rule-context.js:71:49)
    at validate (C:\Users\antonmo\AppData\Local\nvs\node\8.4.0\x64\node_modules\@sonarwhal\sonar\dist\src\lib\rules\no-disallowed-headers\no-disallowed-headers.js:42:31)
    at Immediate.setImmediate [as _onImmediate] (C:\Users\antonmo\AppData\Local\nvs\node\8.4.0\x64\node_modules\@sonarwhal\sonar\dist\src\lib\sonar.js:133:46)
    at runCallback (timers.js:781:20)
    at tryOnImmediate (timers.js:743:5)
    at processImmediate [as _immediateCallback] (timers.js:714:5)
(node:17152) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 5): TypeError: Cannot read property 'outerHTML' of undefined
(node:17152) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
(node:17152) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 6): TypeError: Cannot read property 'outerHTML' of undefined
...
| https://fonts.googleapi … rrat:200,300,400,500,600 downloaded(node:17152) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 22): TypeError: Cannot read property 'outerHTML' of undefined
(node:17152) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 24): TypeError: Cannot read property 'outerHTML' of undefined
...
/ https://sonarwhal.com/images/iceberg-left.svg downloaded(node:17152) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 36): TypeError: Cannot read property 'outerHTML' of undefined
(node:17152) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 38): TypeError: Cannot read property 'outerHTML' of undefined
...
- https://sonarwhal.com/images/narwhal.svg downloaded(node:17152) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 54): TypeError: Cannot read property 'outerHTML' of undefined
(node:17152) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 56): TypeError: Cannot read property 'outerHTML' of undefined
...

Doesn't repro on cdp.
Changing the formatter to stylish produces also an invalid output but not as bad looking as the previous one.

@molant
Copy link
Member Author

molant commented Aug 30, 2017

Can't repro it any more so closing for now. Will reopen if I can reproduce it consistently.

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

No branches or pull requests

1 participant