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.

Sign up for GitHub

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

Jump to bottom

Workaroud for SAML ListUserAccounts limit #287

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

Workaroud for SAML ListUserAccounts limit #287

Changes from 2 commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
127fb39
Workaroud for SAML ListUserAccounts limit
kargaily Aug 19, 2022
c084477
Added check for NextPageToken in ListUserAccounts response
kargaily Aug 19, 2022
99f88fa
Avoid usage of ListUserAccounts method
kargaily Aug 23, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 15 additions & 1 deletion yandex/data_source_yandex_organizationmanager_saml_federation_user_account.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,21 @@ func dataSourceYandexOrganizationManagerSamlFederationUserAccountRead(d *schema.
return err
}

for _, account := range resp.UserAccounts {
UserAccounts := resp.UserAccounts
Copy link

Choose a reason for hiding this comment

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

А для чего здесь делается вызов метода ListUserAccounts?
Метод AddUserAccounts возвращает операцию, в поле result у которой и так будет список пользователей.

Copy link

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.

Спасибо за коммент, обновил, теперь Id забирается из результата операции.

Copy link

Choose a reason for hiding this comment

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

Я не очень правильно выразился.
Здесь для резолва пользователя (проверки того, что он есть в федерации) используется addUser операция. Это не очень корректно, но что уж поделать, убирать это не стоит, чтобы сохранить обратную совместимость. Однако из-за использования этой операции пользователем с доступом только на просмотр аккаунтов, а не создание, вообще не может пользовать ресурсом.

Предлагаю сначала при помощи операции листинга проверять наличие аккаунта и резолвить его, если найдем. Если не нашли - тогда уже добавляем.


for resp.NextPageToken != "" {
resp, err = config.sdk.OrganizationManagerSAML().Federation().ListUserAccounts(config.Context(), &saml.ListFederatedUserAccountsRequest{
FederationId: federationID,
PageToken: resp.NextPageToken,
})

if err != nil {
return err
}
UserAccounts = append(UserAccounts, resp.UserAccounts...)
}

for _, account := range UserAccounts {
if account.GetSamlUserAccount().GetNameId() == nameID {
d.SetId(account.Id)
return nil
Expand Down