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

Fix BIGINT Page.Events.Console Bug #355

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

JavaScriptDude
Copy link

Prevent hanging of pyppeteer when code subscribes to Page.Events.Console event and a bigint is logged to the JS Console.

Regarding ticket: https://github.com/puppeteer/puppeteer/pull/4016/files

See Puppeteer PR: https://github.com/puppeteer/puppeteer/pull/4016/files

There may be other PRs that should be considered looking at Puppeteer 4016.

Prevent hanging of `pyppeteer` when code subscribes to  `Page.Events.Console` event and a bigint is logged to the JS Console.

Regarding ticket: https://github.com/puppeteer/puppeteer/pull/4016/files

See Puppeteer PR: https://github.com/puppeteer/puppeteer/pull/4016/files

There may be other PRs that should be considered looking at Puppeteer 4016.
@JavaScriptDude
Copy link
Author

JavaScriptDude commented Jan 24, 2022

As per Puppeteer PR 4016, execution_context.py _convertArgument() does not handle bigint values correctly. However, that issue is much lower severity and probably should be handled on a separate ticket.

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

1 participant