Consider using AddJsonStream

[wayne-o]

It was added in netcore 3.0.0-preview6 and would mean we can delete some of the code I needed to copy over from the netcore source.

Would this be favorable? If so i can look to adding a PR

[andrewlock]

Interesting, on the one hand it does make sense, but given this code is supposed to support .NET Core 2.1 as well it's probably not going to be possible.

Plus the fact that VaultSharp uses Newtonsoft.Json under the hood, so you're always going to be pulling that in as a dependency anyway.

Feel free to give it more of a look if you like, but probably not worth the effort at this stage I suspect 🙂