search

aspnet / AspNetSessionState

ASP.NET Session State (not for ASP.NET Core)
MIT License
55 stars 43 forks source link

Cosmos provider - SessionStateActionsConverter incorrect implementation? #63

Closed qxx closed 1 year ago

qxx commented 1 year ago

It seems in ReadJson true is mapped to SessionStateActions.InitializeItem but in WriteJson true is mapped from SessionStateActions.None?

https://github.com/aspnet/AspNetSessionState/blob/4a58f9926fd1f7b6ee4f06dbf56ff9c3cff76a72/src/CosmosDBSessionStateProviderAsync/SessionStateActionsConverter.cs#L17-L30

https://github.com/aspnet/AspNetSessionState/blob/4a58f9926fd1f7b6ee4f06dbf56ff9c3cff76a72/src/CosmosDBSessionStateProviderAsync/SessionStateActionsConverter.cs#L32-L50