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

v0.2.91 #26

Merged
merged 6 commits into from
Dec 19, 2024
Merged

v0.2.91 #26

merged 6 commits into from
Dec 19, 2024

Conversation

D4Vinci
Copy link
Owner

@D4Vinci D4Vinci commented Dec 19, 2024

What's changed

  • Fixed a bug where the logging fetch logging sentence was showing in the first request only.
  • The default behavior for Playwright API while browsing a page is returning the first response that fulfills the load state given to the goto method ["load", "domcontentloaded", "networkidle"] so if a website has a wait page like Cloudflare's one that redirects you to the real website afterward, Playwright will return the first status code which in this case would be something like 403. This update solves this issue for both PlaywrightFetcher and StealthyFetcher as both are using Playwright API so the result depends on Playwright's default behavior no more.
  • Added support for proxies that use SOCKS proxies in the Fetcher class.
  • Fixed the type hint for the wait_selector_state argument so now it will show the accurate values you should use while auto-completing.

Note

A friendly reminder that maintaining and improving Scrapling takes a lot of time and effort which I have been happily doing for months even though it's becoming harder. So, if you like Scrapling and want it to keep improving, you can help by supporting me through the Sponsor button.

This will return the real status response after redirection for websites that have Cloudflare instead of Playwright's default behavior.
This will return the real status response after redirection for websites that have Cloudflare instead of Playwright API's default behavior.
@D4Vinci D4Vinci merged commit ee59914 into main Dec 19, 2024
5 checks passed
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.

1 participant