application_token

[makapoh]

В функции getAppSettings() проверяется наличие application_token и если его нет, то выдается ошибка "error install app, pls install local application"

Проблема в том, что после каждой переустановки локального приложения в Битрикс24 все нормально работает где-то 3 дня, а потом application_token почему-то куда-то исчезает из settings.json

При этом, насколько я понимаю, application_token вообще нигде не используется. Тогда зачем эта проверка?)
Если закомментировать строку "!empty($arData[ 'application_token' ]) &&" то все прекрасно работает)

[anton-zaharov]

Сори, коллега, но проблема в другом.

каждый час апи требует рефреша токена.
Если после протухания токена пойдут почти одновременно два запроса к апи, то оба они будут отправлены за новыми токенами и только один из них будет верным. И если в этой гонке записи новых данных в сеттингс последним будет плохой токен, то сеттингс станет кривым
выход - добавить блокировки при выполнения GetNewAuth()

и соответственно проверять эту блокировку при любом чтении getAppSettings()

Попробую свой пул реквест сделать, самому уже надоела эта ситуация, хотя я просто переписал CRest для себя

[Maximryzhkov]

Сори, коллега, но проблема в другом.

каждый час апи требует рефреша токена. Если после протухания токена пойдут почти одновременно два запроса к апи, то оба они будут отправлены за новыми токенами и только один из них будет верным. И если в этой гонке записи новых данных в сеттингс последним будет плохой токен, то сеттингс станет кривым выход - добавить блокировки при выполнения GetNewAuth()

и соответственно проверять эту блокировку при любом чтении getAppSettings()

Попробую свой пул реквест сделать, самому уже надоела эта ситуация, хотя я просто переписал CRest для себя

@anton-zaharov можете здесь поделиться вашим решением, выложить ваш CRest в коммент без пул реквеста?

[anton-zaharov]

include_once DIR.'/bin/crest.php';
use \CRest as BaseRest;

/**
 * Description of CRest
 *
 * @author Антон Захаров 
 */
class CRest extends BaseRest {
    public static function installApp(){
        
        return parent::installApp();
    }
    static protected $authNow = false; 
    
    protected static function getSettingData()
	{
		$return = [];
                $path = app()->basePath('app/Bitrix24') . '/settings.json';
                $file = fopen($path, 'r');
		if($file && (self::$authNow || flock($file, LOCK_SH)))
		{
			$return = static::expandData(file_get_contents($path));
                        flock($file, LOCK_UN);
                        fclose($file);
			if(defined("C_REST_CLIENT_ID") && !empty(C_REST_CLIENT_ID))
			{
				$return['C_REST_CLIENT_ID'] = C_REST_CLIENT_ID;
			}
			if(defined("C_REST_CLIENT_SECRET") && !empty(C_REST_CLIENT_SECRET))
			{
				$return['C_REST_CLIENT_SECRET'] = C_REST_CLIENT_SECRET;
			}
		}
		return $return;
	}

    protected static function setSettingData($arSettings)
	{
            $path = app()->basePath('app/Bitrix24');
            return  (boolean)file_put_contents($path . '/settings.json', static::wrapData($arSettings));
	}
        
    protected static function GetNewAuth($arParams){
        	$result = [];
		$arSettings = self::getAppSettings();
		if($arSettings !== false)
		{       
                        $path = app()->basePath('app/Bitrix24') . '/settings.json';
                        $file = fopen($path, 'r+');
                        flock($file, LOCK_EX);
                        self::$authNow = true;
			$arParamsAuth = [
				'this_auth' => 'Y',
				'params'    =>
					[
						'client_id'     => $arSettings[ 'C_REST_CLIENT_ID' ],
						'grant_type'    => 'refresh_token',
						'client_secret' => $arSettings[ 'C_REST_CLIENT_SECRET' ],
						'refresh_token' => $arSettings[ "refresh_token" ],
					]
			];
			$newData = static::callCurl($arParamsAuth);
			if(isset($newData[ 'C_REST_CLIENT_ID' ]))
			{
				unset($newData[ 'C_REST_CLIENT_ID' ]);
			}
			if(isset($newData[ 'C_REST_CLIENT_SECRET' ]))
			{
				unset($newData[ 'C_REST_CLIENT_SECRET' ]);
			}
			if(isset($newData[ 'error' ]))
			{
				unset($newData[ 'error' ]);
			}
                        $saved = parent::setAppSettings($newData);
                        flock($file, LOCK_UN);
                        fclose($file);
			if($saved)
			{
				$arParams[ 'this_auth' ] = 'N';
				$result = static::callCurl($arParams);
			}
		}
		return $result;
    }
}

/*
* 3 месяца работы - полет нормальный, не разу не слетела авторизация
* писалось под Lumen framework поэтому есть вызов  app() и пару его хелперов для получения пути 
* к нужному каталогу, но можно заменить обычными php инструкциями
* 
* /