claudiajs / claudia

Deploy Node.js projects to AWS Lambda and API Gateway easily
https://claudiajs.com
MIT License
3.8k stars 276 forks source link

fix for add-cognito-user-pool-trigger, fails when user pool has custom domain #157

Closed csepulv closed 6 years ago

csepulv commented 6 years ago

If a user pool has a custom domain, add-cognito-user-pool-trigger fails as AWS parameter validation fails because the Domain attribute is sent back, but not expected.

It seems like an easy fix: add-cognito-user-pool-trigger.js, `cleanUpPoolConfig', add 'Domain' to the array of attributes to remove.

I tried this locally and that seems to work.

gojko commented 6 years ago

thanks, I'll push this out with the next release