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

Пользовательский заголовок в запрос #82

Closed
ipolikarpov opened this issue May 28, 2020 · 9 comments
Closed

Comments

@ipolikarpov
Copy link

Доброго дня!
Верно ли я понял, что добавить свой заголовок к curl-запросу нельзя?

@gwinn
Copy link
Member

gwinn commented May 28, 2020

О каком заголовке речь?

@ipolikarpov
Copy link
Author

например,
curl_setopt($curlHandler, CURLOPT_HTTPHEADER, array('X-Rlimit-token: ****'));

@gwinn
Copy link
Member

gwinn commented May 28, 2020

В данный момент нет, а можете подробнее рассказать про кейс, когда вам нужно установить такой заголовок?

@muxx
Copy link
Member

muxx commented May 29, 2020

@gwinn это для специально выделенных повышенных лимитов

@ipolikarpov
Copy link
Author

В данный момент нет, а можете подробнее рассказать про кейс, когда вам нужно установить такой заголовок?

В отдельных случаях ТП выдает такого рода заголовок для повышения лимитов обращений к API.
Я пока в проекте хардкорно в Client.php добил его, вот пытаюсь разобраться как грамотно его вам предложить для внесения в проект.

@muxx
Copy link
Member

muxx commented May 29, 2020

Токен типовой с точки зрения именования. Думаю имеет смысл завести отдельный атрибут класса с геттером/сеттером

@ipolikarpov
Copy link
Author

Типа
$client->setCustomHeader(''X-Rlimit-token: ****'')
?

@ipolikarpov
Copy link
Author

Типа
$client->setCustomHeader(''X-Rlimit-token: ****'')
?

Тут даже правильнее будет массив передавать
$client->setCustomHeader(array('X-Rlimit-token: ****'));

@muxx
Copy link
Member

muxx commented May 29, 2020

Нет, типа

$client->setLimitToken('23f23...');
$client->getLimitToken();

Соответственно приватный атрибут класса $limitToken и его подстановка в заголовки, если есть.

@gwinn gwinn closed this as completed Aug 21, 2020
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

No branches or pull requests

3 participants