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

Remove implicit fallback to reflection-based serialization #71746

Merged
merged 11 commits into from
Jul 7, 2022
Merged

Remove implicit fallback to reflection-based serialization #71746

merged 11 commits into from
Jul 7, 2022

Commits on Jul 7, 2022

  1. Remove implicit fallback to reflection-based serialization. Fix dotne… 

    …t#71714

Include JsonSerializerContext in JsonSerializerOptions copy constructor. Fix dotnet#71716

Move reflection-based converter resolution out of JsonSerializerOptions. Fix dotnet#68878
    @eiriktsarpalis
    eiriktsarpalis committed Jul 7, 2022
    Configuration menu
    Copy the full SHA
    708b8cc View commit details
    Browse the repository at this point in the history

  2. Address feedback & add one more test

    @eiriktsarpalis
    eiriktsarpalis committed Jul 7, 2022
    Configuration menu
    Copy the full SHA
    9f78a95 View commit details
    Browse the repository at this point in the history

  3. Update src/libraries/System.Text.Json/src/System/Text/Json/Serializat… 

    …ion/JsonSerializerContext.cs

Co-authored-by: Stephen Toub <stoub@microsoft.com>
    @eiriktsarpalis @stephentoub
    eiriktsarpalis and stephentoub authored Jul 7, 2022
    Configuration menu
    Copy the full SHA
    56b1665 View commit details
    Browse the repository at this point in the history

  4. fix build

    @eiriktsarpalis
    eiriktsarpalis committed Jul 7, 2022
    Configuration menu
    Copy the full SHA
    f829c9b View commit details
    Browse the repository at this point in the history

  5. Bring back throwing behavior in JsonSerializerContext and add tests

    @eiriktsarpalis
    eiriktsarpalis committed Jul 7, 2022
    Configuration menu
    Copy the full SHA
    98c3349 View commit details
    Browse the repository at this point in the history

  6. Only create caching contexts if a resolver is populated

    @eiriktsarpalis
    eiriktsarpalis committed Jul 7, 2022
    Configuration menu
    Copy the full SHA
    fe511ea View commit details
    Browse the repository at this point in the history

  7. Add null test for JsonSerializerContext interface implementation.

    @eiriktsarpalis
    eiriktsarpalis committed Jul 7, 2022
    Configuration menu
    Copy the full SHA
    59b4b38 View commit details
    Browse the repository at this point in the history

  8. skip RemoteExecutor test in netfx targets

    @eiriktsarpalis
    eiriktsarpalis committed Jul 7, 2022
    Configuration menu
    Copy the full SHA
    f7e37cd View commit details
    Browse the repository at this point in the history

  9. Add DefaultJsonTypeInfoResolver test for types with JsonConverterAttr… 

    …ibute
    @eiriktsarpalis
    eiriktsarpalis committed Jul 7, 2022
    Configuration menu
    Copy the full SHA
    38c4672 View commit details
    Browse the repository at this point in the history

  10. remove nullability annotation

    @eiriktsarpalis
    eiriktsarpalis committed Jul 7, 2022
    Configuration menu
    Copy the full SHA
    e029c6e View commit details
    Browse the repository at this point in the history

  11. Update src/libraries/System.Text.Json/src/System/Text/Json/Serializat… 

    …ion/JsonSerializerOptions.Converters.cs

Co-authored-by: Krzysztof Wicher <mordotymoja@gmail.com>
    @eiriktsarpalis @krwq
    eiriktsarpalis and krwq authored Jul 7, 2022
    Configuration menu
    Copy the full SHA
    ed6c70c View commit details
    Browse the repository at this point in the history