Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add experimental browser.debugInfo #11748

Merged
merged 1 commit into from Jan 25, 2024
Merged

Conversation

OrKoN
Copy link
Collaborator

@OrKoN OrKoN commented Jan 24, 2024

This is an experimental method that provides debugging information to help debug issues with Puppeteer.
Currently, it only provides the stacktraces for pending protocol calls.

Note: the test failure is unrelated and tracked in #11749 It's revealed by the change in sharding caused by introduction of a new spec.

@OrKoN OrKoN force-pushed the orkon/getPendingProtocolErrors branch 2 times, most recently from 3dc8604 to 296d8c4 Compare January 25, 2024 07:03
@OrKoN OrKoN changed the title chore: experiment to see pending CDP calls feat: add browser.debugInfo Jan 25, 2024
@OrKoN OrKoN changed the title feat: add browser.debugInfo feat: add experimental browser.debugInfo Jan 25, 2024
@OrKoN OrKoN marked this pull request as ready for review January 25, 2024 07:04
@OrKoN OrKoN force-pushed the orkon/getPendingProtocolErrors branch 3 times, most recently from 3a0f449 to 1c43d44 Compare January 25, 2024 08:57
Copy link
Collaborator

@Lightning00Blade Lightning00Blade left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM + text change

* @internal
*/
getPendingProtocolErrors(): Error[] {
const result: Error[] = [];
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit: Simplify to a map?

packages/puppeteer-core/src/api/Browser.ts Outdated Show resolved Hide resolved
@OrKoN OrKoN force-pushed the orkon/getPendingProtocolErrors branch 2 times, most recently from 3394d60 to 7317010 Compare January 25, 2024 09:39
Copy link
Contributor

@jrandolf jrandolf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm % comments

This is an experimental method that would provide
debugging information to help debug issues with Puppeteer.
Currently, it only provides the stacktraces for pending protocol
calls.
@OrKoN OrKoN force-pushed the orkon/getPendingProtocolErrors branch from 7317010 to d3a2646 Compare January 25, 2024 14:48
@OrKoN OrKoN enabled auto-merge (squash) January 25, 2024 14:49
@OrKoN OrKoN merged commit f88e1da into main Jan 25, 2024
38 checks passed
@OrKoN OrKoN deleted the orkon/getPendingProtocolErrors branch January 25, 2024 14:57
@release-please release-please bot mentioned this pull request Jan 25, 2024
@Lightning00Blade
Copy link
Collaborator

We miss adding this to the website, can you file an issue or PR for that?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants