spring-cloud-aws icon indicating copy to clipboard operation
spring-cloud-aws copied to clipboard

AWS Cognito Integration

Open Forfend opened this issue 1 year ago • 9 comments

:loudspeaker: Type of change

  • [ ] Bugfix
  • [x] New feature
  • [ ] Enhancement
  • [ ] Refactoring

:scroll: Description

This PR provides the PoC of AWS Cognito Integration, as discussed with @maciejwalkowiak in #1246. The integration provides a higher level of abstraction over AWS Cognito user pool basic auth operations such as creating a user, logging in, and resetting a password.

:bulb: Motivation and Context

This feature provides basic (as of this version) integration with AWS Cognito and can be enhanced with future PRs.

:green_heart: How did you test it?

Tested it with the help of an extra pet project. More unit & integration tests are coming in the next commits in the scope of this PR.

:pencil: Checklist

  • [x] I reviewed submitted code
  • [x] I added tests to verify changes
  • [ ] I updated reference documentation to reflect the change
  • [x] All tests passing
  • [ ] No breaking changes

:crystal_ball: Next steps

This integration can be extended by adding functionality for creating and managing user pools, to be discussed...

Forfend avatar Oct 21 '24 20:10 Forfend

@MatejNedic anything you would like to add?

maciejwalkowiak avatar Nov 11 '24 21:11 maciejwalkowiak

@Forfend could you also add a sample application in spring-cloud-aws-samples directory?

maciejwalkowiak avatar Nov 11 '24 21:11 maciejwalkowiak

@maciejwalkowiak yes, in progress

Forfend avatar Nov 13 '24 11:11 Forfend

Will take a deep look tomorrow on first glance I like it.

Use case itself is valid, I like it good job @Forfend 👍

MatejNedic avatar Nov 13 '24 19:11 MatejNedic

@Forfend apologies I wasn't specific enough. I meant more realistic example - how to use it with Spring Security for example. I will try to figure it out today myself.

maciejwalkowiak avatar Nov 15 '24 06:11 maciejwalkowiak

@maciejwalkowiak I added some real-world examples and their usage with Spring Security

Forfend avatar Nov 22 '24 14:11 Forfend

Hi @MatejNedic, thanks for the review. I fixed all your comments

Forfend avatar Dec 09 '24 17:12 Forfend

Thanks @Forfend! We are not going to include it in next release as I need to spend a little more time with it and play.

maciejwalkowiak avatar Dec 14 '24 07:12 maciejwalkowiak

Hi @maciejwalkowiak do you have any updates on this PR?

Forfend avatar May 15 '25 19:05 Forfend