-
Notifications
You must be signed in to change notification settings - Fork 54
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
manage proxies better #59
Comments
That's a good todo list :) I can't use pirate-get ATM because I can't reach proxybay.co/list.txt. I imagine doable to have a |
@vikstrous I'd also add to the list caching the last working proxy and trying that one first -> once I did a search and 10th proxy worked. I did another search seconds later and had to wait-out 9 requests fails -> maybe something like:
That would reduce tries on subsequest request to maximum 2 |
Yeah that would be nice. We'd have to write out a file somewhere and check it first.
…On December 7, 2017 2:56:17 PM EST, Olgied Kasprowicz ***@***.***> wrote:
@vikstrous I'd also add to the list caching the last working proxy and
trying that one first -> once I did a search and 10th proxy worked. I
did another search seconds later and had to wait-out 9 requests fails
-> maybe something like:
```pseudocode
if original pirate bay fails and has cached working proxy:
try that proxy
if that proxy fails:
go through the list (without the tried proxy)
```
That would reduce tries on subsequest request to maximum 2
Would you think of this as desired behavior?
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
#59 (comment)
--
Sent from my Android device with K-9 Mail. Please excuse my brevity.
|
We have to address a few issues:
We'll need some refactoring to make this happen and we really need proper testing once the codebase is testable.
I'd say this is blocked on #44 to avoid breaking anything in the process of adding these features.
The text was updated successfully, but these errors were encountered: