Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Support round trip serialization of AuthenticationProperties with System.Text.Json #31330

Merged
merged 3 commits into from
Mar 31, 2021
Merged

Support round trip serialization of AuthenticationProperties with System.Text.Json #31330

merged 3 commits into from
Mar 31, 2021

Commits on Mar 28, 2021

  1. Support roundtrip serialization with S.T.Json 

    Add attribute to support round-trip serialization of AuthenticationProperties with System.Text.Json.
Resolves dotnet#20722.
    @martincostello
    martincostello committed Mar 28, 2021
    Configuration menu
    Copy the full SHA
    bf81ce3 View commit details
    Browse the repository at this point in the history

  2. Refactor test 

    Refactor test to not overwrite the input.
    @martincostello
    martincostello committed Mar 28, 2021
    Configuration menu
    Copy the full SHA
    89e21ba View commit details
    Browse the repository at this point in the history

Commits on Mar 29, 2021

  1. Add comment for clarity 

    Add comment explaining that parameters are expected not to be round-tripped as suggested by code review.
    @martincostello
    martincostello committed Mar 29, 2021
    Configuration menu
    Copy the full SHA
    51a7085 View commit details
    Browse the repository at this point in the history