Skip to content

Conversation

@jennyhougen
Copy link
Collaborator

  • Some cleanup in documentateion
  • Removed unused confusing paramteter from Blazor InMemoryUserTokenStore
  • Updated package version and publilshed beta version of Fhi.Authentication.Extensions

@jennyhougen jennyhougen self-assigned this May 15, 2025
@jennyhougen jennyhougen requested review from a user and GeirMG May 15, 2025 07:12
Copy link
Contributor

@GeirMG GeirMG left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could also fix the one warning in this solution:

Found markup element with unexpected name 'Logout'. If this is intended to be a component, add a @using directive for its namespace.

* The AddOpenIdConnectCookieEventServices default is 60 minutes.
* ***************************************************************************************/
options.ExpireTimeSpan = TimeSpan.FromSeconds(30);
options.ExpireTimeSpan = TimeSpan.FromSeconds(6000);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment mentions some defaults, but the code is none of them?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is only sample on how to set it. Can add a sentance about that.

jennyhougen and others added 2 commits May 15, 2025 10:12
…tion/InMemoryTokenStore.cs

Co-authored-by: Geir Marius <gmgjul@gmail.com>
@jennyhougen jennyhougen merged commit 81b2c24 into main May 16, 2025
3 checks passed
@jennyhougen jennyhougen deleted the release_beta_authenticationextensions branch May 16, 2025 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants