Repository: cloud-custodian/cloud-custodian

Issue:
AWS user pool and identity pool resources.json has minimal information
### Describe the bug

For the below custodian policy, the resources.json is created for each of the policy on successful execution. For user-pool and identity-pool, the resources.json file does not include full description/configuration of the resources, instead it contains very minimal information like ID, NAME, creation date etc. (as mentioned below)  whereas for EC2 and Lambda, the populated resources.json has hundreds of metadata information.


---
##custodian.yaml
policies:
  - name: cognito-checkauditmode
    resource: aws.user-pool

  - name: identity-checkauditmode
    resource: identity-pool

  - name: ec2-checkrunning
    resource: ec2

- name: find-all-lambdas
   resource: aws.lambda

--- 
##resources.json -  cognito-checkauditmode

[
  {
    "Id": "xxxxxxxxxxxxxx",
    "Name": "xxxxxxxxxxxxxxxxxxx",
    "LambdaConfig": {},
    "LastModifiedDate": "2023-06-29T08:56:18.028000-05:00",
    "CreationDate": "2023-06-29T08:56:17.860000-05:00",
    "Tags": []
  },
  {
    "Id": "xxxxxxxxxxxxxxxxxxx",
    "Name": "xxxxxxxxxxxxxxxxxxx",
    "LambdaConfig": {},
    "LastModifiedDate": "2020-06-11T17:15:18.951000-05:00",
    "CreationDate": "2020-02-21T11:39:18.108000-06:00",
    "Tags": []
  }
]


---
## resources.json - identity-checkauditmode

[
  {
    "IdentityPoolId": "xxxxxxxxxxxxxxxxxxx",
    "IdentityPoolName": "xxxxxxxxxxxxxxxxxxx",
    "Tags": []
  }
]

### What did you expect to happen?

Expecting a large json file with full configuration of the resource. Below is the AWS CLI command and the truncated response from CLI. Expecting a similar response. 


---
aws cognito-idp describe-user-pool --user-pool-id xxxxxxxxxxxxxxxxxxx
---
truncated response
{
    "UserPool": {
        "Id": "xxxxxxxxxxxxxxxxxxx",
        "Name": "xxxxxxxxxxxxxxxxxxx",
        "Policies": {
            "PasswordPolicy": {
                "MinimumLength": 8,
                "RequireUppercase": true,
                "RequireLowercase": true,
                "RequireNumbers": true,
                "RequireSymbols": true,
                "TemporaryPasswordValidityDays": 7
            }
        },
        "DeletionProtection": "INACTIVE",
        "LambdaConfig": {},
        "LastModifiedDate": "2020-06-11T17:15:18.951000-05:00",
        "CreationDate": "2020-02-21T11:39:18.108000-06:00",
        "SchemaAttributes": [
            {
                "Name": "sub",
                "AttributeDataType": "String",
                "DeveloperOnlyAttribute": false,
                "Mutable": false,
                "Required": true,
                "StringAttributeConstraints": {
                    "MinLength": "1",
                    "MaxLength": "2048"
                }
            },

### Cloud Provider

Amazon Web Services (AWS)

### Cloud Custodian version and dependency information

```shell
Custodian:   0.9.27
Python:      3.11.4 (main, Jun  7 2023, 00:34:59) [Clang 14.0.3 (clang-1403.0.22.14.1)]
Platform:    posix.uname_result(sysname='Darwin', nodename='MABPWKJJ4T9RYW', release='22.5.0', version='Darwin Kernel Version 22.5.0: Thu Jun  8 22:22:23 PDT 2023; root:xnu-8796.121.3~7/RELEASE_ARM64_T6020', machine='arm64')
Using venv:  False
Docker: False
Installed: 

argcomplete==3.0.8
attrs==23.1.0
boto3==1.26.139
botocore==1.29.139
docutils==0.18.1
importlib-metadata==5.2.0
jmespath==1.0.1
jsonschema==4.17.3
pyrsistent==0.19.3
python-dateutil==2.8.2
pyyaml==6.0
s3transfer==0.6.1
six==1.16.0
tabulate==0.9.0
typing-extensions==4.6.3
urllib3==1.26.16
zipp==3.15.0
```


### Policy

```shell
##custodian.yaml
policies:
  - name: cognito-checkauditmode
    resource: aws.user-pool

  - name: identity-checkauditmode
    resource: identity-pool

  - name: ec2-checkrunning
    resource: ec2

- name: find-all-lambdas
   resource: aws.lambda
```


### Relevant log/traceback output

```shell
2023-06-26 20:09:45,838 - custodian.policy - INFO - policy:cognito-checkauditmode resource:aws.user-pool region:us-east-1 count:1 time:0.00
2023-06-26 20:20:16,225 - custodian.policy - INFO - policy:cognito-checkauditmode resource:aws.user-pool region:us-east-1 count:1 time:0.70
2023-06-26 20:25:23,030 - custodian.policy - INFO - policy:cognito-checkauditmode resource:user-pool region:us-east-1 count:1 time:0.00
2023-06-26 23:09:38,143 - custodian.policy - INFO - policy:cognito-checkauditmode resource:user-pool region:us-east-1 count:1 time:0.73
2023-06-26 23:13:37,202 - custodian.policy - INFO - policy:cognito-checkauditmode resource:user-pool region:us-east-1 count:1 time:0.00
2023-06-26 23:17:02,042 - custodian.policy - INFO - policy:cognito-checkauditmode resource:user-pool region:us-east-1 count:1 time:0.00
2023-06-26 23:18:59,196 - custodian.policy - INFO - policy:cognito-checkauditmode resource:user-pool region:us-east-1 count:0 time:0.00
2023-06-26 23:28:37,082 - custodian.policy - INFO - policy:cognito-checkauditmode resource:user-pool region:us-east-1 count:0 time:0.67
2023-06-27 09:11:53,373 - custodian.policy - INFO - policy:cognito-checkauditmode resource:user-pool region:us-east-1 count:1 time:0.67
2023-06-27 09:13:07,745 - custodian.policy - INFO - policy:cognito-checkauditmode resource:user-pool region:us-east-1 count:1 time:0.00
2023-06-27 09:22:13,584 - custodian.policy - INFO - policy:cognito-checkauditmode resource:user-pool region:us-east-1 count:0 time:0.00
2023-06-27 09:22:42,984 - custodian.policy - INFO - policy:cognito-checkauditmode resource:user-pool region:us-east-1 count:0 time:0.65
2023-06-27 09:24:43,016 - custodian.policy - INFO - policy:cognito-checkauditmode resource:aws.user-pool region:us-east-1 count:0 time:0.62
2023-06-27 09:27:15,604 - custodian.policy - INFO - policy:cognito-checkauditmode resource:aws.user-pool region:us-east-1 count:1 time:0.64
2023-06-29 08:58:25,076 - custodian.policy - INFO - policy:cognito-checkauditmode resource:aws.user-pool region:us-east-1 count:2 time:0.64
```


### Extra information or context

Applied few additional filters and that as well failed. I believe, the filters will work only after the describe is successful


Discussion:

Resolving patch:
diff --git a/c7n/resources/cognito.py b/c7n/resources/cognito.py
--- a/c7n/resources/cognito.py
+++ b/c7n/resources/cognito.py
@@ -11,6 +11,7 @@
 
 class DescribeIdentityPool(DescribeSource):
     def augment(self, resources):
+        resources = super().augment(resources)
         return universal_augment(self.manager, resources)
 
 


Test patch:
diff --git a/tests/test_cognito.py b/tests/test_cognito.py
--- a/tests/test_cognito.py
+++ b/tests/test_cognito.py
@@ -51,6 +51,12 @@ def test_query_identity_pool(self):
             sorted([n["IdentityPoolName"] for n in resources]),
             ["origin_MOBILEHUB_1667653900", "test_delete_id_pool"],
         )
+        # Confirm that our augment pass has tag information and detail
+        # from describe_identity_pool
+        self.assertLessEqual(
+            {"IdentityPoolId", "Tags", "CognitoIdentityProviders"},
+            set(resources[0])
+        )
 
     def test_delete_identity_pool(self):
         factory = self.replay_flight_data("test_cognito-identity-pool_delete")