Почему перестал работать код получения валют ЦБ?

Question

[artyak1121]

Сегодня на сайте появилась ошибка

failed to open stream: Redirection limit reached, aborting

как я понял, с сервера сильно много запросов было и сайт временно забанил Ip сервера.

Мой код:

public static function getActualCurs($rub) {
        $link = 'http://www.cbr.ru/scripts/XML_daily.asp';
        $str  = file_get_contents($link);

        preg_match('#<Valute ID="R01235">.*?.<Value>(.*?)</Value>.*?</Valute>#is', $str, $value);

        $usd = $value[1];

        $dollar = $rub/$usd;

        return round($dollar,3);
    }

Что можно в нем поменять, чтоб в дальнейшем не было проблем?

Comments

[Adamos]

https

[alex-1917]

все равно забанят.
кешировать на сутки надо.
тот же тупой бот от яндекса, заходя на страницу, тоже, наверное, вызывает запрос к ЦБ? и т.д.

Answer 1

[Siverius]

Возможно было бы неплохо не обращаться на сервер за файликом xml каждый раз. Мне представляется неплохим решением кэшировать этот xml - при обращении к методу getActualCurs проверять наличие сегодняшнего файла с курсами в папке на вашем сервере, если такового нет - обращаться на сервер ЦБР за новым, если есть - использовать его. Если я правильно понимаю, курсы меняются только раз в сутки, т.о.вы снизите частоту запросов до 1 в сутки.

Comments

[Adamos]

А вам кто-то сказал, что эта функция вызывается чаще, чем раз в сутки?
ТС ошибается в причинах, у него никаких санкций от ЦБ нет, в ошибке "redirection limit" просто потому, что он лезет на HTTP-адрес.

[Siverius]

Adamos, вы правы, мне никто не говорил, что эта функция вызывается чаще раза в сутки, я наивно предположил, что это очевидно.
Я проверил скрипт автора - если лезть на http, то ЦБР тоже отдаёт данные. Думаю, проблема всё же не в протоколе. Но, конечно же, могу и ошибаться.

[artyak1121]

Adamos, Siverius, вот прикол в том, что по http данные тоже получает функция

[Adamos]

Siverius,

если лезть на http, то ЦБР тоже отдаёт данные

Сначала происходит редирект на HTTPS, оттуда отдает. Судя по ошибке ТС - для начала стоит его устранить.

я наивно предположил, что это очевидно

Нет. Этот класс ничто не мешает использовать исключительно в скрипте, вызываемом по крону. У меня примерно такой же работает с ЦБ с тех пор, как заблочили Yahoo Finance.

P.S. Хотя ТС сам пишет про "много запросов с сервера". Да, пардон, тогда очевидно. Как и ваше предложение так не делать ;)

Answer 2

[Gic0]

тебя заблокировали за частое обращение.
варианта 2
1. кэшировать и обновлять только если прошло достаточно времени (насколько помню, там курс обнавляется в 12-14 часов)
2. использовать не ЦБ, а сайт cbr-xml-daily