IntentArchitect / Support

A repository dedicated to handling issues and support queries
3 stars 0 forks source link

appsettings.json JwtToken:RefreshTokenExpiryMinutes naming problem #103

Closed SarperMUTLUBAY closed 2 weeks ago

SarperMUTLUBAY commented 2 weeks ago

What happened?

There is a naming issue between TokenService and appsettings.json. We can set the RefreshTokenExpiry value as minutes according to the appsettings.json file, but TokenService processes the same value as days.

image

What version of Intent Architect are you using?

4.3.4

Additional information

Project type : Clean architecture .NET Security : Identity Account Management

JonathanLydall commented 2 weeks ago

Hi @SarperMUTLUBAY,

Thank you very much for taking the time to report this.

We have now updated the Intent.AspNetCore.Identity.AccountController module to pre-release version 3.1.6-pre.0 which fixes this issue.

Please don't hesitate to reach out again should you have any further comments or questions.

JonathanLydall commented 2 weeks ago

Hi @SarperMUTLUBAY,

Another update here, after further consideration from our side, we'll be changing this to instead take in a TimeSpan string and we'll be updating the appsettings.json field name accordingly.

I will give an update here once this update has been made.

JonathanLydall commented 2 weeks ago

Hi @SarperMUTLUBAY,

We now have version 4.0.0-pre.0 which changes the fields to have be TimeSpan suffixes and values, this has also been mentioned in the release notes:

image

SarperMUTLUBAY commented 2 weeks ago

Hi @JonathanLydall, Thanks for the fast reply and fast fix. I checked now and everything looks good.