-
Notifications
You must be signed in to change notification settings - Fork 48
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
cloudscraper, update csrf_regexp, auth dict #21
Conversation
@haochi any chance you can review/approve this PR? Many thanks for providing this library! |
+1 |
1 similar comment
+1 |
I think |
Also there's a typo in line 60 of
|
I want to use this API and open the account every morning using Selenium automatically. Using API, I wan to download all transactions, included just showed up in the account (but not fully posted) and balances of all the accounts. I would like to know few things.
Thanks in advance. |
I am unable to login using the latest pull #21 . |
@UnusualPi , I also encountered this issue and realized just updating the User-Agent will satisfy CloudFlare's checks (see #24 ). I'm sure there are many other ways to address, but the User-Agent tweak avoids adding a dependency to the library. |
I like this better than my solution, much cleaner without a new dependency. I assume the user agent validation is the primary reason cloud flare would catch it so should work for most. |
I'm experiencing the same errors in the same situations. |
Looks like @UnusualPi deleted their fork of this repo. I made the same changes they made in my own repo here: https://github.com/banool/personalcapital. |
handling for cloudflare bot protection
update csrf_regexp for new page format
update auth payloads for new dict structure