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

[Bug] Websites using prototype library crash the browser extension #3201

Open
8 of 11 tasks
ldevernay opened this issue Oct 29, 2019 · 2 comments
Open
8 of 11 tasks

[Bug] Websites using prototype library crash the browser extension #3201

ldevernay opened this issue Oct 29, 2019 · 2 comments

Comments

@ldevernay
Copy link

🐛 Bug report

Description

Audited website is : https://www.observatoire-dpe.fr/index.php/
Firefox plugin crashes totally.
Online webhint crashes too, but only on accessibility.

Environment

webhint configuration

Categories:

  • Accessibility
  • Compatibility
  • Performance
  • Pitfalls
  • PWA
  • Security

Your target browsers:

  • Recommended settings
  • Custom:

Ignored resources:

    • None
    • Different origin
    • Custom:

Debug output

hint #1: Error in webhint analyzing this hint
https://webhint.io/scanner/6fa660a0-2ce2-4127-aba4-674aa7945d69#hint-axe/aria-1

@molant molant changed the title [Bug] Bug description [Bug] observatoire-dpe.fr crashes Firefox extension Oct 29, 2019
@molant molant added this to the 1911-1 milestone Oct 29, 2019
@antross
Copy link
Member

antross commented Oct 29, 2019

Thanks for the report @ldevernay!

Digging into this a bit I see the browser extension seems to fail when trying to snapshot the DOM:

"TypeError: Cannot read property 'attribs' of undefined
    at snapshot (<anonymous>:131:32)
    at https://www.observatoire-dpe.fr/js/prototype-1.6.0.2.js:635:20
    at https://www.observatoire-dpe.fr/js/prototype-1.6.0.2.js:595:9
    at Array.forEach (<anonymous>)
    at Array.each (https://www.observatoire-dpe.fr/js/prototype-1.6.0.2.js:594:12)
    at Array.collect (https://www.observatoire-dpe.fr/js/prototype-1.6.0.2.js:634:10)
    at snapshot (<anonymous>:129:45)
    at https://www.observatoire-dpe.fr/js/prototype-1.6.0.2.js:635:20
    at https://www.observatoire-dpe.fr/js/prototype-1.6.0.2.js:595:9
    at Array.forEach (<anonymous>)"

Given the library in the callstack I'm suspecting both webhint's browser extension DOM snapshot utility and axe-core (which powers webhint's accessibility checks) are being affected by overrides to native APIs in the page.

@molant molant changed the title [Bug] observatoire-dpe.fr crashes Firefox extension [Bug] Websites using prototype library crash the browser extension Oct 29, 2019
@molant molant removed this from the 1911-1 milestone Nov 11, 2019
@molant
Copy link
Member

molant commented Nov 11, 2019

Seems that #3262 might be related to this as well.

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

3 participants