[Reporting] Network Policy: Do not throw from the intercept handler (#75105)

tsullivan · elasticmachine · tsullivan · commit dab9e7305b55 · 2020-08-18T13:31:04.000-07:00
Co-authored-by: Elastic Machine &lt;elasticmachine@users.noreply.github.com&gt;
diff --git a/x-pack/plugins/reporting/server/browsers/chromium/driver/chromium_driver.ts b/x-pack/plugins/reporting/server/browsers/chromium/driver/chromium_driver.ts
@@ -303,7 +303,7 @@ export class HeadlessChromiumDriver {
       if (!allowed || !this.allowRequest(interceptedUrl)) {
         this.page.browser().close();
         logger.error(getDisallowedOutgoingUrlError(interceptedUrl));
-        throw getDisallowedOutgoingUrlError(interceptedUrl);
+        return;
       }
     });
 

Original file line number	Diff line number	Diff line change
`@@ -303,7 +303,7 @@ export class HeadlessChromiumDriver {`
`303`	`303`	`if (!allowed \|\| !this.allowRequest(interceptedUrl)) {`
`304`	`304`	`this.page.browser().close();`
`305`	`305`	`logger.error(getDisallowedOutgoingUrlError(interceptedUrl));`
`306`		`- throw getDisallowedOutgoingUrlError(interceptedUrl);`
	`306`	`+ return;`
`307`	`307`	`}`
`308`	`308`	`});`
`309`	`309`