aws / aws/aws-lambda-go

Missing field enableSMSMFA on struct events.CognitoEventUserPoolsMigrateUserResponse

Open
#553 0 comments 1 reaction 0 assignees View on GitHub
enhancement type/events
Dominant language
Go
Stars
3.8k
Forks
578
Avg merge
8h 18m
Merged PRs (30d)
1

Description

**Is your feature request related to a problem? Please describe.**
According to cognito migrate user lambda trigger documentation the event struct could use a field called enableSMSMFA to allow force the migrated user to trigger the SMSMFA step.

That field is not present in the current events package.

**Describe the solution you'd like**
On file `events/cognito.go`:
```
// CognitoEventUserPoolsMigrateUserResponse contains the response portion of a MigrateUser event
type CognitoEventUserPoolsMigrateUserResponse struct {
UserAttributes map[string]string `json:"userAttributes"`
FinalUserStatus string `json:"finalUserStatus"`
MessageAction string `json:"messageAction"`
DesiredDeliveryMediums []string `json:"desiredDeliveryMediums"`
ForceAliasCreation bool `json:"forceAliasCreation"`
EnableSMSMFA bool `json:"enableSMSMFA"`
}
```

**Describe alternatives you've considered**
For now I'm trying to create a local struct that inlude the original plus that field:
```
type UserMigrationEventResponse struct {
events.CognitoEventUserPoolsMigrateUserResponse
EnableSMSMFA bool `json:"enableSMSMFA"`
}

type UserMigrateAuthEvent struct {
events.CognitoEventUserPoolsHeader
events.CognitoEventUserPoolsMigrateUserRequest `json:"request"`
UserMigrationEventResponse `json:"response"`
}
```

**Additional context**
Documentation of the event: https://docs.aws.amazon.com/cognito/latest/developerguide/user-pool-lambda-migrate-user.html#cognito-user-pools-lambda-trigger-syntax-user-migration

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.