-
-
Notifications
You must be signed in to change notification settings - Fork 757
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
Fixed entire application. Should work as intended now. #612
base: master
Are you sure you want to change the base?
Conversation
… loggingPrefs and improved error handling. Working as of 07-07-2024 by Leaske
Should be its original values
should be its orignal value, this was used for testing
Removed proxies, was used for testing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Appreciate the effort you put there, @leaskeg, though it seems like there are still some issues when I run it. It attempts to get the chrome driver automatically but fails to do so. I attached a screenshot of the error, including the written text. PS C:\Users\amind\OneDrive\Desktop\Projects\YouTube-Viewer> py -3.10 .\youtube_viewer.py
|
Could you attempt to download the chromedriver manually and add it to the folder? Much like below? I will attempt to work on the automatic version. But for now, this should be working. Fingers crossed. |
I manually inserted a ChromeDriver and gave it a fixed path using For the time being, I installed ChromeDriver and called it
Replaced the following lines at line 113:
It works so far. But yeah, it should automatically download Chrome drivers seamlessly. I have other scripts currently using undetected Selenium that always download the latest Chrome driver correctly, so the issue lies with how it is implemented now. Note I am using Windows. |
I will attempt to make the automatic download work sometime next week :) Glad it works manually tho. |
Thanks, @leaskeg, for your efforts! And I appreciate what you have done to bring the project back to life. I am curious about your current fixes. Are the views still active on your analytics? |
They are actually! Tried with 100 views from webshares proxies. They counted. |
@leaskeg And if I get " |
I will get back to you on that. Its possible that it detects the browser as being automated which is not ideal, because youtube would be able to detect the views as bots. But that would indicate a problem with the undetected-chromedriver im using. I will look it over! |
I have fixed the detection issue. I will attempt to fix the automatic chromedriver mentioned above before releasing. And thank you for all the feedbacks. It really helps, you can only test so much locally 👍 |
This version should hopefully fix the issues people are mentioning here in the comments. I tested locally and it downloads the chromedriver automatically, and its also undetected. Test it out and let me know |
It works, ( undetected False) however it started triggering a few other issues :) inside chrome window: From the python: I'll continue test though :) |
I really appreciate your update. It works fine on Windows, but I’m having issues with Linux. When testing using Google Colab, it’s not working as expected. Thanks! ChromeDriver 126.0.6478.126 (d36ace6122e0a59570e258d82441395206d60e1c-refs/branch-heads/6478@{#1591}) |
Summary
This pull request addresses an issues where the repo is not working anymore. Tested locally and it works for me as it did before.
Changes
loggingPrefs
capability from the WebDriver initialization since it was causing an issue with the new version.loggingPrefs
capability was causing aninvalid argument: unrecognized capability
error. Removing this capability resolves the issue and allows multiple workers to initialize correctly.Testing
I think thats pretty much it. Works like before - tested with proxies from webshare
Please review and merge this to fix the application