Skip to content
This repository has been archived by the owner on Dec 13, 2022. It is now read-only.

AddOidcStateDataFormatterCache broken with new JSON serializer #4268

Closed
brockallen opened this issue Apr 10, 2020 · 4 comments
Closed

AddOidcStateDataFormatterCache broken with new JSON serializer #4268

brockallen opened this issue Apr 10, 2020 · 4 comments
Assignees
@brockallen
Labels
bug
Milestone
4.0

Comments

@brockallen
Copy link
Member

Internally uses the DistributedCacheStateDataFormatter which uses the new ObjectSerializer to round trip the AuthenticationProperties. The bug is in the ObjectSerializer which doesn't re-hydrate the AuthenticationProperties properly. The fix is more of a workaround to manually create the AuthenticationProperties and only serialize the dictionary of items into json.
@brockallen brockallen added the bug label Apr 10, 2020
@brockallen brockallen added this to the 4.0 milestone Apr 10, 2020
brockallen added a commit that referenced this issue Apr 10, 2020
@brockallen
change how we serialize AuthenticationProperties #4268
6c176a6
@brockallen brockallen self-assigned this Apr 10, 2020
@brockallen
Copy link
Member Author

Fixed.

@brockallen
Copy link
Member Author

dotnet/aspnetcore#20722

@lock
Copy link

lock bot commented May 20, 2020

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

1 similar comment
@lock
Copy link

lock bot commented May 20, 2020

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators May 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@brockallen brockallen

Labels
bug
Projects
None yet
Milestone
4.0
Development

No branches or pull requests
1 participant
@brockallen