fix 20 accounts limit

[rusik69]

fixing 20 accounts limit

by default list_accounts only returns some default amount of accounts

In my case i was able to get only reports for 20 accounts, adding pagination to lambda has fixed it

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

[vdbrooks]

Your PR helped me quickly narrow down the issue, so thank you. There's a minor bug I ran into when implementing it though. The JSON in listed_accounts is slightly different after you added the orgClient paginator, and so the for loop in main needed to account for that.

Instead of:

for account in listedAccounts['Accounts']: if account['Status'] != 'SUSPENDED': assumeRole(account['Id'])

I had to replace it with:

for account in listedAccounts: if account['Status'] != 'SUSPENDED': assumeRole(account['Id']) return("Completed! Failed Accounts: ", failedAccounts)

As the accounts were no longer stored in a key called accounts. Outside of this, I'm a little surprised this fix hasn't been merged yet?

[reubensammut]

Thanks @rusik69 for the fix. Ideally should be added to the CloudFormation template as well.