AddCognitoIdentity raises StackOverflow exception about aws-aspnet-cognito-identity-provider

[assyadh]

Hi aussiearef@,

Your solution configuration file contains secrets. I would suggest removing them and revoking the app client id/secret key.

Here's a good tool to prevent that may from happening:

https://github.com/awslabs/git-secrets

[aussiearef]

Thanks @assyadh
I removed the secrets.

Update on the issue:
If I run the website using "dotnet run" or in Visual Studio 2019 Preview, I get the StackOverflow exception. However if I run the .exe file of the website, it works fine!
I am trying to get a dump file and see where the issue resides. So far I know that if I remove AddCognitoIdentity there will be no issue

[aussiearef]

Update:
So I have used some debuggers to see where the Stackoverflow is originated form. It turns out that stack overflow happens when a serialization/serialization happens .
As you see in the image below there is a cycle detected.

I guess one of your models has a self reference and so during serialization/deserializaion stackoverflow happens

[aussiearef]

I also ran a profiler and it shows that Identity.Cognito.dll has a self reference that chews up cpu

[assyadh]

I am currently out of the office. I'll have a look when I'm back early January.

[assyadh]

I am not able to reproduce the issue using your webapp. It works properly in VS2019/.NET Core 3.0 CLI as well as when running the exe directly.

Are you still hitting the issue?

[gopinathrimc]

I am having the same issue. I am just running the Sample project from here[1] in visual studio 2017 with .netcore 2.2. It runs for while but after a sometime, if i start a new instance from VS i get the stackoverflow exception. Then i have to kill my iis express instance, close the browser instance and restart VS to get it to work again.
@aussiearef where you able to resolve this issue?

[1] https://github.com/aws/aws-aspnet-cognito-identity-provider/tree/master/samples/Samples

[assyadh]

Reopening due to additional reports.

[assyadh]

Reopening due to additional reports.

[assyadh]

Hi everyone,

I have just release a new version 0.9.02 that should solve the issue:

https://www.nuget.org/packages/Amazon.AspNetCore.Identity.Cognito/0.9.0.2

Please let me know how that works for you!

Hamza