-
Notifications
You must be signed in to change notification settings - Fork 124
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
Sources #31
Comments
Hi |
why don't you add a regex for ip:port instead, saves you the trouble. |
Because I'm not just good at regex, :D kidding, but having a well-structured scraper is better than just using regex in my prospective. Regex is my last solution always. Do you think regex is a better option? |
Well, sure why not? it can barely mess with your well structured code as it only consists of 5-6 lines of extra code (for simpler terms), without requiring any fancy structuring to the code. |
It can be a good solution, but I have to test it. If you have time you can do a sample on your fork maybe? And we can check the time for doing benchmarks. |
sure thing, I'll let you know once i have done it |
Thank you! |
Hii there I used this scraper and it was good but it will be more better if you please provide option to manually add sources for proxies.
Like different sources files http,socks4,socks5 so that we can have more proxies from our own desired url/website.
Please look into this.
The text was updated successfully, but these errors were encountered: