Skip to content

Update AngryCurl.class.php#7

Open
Ishodnikov wants to merge 1 commit into2naive:masterfrom
Ishodnikov:patch-1
Open

Update AngryCurl.class.php#7
Ishodnikov wants to merge 1 commit into2naive:masterfrom
Ishodnikov:patch-1

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