feat(cognito): user pool resource server

[iRoachie]

Also allow adding resource server via userPool.addResourceServer

---

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

[gitpod-io]

[iRoachie]

@nija-at All good here?

[nija-at]

Some comments below.

Pull request has been modified.

[iRoachie]

@nija-at Updated. Need your feedback on the dependency pattern though. When pairing with an app client, the resource server and its scopes need to be created first before the client can reference it. I realized this when I was doing the integ test that sometimes the client will be created before the resource server, which will result in the stack failing since the scope can't be found.

We need to create a dependency to ensure it happens in the right order. I did

aws-cdk/packages/@aws-cdk/aws-cognito/test/integ.user-pool-resource-server.ts

Line 42 in 5e0d241

client.node.addDependency(resourceServer);

.

Is this the best way or should we do this internally in UserPoolClient, and accept a resourceServers prop?

Pull request has been modified.

[iRoachie]

@nija-at Suggested changes made.

[nija-at]

Hey @iRoachie - I made a small adjustment to the API in this PR - 2715726#diff-c27648629074407952dc3ab03cdf2ec0193f1eb61372913b8c92265051cc553a.

Let me know if you think if it's better.

[iRoachie]

@nija-at Definitely better. Removes the possibility for misspelling scope name between server and client 👍

[mergify]

Thank you for contributing! Your pull request will be updated from master and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

[aws-cdk-automation]

AWS CodeBuild CI Report

• CodeBuild project: AutoBuildProject6AEA49D1-qxepHUsryhcu
• Commit ID: 06039bf
• Result: SUCCEEDED
• Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

[mergify]

Thank you for contributing! Your pull request will be updated from master and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

[iRoachie]

Thanks again for your patience @nija-at 🥇