Skip to content
This repository was archived by the owner on May 16, 2023. It is now read-only.

cognito: Fix nullable cognito pre-token-gen event ClaimsOverrideDetails #105

Conversation

sebboer
Copy link

@sebboer sebboer commented Sep 1, 2022

The actual incoming event for the Cognito pre-token-gen should have the claimsOverrideDetails field as optional.
This is what the incoming event looks like:

{
  "version": "1",
  "triggerSource": "PreTokenGen",
  "region": "region",
  "userPoolId": "userPoolId",
  "userName": "userName",
  "callerContext": {
    "awsSdkVersion": "calling aws sdk with version",
    "clientId": "apps client id"
  },
  "request": {
    "userAttributes": {
      "email": "email",
      "phone_number": "phone_number"
    },
    "groupConfiguration": {
      "groupsToOverride": ["group-A", "group-B", "group-C"],
      "iamRolesToOverride": ["arn:aws:iam::XXXXXXXXXXXX:role/sns_callerA", "arn:aws:iam::XXXXXXXXX:role/sns_callerB", "arn:aws:iam::XXXXXXXXXX:role/sns_callerC"],
      "preferredRole": "arn:aws:iam::XXXXXXXXXXX:role/sns_caller"
    },
    "clientMetadata": {
      "exampleMetadataKey": "example metadata value"
    }
  },
  "response": {
    "claimsOverrideDetails": null
  }
}

The events for cognito are not really well documented and differ from the actual events.

@sebboer sebboer changed the title Fix nullable cognito pre-token-gen event ClaimsOverrideDetails cognit: Fix nullable cognito pre-token-gen event ClaimsOverrideDetails Sep 1, 2022
@sebboer sebboer changed the title cognit: Fix nullable cognito pre-token-gen event ClaimsOverrideDetails cognito: Fix nullable cognito pre-token-gen event ClaimsOverrideDetails Sep 1, 2022
@calavera calavera merged commit be3008c into calavera:master Sep 2, 2022
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants