Skip to content
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

Update AngryCurl.class.php #7

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Ishodnikov
Copy link

В функции filter_alive_proxy при проверке прокси, через функцию request - проверяется не нужная прокси ,а любая. Добавляем проверку на существование опции CURLOPT_PROXY

@2naive
Copy link
Owner

2naive commented Aug 1, 2015

Добрый день!

Поясните, пожалуйста необходимость данного изменения?
Изначально предполагается, что при использования внешнего списка прокси-серверов, - приоритет у этого списка, а значит значение CURLOPT_PROXY для конкретного запроса, заданное через options игнорируется.

Спасибо

@Ishodnikov
Copy link
Author

Смотри, отследи цепочку:
https://github.com/2naive/AngryCurl/blob/master/classes/AngryCurl.class.php#L323
https://github.com/2naive/AngryCurl/blob/master/classes/AngryCurl.class.php#L387
https://github.com/2naive/AngryCurl/blob/master/classes/AngryCurl.class.php#L129

На 129 строке мы должны проверять конкретный адрес, который передали с 387 в последнем параметре $options, вместо этого мы проверяем любой адрес.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants