Git Product home page Git Product logo

Comments (13)

slang25 avatar slang25 commented on July 18, 2024 1

That would be amazing, in the meantime if I figure out anything more I'll let you know 😃

from httpclient-interception.

slang25 avatar slang25 commented on July 18, 2024 1

Pulled it down and can confirm this has fixed it 😄 Nice one

from httpclient-interception.

martincostello avatar martincostello commented on July 18, 2024

Yay, breaking changes are fun! 😭

from httpclient-interception.

martincostello avatar martincostello commented on July 18, 2024

This is particularly weird as we're happily using 12.0.2 internally in our apps' tests.

from httpclient-interception.

martincostello avatar martincostello commented on July 18, 2024

Though we're not using the overload that takes a JsonSerializerSettings.

from httpclient-interception.

slang25 avatar slang25 commented on July 18, 2024

Also, I've not tried it on windows with Full Framework, it might be a mono only quirk, not sure yet.
Edit: Works on netcoreapp2.x

from httpclient-interception.

martincostello avatar martincostello commented on July 18, 2024

Actually we are using that overload, it's just that it's an optional parameter:

https://github.com/justeat/httpclient-interception/blob/fe8ae42cee9f94802421ca4ac32279821075aca6/src/HttpClientInterception/HttpRequestInterceptionBuilderExtensions.cs#L213-L217

from httpclient-interception.

martincostello avatar martincostello commented on July 18, 2024

If this is a blocker it'll have to wait until .NET Core 3.0 ships in September, because then we can ship a 3.0.0 of this as a breaking change and remove the dependency on Netwonsoft.Json to give people the option to use System.Text.Json instead.

(And it would be tidier to ship a 3.0 to go with 3.0 😄)

from httpclient-interception.

slang25 avatar slang25 commented on July 18, 2024

I ideally need a fix soonish, my workaround would be running this on core, but we want mono cause it's the closest thing to xamarin iOS/Android which is our target platform.

That exception tells me it can't find WithJsonContent, so I don't get why the version of newtonsoft has anything to do with it, surely the exception message is wrong and it means some other method?

from httpclient-interception.

martincostello avatar martincostello commented on July 18, 2024

Yeah, it's a weird one. Something about optional parameters and/or a change somewhere in 12.x with JsonSerializerSettings?

from httpclient-interception.

martincostello avatar martincostello commented on July 18, 2024

I'd be happy to look at doing a beta of a 3.0.0 with changes to decouple Newtonsoft.Json in the soon-ish future (within a week?) and pushing it to NuGet to unblock you though.

from httpclient-interception.

martincostello avatar martincostello commented on July 18, 2024

I'll see if I can pick up the above this week.

from httpclient-interception.

martincostello avatar martincostello commented on July 18, 2024

See #127 for a fix candidate.

from httpclient-interception.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.