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

Restored functionality for download bytes method #1174

Merged
merged 1 commit into from
Jul 29, 2024

Conversation

bendavidsteel
Copy link
Contributor

Just need a few specific headers, and the cookies from the original page fetch.

I couldn't get the evaluate JS method working so just resorted to requests, lmk if the JS is super necessary for this.

Copy link
Owner

@davidteather davidteather left a comment

Choose a reason for hiding this comment

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

LGTM 🔥 PRs as usual

@davidteather davidteather changed the base branch from main to v6.4.0 July 29, 2024 19:17
@davidteather davidteather merged commit 9bc381e into davidteather:v6.4.0 Jul 29, 2024
0 of 2 checks passed
davidteather added a commit that referenced this pull request Jul 29, 2024
* Fix user.videos crawling issue (#1141)
* added support for statsV2 (#1143)
* Restored functionality for download bytes method (#1174)
* [PlayList] Add playlist to the user scrapy api (#1177)
* bump to 6.4.0

---------

Co-authored-by: Phat Luu Huynh <[email protected]>
Co-authored-by: ekorian <[email protected]>
Co-authored-by: Ben Steel <[email protected]>
Co-authored-by: wu5bocheng <[email protected]>
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.

2 participants