iw4p / proxy-scraper

scrape proxies from more than 5 different sources and check which ones are still alive
MIT License
492 stars 126 forks source link

add support for proxy checker socks4 and socks5 #37

Closed FosterG4 closed 4 months ago

FosterG4 commented 4 months ago

add support for proxy checker socks4 and socks5

iw4p commented 4 months ago

Hi! thank you for your contribution! Can you please run flake8 . to make sure there's no error to fix the checks?

FosterG4 commented 4 months ago

Hi! thank you for your contribution! Can you please run flake8 . to make sure there's no error to fix the checks?

I have running flake8.exe . and flake8.exe --count . do I have to re-push?

iw4p commented 4 months ago

Can you see the failed checks to make sure?

Sent from Proton Mail for iOS

On Tue, Apr 23, 2024 at 1:50 PM, Dikky Hardian @.***(mailto:On Tue, Apr 23, 2024 at 1:50 PM, Dikky Hardian < wrote:

Hi! thank you for your contribution! Can you please run flake8 . to make sure there's no error to fix the checks?

I have flake8.exe running. and flake8.exe --count . do I have to re-push?

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.Message ID: @.***>

FosterG4 commented 4 months ago

Can you see the failed checks to make sure? Sent from Proton Mail for iOS On Tue, Apr 23, 2024 at 1:50 PM, Dikky Hardian @.(mailto:On Tue, Apr 23, 2024 at 1:50 PM, Dikky Hardian < wrote: > Hi! thank you for your contribution! Can you please run flake8 . to make sure there's no error to fix the checks? I have flake8.exe running. and flake8.exe --count . do I have to re-push? — Reply to this email directly, [view it on GitHub](#37 (comment)), or unsubscribe. You are receiving this because you commented.Message ID: @.>

this is strange I tried running flake8 on my local and there's no error but on github there is image

iw4p commented 4 months ago

Are you using this configuration exactly?

https://github.com/iw4p/proxy-scraper/blob/master/.flake8

Sent from Proton Mail for iOS

On Tue, Apr 23, 2024 at 2:29 PM, Dikky Hardian @.***(mailto:On Tue, Apr 23, 2024 at 2:29 PM, Dikky Hardian < wrote:

Can you see the failed checks to make sure? Sent from Proton Mail for iOS On Tue, Apr 23, 2024 at 1:50 PM, Dikky Hardian @.(mailto:On Tue, Apr 23, 2024 at 1:50 PM, Dikky Hardian <[wrote: > Hi! thank you for your contribution! Can you please run flake8 . to make sure there's no error to fix the checks? I have flake8.exe running. and flake8.exe --count . do I have to re-push? — Reply to this email directly, [view it on GitHub](]())#37 (comment)), or unsubscribe. You are receiving this because you commented.Message ID: @.>

this is strange I tried running flake8 on my local and there's no error but on github there is image.png (view on web)

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.Message ID: @.***>

FosterG4 commented 4 months ago

Are you using this configuration exactly? https://github.com/iw4p/proxy-scraper/blob/master/.flake8 Sent from Proton Mail for iOS On Tue, Apr 23, 2024 at 2:29 PM, Dikky Hardian @.(mailto:On Tue, Apr 23, 2024 at 2:29 PM, Dikky Hardian < wrote: > Can you see the failed checks to make sure? Sent from Proton Mail for iOS > > On Tue, Apr 23, 2024 at 1:50 PM, Dikky Hardian @.(mailto:On Tue, Apr 23, 2024 at 1:50 PM, Dikky Hardian <[wrote: > Hi! thank you for your contribution! Can you please run flake8 . to make sure there's no error to fix the checks? I have flake8.exe running. and flake8.exe --count . do I have to re-push? — Reply to this email directly, [view it on GitHub](]())[#37 (comment)](#37 (comment))), or unsubscribe. You are receiving this because you commented.Message ID: @.> this is strange I tried running flake8 on my local and there's no error but on github there is image.png (view on web) — Reply to this email directly, [view it on GitHub](#37 (comment)), or unsubscribe. You are receiving this because you commented.Message ID: @.>

found it i need run isort.exe .\proxyChecker.py image

iw4p commented 4 months ago

Yay 🥳 merged. Thank you.