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 ixigua download failed #3001

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from
Open

fix ixigua download failed #3001

wants to merge 3 commits into from

Conversation

dxsooo
Copy link

@dxsooo dxsooo commented Mar 2, 2023

It seems that the regex should update

update regex
add ixigua test
update regex
@soimort-bot
Copy link
Collaborator

Hello @dxsooo,
Thanks for the Pull Request. We ❤️ our contributors!
Please wait for one of our human maintainers to review your patches. This may take a few days to weeks. Also, please understand that although your Pull Request may or may not be eventually merged, we value all contributions equally.

祝您健康!

@@ -33,7 +33,7 @@ def ixigua_download(url, output_dir='.', merge=True, info_only=False, stream_id=
_cookies.append(c.strip().split(' ')[0])
headers['cookie'] += ' '.join(_cookies)

match_txt = match1(html, r"<script id=\"SSR_HYDRATED_DATA\" nonce=\"argus-csp-token\">window._SSR_HYDRATED_DATA=(.*?)<\/script>")
match_txt = match1(html, r"<script id=\"SSR_HYDRATED_DATA\" nonce=.*?>window._SSR_HYDRATED_DATA=(.*?)<\/script>")

Choose a reason for hiding this comment

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

It just works ! Thank you !

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.

3 participants