-
Notifications
You must be signed in to change notification settings - Fork 142
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
Yahoo Finance Premium instituting recaptcha #254
Comments
Is this what's happening in you code: |
Attach a screenshot. |
can you add some visual files when you are getting error and when you are getting a normal expected result |
some Questions: the login is done in base.py i think whenver a user logins it this part of the code must be executed 3)how can i debug at my local where can i get my Username password |
Is it possible to get your email id so that i can mesage you directly. |
Cause of the error might be that in username page contains multiple names with 'id=login-username' because of it was not able to login and resulting in error and providing us message:"Unable to login and/or retrieve the appropriate cookies. This is " most likely due to Yahoo Finance instituting recaptcha, which " this package does not support." i changed the method of finding the element to By.XPATH Signed-off-by: Samir Gorai <[email protected]>
Possible FIX can yo look at #255 |
Hello @dpguthrie @me1029134 i TESTED THE CODE with my changes #255 import yahooquery as yq AND THE RESULT WAS DevTools listening on ws://127.0.0.1:64734/devtools/browser/41a2456b-89ec-4df2-b6a5-d65774e7c308 CAN YOU CHECK ONCE AT YOUR SETUP WITH YOUR id |
Unfortunately after adding this line: |
@me1029134 how can i get the build please after my changes. |
I am able to login into login.yahoo.com using the following script """ while(1): time.sleep(5) I think the problem is with I can also see that it was modified in last commit. |
@me1029134 @dpguthrie can you PLease check once i have made some changes and commited Thank you |
I believe the method we are trying to do is pull the cookies from a chrome log in session and load them into the Selenium session. |
I have a prototype fix that seems to work for me. I noticed if I put a 20 second wait after the login and before any of the pulls, it seems to not get hung up for some reason. I added that and I added just saving the entire session after a good login. It would be better if you could just pass in the cookies / session, that seems like the correct way to do it. Here is the fix that worked for me at least:
|
@dpguthrie Do you have a high level design any document/image to understand your library? |
@samirgorai Nope, sorry. |
Describe the bug
I believe there is some kind of recaptcha problem. It's not on all the request though maybe like half of them. Below is my error.
DevTools listening on ws://127.0.0.1:63373/devtools/browser/661f6e71-8cf3-4067-bbfe-3966923a90ab
[1230/140638.380:ERROR:gl_utils.cc(412)] [.WebGL-00001D8400E82200]GL Driver Message (OpenGL, Performance, GL_CLOSE_PATH_NV, High): GPU stall due to ReadPixels
[1230/140640.413:ERROR:gl_utils.cc(412)] [.WebGL-00001D84002B3F00]GL Driver Message (OpenGL, Performance, GL_CLOSE_PATH_NV, High): GPU stall due to ReadPixels
Unable to login and/or retrieve the appropriate cookies. This is most likely due to Yahoo Finance instituting recaptcha, which this package does not support.
To Reproduce
Steps to reproduce the behavior:
query = yq.Ticker('ASGTF', username= "UserEmail", password="PW")
I get:
{'ASGTF': 'User is not logged in'}
Seems to do it about half the time and different tickers or pulling the same ticker multiple times.
Expected behavior
I'm expecting to get p_all_financial_data. I see it when I'm login in.
I verified I get it when I am logged in.
Screenshots
Desktop (please complete the following information):
Smartphone (please complete the following information):
Additional context
Comment from dpguthrie describing the problem and solution probably a little more.
#251 (comment)
I thought I had it fixed but it was not.
The text was updated successfully, but these errors were encountered: