unitpay / php-sdk Goto Github PK

Здравствуйте. Не плохо было бы вынести UnitPay.php из глобального пространства имен. Это будет полезно в случае когда в проектах уже есть класс UnitPay используемый например для общих действий с разными ПС.

Предлагаю в функцию UnitPay::api добавить автоматическое добавление параметра ip пользователя

Правильно я понимаю, что тестовый ключ доступа не позволяет генерировать рабочую ссылку с помощью метода form (для адреса вида https://unitpay.money/pay/)?

Обновился на версию 2.0.3 - стали сыпаться ошибки Wrong signature.
Откатился на 2.0.2 - все ок.

UnitPay::checkHandlerRequest
Retrieving IP from $_SERVER['REMOTE_ADDR'] could lead to error while PROXY is being used on partner's site. For example: cloudflare.com.
It would be better to add optional $ip param to method and PROXY option in __constructor.

Есть мнение, что делать запросы по https через функцию file_get_contents - это не лучшая идея. Предлагаю вместо этого использовать что-то вроде

	public function httpGetContents($url) {
		$curl = curl_init();

		curl_setopt($curl, CURLOPT_URL, $url);
		curl_setopt($curl, CURLOPT_RETURNTRANSFER, true);

		$result = curl_exec($curl);
		curl_close($curl);

		return $result;
	}

after adding to composer.json in require "unitpay/php-sdk":"master"
on composer update having this error

Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - The requested package unitpay/php-sdk could not be found in any version, there may be a typo in the package name.

Когда подключал вашу систему, не мог получить параметры desc, sum, из-за того что в коде Unitpay.php, была обнаружена ошибка последовательности, то есть,
до изменения:

        $vitalParams = array(
            'account'  => $account,
            'currency' => $currency,
            'desc'     => $desc,
            'sum'      => $sum
        );
        $this->params = array_merge($this->params, $vitalParams);
 
        if ($this->secretKey) {
            $this->params['signature'] = $this->getSignature($vitalParams);
        }

        $this->params['locale'] = $locale;

после изменений:

       $vitalParams = array(
            'account'  => $account,
            'currency' => $currency,
            'desc'     => $desc,
            'sum'      => $sum
        );
 
        if ($this->secretKey) {
            $this->params['signature'] = $this->getSignature($vitalParams);
        }
 
        $this->params = array_merge($this->params, $vitalParams);
 
        $this->params['locale'] = $locale;

То есть поменял местами. Надеюсь исправите)

"require":{"unitpay/php-sdk":"master"}
Не будет работать.

Исправьте на
"require":{"unitpay/php-sdk":"dev-master"}

1. Домен, который
   $unitpay = new UnitPay($domain, $secretKey);
   вообще стоит назвать api domain или еще как, но только не Your domain, потому, что для клиентов your domain это их домен, домен магазина и никак не домен, к которому идут запросы на api unitpay

2. Бага
   Уж не знаю где, но судя по всему на сервере помимо params['signature'] где-то вписывается еще и params['sign']
   Наверно достаточно давно уже.
   Как вы думаете, работает ли проверка сигнатуры с текущим описанным в документации алгоритмом? Нет, очевидно не работает.
   Зато где-то в соседнем проекте было найдено
   public function getSignature(array $params, $method = null)
   {

•    unset($params['sign']); // вот эта строчка, добавление которой внезапно чинит проверку
    unset($params['signature']);
  

3. Ключик который secretKey оказывается не один, их три. проекта, api и тестовый api.
   внезапно, они оказываются одинаковыми в плане имени переменной и надо просто вот тут
   $unitpay = new UnitPay($domain, $secretKey);
   во втором параметре их подбирать по одному, чтобы понять где (в смысле при создании платежа или проверке) и какой (из трех) ставить
   оказалось, их надо ставить в одном месте один, в другом - другой. 6 вариантов, их всего лишь перебрать...

Это все бы починить. И в документации тоже, но
п2. Критичный, как наверно видно, те хотелось бы его в первую очередь поправить и влить, чтобы работало без костылей.