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

[not ready] Fix scraper #17

Closed
wants to merge 6 commits into from
Closed

[not ready] Fix scraper #17

wants to merge 6 commits into from

Conversation

dawsbot
Copy link
Owner

@dawsbot dawsbot commented Mar 6, 2024

No description provided.

function openBrowser(): Promise<puppeteer.Browser> {
return puppeteer.connect({
browserWSEndpoint:
"ws://127.0.0.1:9222/devtools/browser/af1a102b-63b1-41f8-bb83-7e4f6d4d1f22",
Copy link
Owner Author

Choose a reason for hiding this comment

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

We are going to swap this out.

puppeteer-core will become playwright and we will also undo this browserWSEndpoint hack and just connect to the default (probably chrome).

The downside of this is that we will likely also need to do a login when on the Etherscan page immediately since the rate-limits are stricter for logged-out users

dawsbot and others added 3 commits March 6, 2024 09:24
* fix error parsing and add dynamic hostname generation\n\nadded error-parse.ts which contains a function taking in any js Error type and displays a user readable error that goes a little more in depth.\nin puppeteer-fetch-all.ts i imported the hostname function from the os library in js and put it in place for the static '127.0.0.1' previously used in openBrowser()

* revert readme.md

* revert readme

* Move from puppeteer to playwright

---------

Co-authored-by: dawsbot <dawsonbotsford@gmail.com>
@dawsbot dawsbot closed this Mar 12, 2024
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

2 participants