If no Page.Request event handler is specified, await page.SetRequestInterceptionAsync(true) causes the browser to hang

Description

If no Page.Request += event handler is specified, await page.SetRequestInterceptionAsync(true); causes the browser to hang

Complete minimal example reproducing the issue

    public static async Task Do()
    {
        await new BrowserFetcher().DownloadAsync(BrowserFetcher.DefaultRevision);
        var browser = await Puppeteer.LaunchAsync(new LaunchOptions
        {
            Headless = false,
        });
        var page = (await browser.PagesAsync()).First();
        await page.SetRequestInterceptionAsync(true);
        await page.GoToAsync("https://www.google.com/");
    }

Expected behavior:

I would assume it shouldn’t hang just because there is no event handler defined

Actual behavior:

It hangs

Versions

• Which version of PuppeteerSharp are you using? 2.0.4
• Which .NET runtime and version are you targeting? 4.8

[KoalaBear84]

Well, I did also went down this rabbit hole and was lost for maybe an hour why it didn’t work anymore 😃

I wouldn’t mind that it throws and exception when SetRequestInterceptionAsync is set but no event handler is set. But maybe that could break something indeed. Just wanted to let you know that you are not alone 😃

[d668]

Compiler warning would be nice at least. Node is a constant pain, that’s why we use C# over here