load never fires when setting setRequestInterceptionEnabled to true (and disable JS)

https://github.com/GoogleChrome/puppeteer/pull/565 fixed most cases I found with pages never reporting the load event in puppeteer, but I have here found another url that breaks in the same way - only after calling setRequestInterceptionEnabled(true).

Environment

Puppeteer: 0.11.0 Node: 7.10.1. OS: macOS Sierra

Reduced test case (updated)

Js disabled + request interception => never loads

This is the real problem that relates to this issue. Originally when I posted I missed the critical detail that JS needs to be disabled for the load to never happen:

const puppeteer = require('puppeteer');
(async() => {
const browser = await puppeteer.launch({
      ignoreHTTPSErrors: true,
      args: ['--disable-setuid-sandbox', '--no-sandbox']
    })
const page = await browser.newPage()

await page.setJavaScriptEnabled(false)
await page.setRequestInterceptionEnabled(true)
page.on('request', intercepted => intercepted.continue())

await page.goto('https://apartmentsdoralfl.com/')
console.log('page load DONE') // never happens
browser.close();
})();

**NOTE that the url I’m using in this test case it not the one I had when I originally raised the issue - the original url now works (and along with it about 80% of the problematic url’s I had) - but the one listed here does not.

---

Thanks for all the hard work on puppeteer, it’s shaping up for me to be able to launch a new major version of Penthouse with it replacing phantomjs. 👍

[addyosmani]

@aslushnikov Thanks for taking a look 👋 If it helps at all with prioritization, @pocketjoso’s work to use Puppeteer in Penthouse is important for the ecosystem as tools like critical are relying on it for a shift over to Chrome headless.

[aslushnikov]

@pocketjoso ok thanks, I can repro now, will take a look.