Cognito user attributes type

[NSCoder]

Hi Mantainers!

Thank you for all the effort that you have put into this library, is fantastic to use and very well documented. I was using CognitoIdentityProvider for one of my lambdas and I realized that the User attributes is an array type AttributeListType when it would be easier to use if it where a dictionary.

https://github.com/swift-aws/aws-sdk-swift/blob/23ecff293ad950d1dcae7d7745474835c444cbc7/models/apis/cognito-idp/2016-04-18/api-2.json#L2400-L2403

What do you think? I'm happy to submit a PR if we want to move forward with this change.

[jonnymacs]

@juankaram hello back! And thank you for using this library!

As far as your request - eager / happy to review a PR. I will say that the api definitions come from the api json files which are published by amazon and pulled over from the go sdk. The code is then generated from the json directly, and the aws interfaces expect the structure and datatypes defined in the json.

There is some precedent for making edits, to the json for a service if you want to try it out. If it works, submit a PR, and please update the Readme at models/README.md with details

So I understand, you are proposing to make AttributeListType a map?

[NSCoder]

Yes! The problem is that if you pull all the users for a specific group and then try to read a specific attribute like email, you have to have a nested iteration which is not bad but it could be better if we could just access the value directly.

Sure thing, let me give it a shot and report back.

[adam-fowler]

@juankaram do you have anything else to add to this? For your purposes you could create a dictionary from the array returned in the response.

[adam-fowler]

As there has been no response I am closing this