elastic / kibana

Your window into the Elastic Stack
https://www.elastic.co/products/kibana
Other
19.6k stars 8.21k forks source link

[Reporting/Dev] Remove setup code for Remote Debugger - No longer using this feature #101233

Closed tsullivan closed 1 week ago

tsullivan commented 3 years ago

Before upgrading Chromium to 8.0.0, a developer could see the screen of the headless browser when testing by adding:

xpack.reporting.capture.browsers.chromium.inspect: true

When the setting works, it shows the screen of the headless browser while a report is executing (screenshot from running 7.12 branch): image

After upgrading, something broke in how this is supposed to work: it no longer shows anything: image

elasticmachine commented 3 years ago

Pinging @elastic/kibana-reporting-services (Team:Reporting Services)

elasticmachine commented 3 years ago

Pinging @elastic/kibana-app-services (Team:AppServices)

tsullivan commented 3 years ago

Relevant Chrome DevTools Protocol documentation: https://chromedevtools.github.io/devtools-protocol/tot/Debugger/#method-enable

tsullivan commented 1 year ago

Let's go ahead and remove the code tied to attaching to a remote debugger for development

tsullivan commented 1 year ago

We can leave this issue open to track the work to remove the remote debugger setup.