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

[patreon] Disable TLS 1.2 by default #2249

Merged
merged 1 commit into from
Jan 30, 2022
Merged

[patreon] Disable TLS 1.2 by default #2249

merged 1 commit into from
Jan 30, 2022

Conversation

shinji257
Copy link
Contributor

At the insistence of others, I updated the code a bit to disable TLS 1.2 by default on Patreon. It is still a configurable option.

Should have really done this on the last PR but didn't think about it at the time. Based on testing this should be good to go for Patreon. Now people don't have to disable it manually in the config for Patreon.

Sorry to mikf since I know my last PR just got accepted recently.

Disables TLS 1.2 on Patreon by default.
@mikf mikf merged commit 4c651f6 into mikf:master Jan 30, 2022
@shinji257
Copy link
Contributor Author

This closes issues #2182 #2060 #1922 #1807 #1117

@valdearg
Copy link

valdearg commented Feb 7, 2022

Were there any other changes required for this? I've updated to the very latest version and getting the capcha errors still.

[gallery-dl][debug] Python 3.10.0 - Windows-10-10.0.19044-SP0
[gallery-dl][debug] requests 2.26.0 - urllib3 1.26.7
[gallery-dl][debug] Starting DownloadJob for 'https://www.patreon.com/eeelyeee'
[patreon][debug] TLS 1.2 disabled.
[patreon][debug] Using PatreonCreatorExtractor for 'https://www.patreon.com/eeelyeee'
[urllib3.connectionpool][debug] Starting new HTTPS connection (1): www.patreon.com:443
[urllib3.connectionpool][debug] https://www.patreon.com:443 "GET /eeelyeee/posts HTTP/1.1" 403 None
[patreon][warning] Cloudflare CAPTCHA
[patreon][error] HttpError: '403 Forbidden' for 'https://www.patreon.com/eeelyeee/posts'

I've tried various ways of feeding in the cookies, including --cookies , cookies path in the config file, and just specifying the session_id in the config file.

I am able to login to patreon through the website on the same computer.

@shinji257
Copy link
Contributor Author

Nope. You just need the session cookie from Patreon like normal. You don't even need the __cf_bm cookie for Cloudflare anymore. I just checked it this morning and it works fine. In addition if you set a user agent previously to get around Cloudflare you can get rid of that. Just use the user agent that Gallery-DL uses by default.

@shinji257 shinji257 deleted the patch-2 branch February 7, 2022 16:17
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