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

Получение информации об организациях пользователя #29

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

shahruslan
Copy link

Раньше информацию можно было получить одним запросом. Теперь нужно использовать два запроса. В первом запросе получаем идентификаторы организаций, а во втором уже непосредственно данные. Конечно, если заранее знать идентификатор организации, можно обойтись и одним запросом.
На тестовых аккаунтах работает без ошибок, на боевых пока проверить не смог.

return "http://esia.gosuslugi.ru/{$scope}?org_oid={$orgId}";
}, $scopes);

$this->refreshToken($scopes);
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

А для чего обновлять токен при запросе организации?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Для того, чтобы передать скоуп вида http://esia.gosuslugi.ru/org_inn?org_oid=#ид_организации_который_нам_заранее_не_известен#
этот идентификатор мы получаем только после получения списка организаций. И если мы не обновим токен, то мы не можем запросить инн(например) организации, потому что скоуп org_inn не передан при первом обращении. При обновлении токена, мы передаем новые скоупы, по которум уже сможем получить данные организации.
Как я и написал выше, если идентификатор заранее известен, то можно обойтись и одним запросом, без обновления токена. Но этот случай очень редкий.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Странно, а разве пользователь не должен дать согласие на получение данных из другого скоупа, нежели был изначально запрошен при получение токена? Есть ссылка на документацию где можно почитать об этом?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Да, странно. Если передать скоуп вида http://esia.gosuslugi.ru/org_inn?org_oid=123 при первом запросе на получение авторизационного кода, то да, запрос на согласие предоставления инн организации появится. Но org_oid мы получаем только после того как запросим скоуп usr_org. Так что тут или еще раз сделать редирект на $portalUrl, но уже со скоупами организаций. Или вот таким способом.

Документация вот https://digital.gov.ru/ru/activity/directions/13/#section-docs
Методические рекомендации ... Раздел В.3

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

А ты пробовал из пункта В.4?

image

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

$orgId = $matches[1];

$scopes = array_map(function ($scope) use ($orgId) {
return "http://esia.gosuslugi.ru/{$scope}?org_oid={$orgId}";
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Тут наверно должно было использовано свойство $portalUrl

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Нет, валидный скоуп именно такой, должен начинаться с http://esia.gosuslugi.ru/
Если мы будем передавать $portalUrl, то на тестовом стенде ЕСИА это не сработает, потому как там $portalUrl отличается.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Может вынести в константу?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Да, можно.
А как лучше назвать, ESIA_URL ?)

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Можно думаю SCOPE_DOMAIN

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Или SCOPE_URL

@fr05t1k
Copy link
Owner

fr05t1k commented Apr 13, 2020

Ну и не плохо было бы покрыть тестами методы refreshToken, getOrganizationLinks, getOrganizationInfo

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