webhintio / hint

💡 A hinting engine for the web

https://webhint.io/

Apache License 2.0

3.62k stars 670 forks source link

Open pmd-bgermain opened 4 years ago

pmd-bgermain commented 4 years ago

🐛 Bug report

I installed webhint locally on my computer and when I launch a hint test with hint https://www.voici.fr the test never ends.

Using default configuration (or edited one)

No debug file generated until I force exit (and get a exit code as debug output)

While tracking the issue locally, I found that removing the call to polyfill.io and webpack builded js gets webhint to work normally again.

I hope that will help

molant commented 4 years ago

@pmd-bgermain what's the configuration you are using? Are you using the default one (and thus running chrome?)

Thanks!

pmd-bgermain commented 4 years ago

@molant I am using the default configuration (I tried to create a custom one without success)

molant commented 4 years ago

This might be related to #3201

pmd-bgermain commented 4 years ago

More dive into the problem leads us to that resource :

<link rel="preload" href="https://tra.scds.pmdstatic.net/pmd-browsertools/1.7.44/browsertools.js" as="script">

when removed and replaced with <script> tags the tests can go to the end.

We found a workaround with waitUntil: loaded but we aren't confident in using that option.

molant commented 4 years ago

I don't know the internal of the page, but for reference this is what each option does for waitUntil:

load - consider navigation to be finished when the load event is fired.
domcontentloaded - consider navigation to be finished when the DOMContentLoaded event is fired.
networkidle0 - consider navigation to be finished when there are no more than 0 network connections for at least 500 ms.
networkidle2 - consider navigation to be finished when there are no more than 2 network connections for at least 500 ms.

Are you experiencing the same problems with the browser extension?

Hopefully we will fix the current problems we have with CI to properly look into this.