This repository has been archived by the owner on Apr 8, 2020. It is now read-only.

Fix JsonTextWriter usage if necessary #822

Closed

kichalla opened this issue Mar 28, 2017 · 1 comment

Member

kichalla commented Mar 28, 2017

MVC had the following issue filed:
aspnet/Mvc#5413

Json.Net 10.0.1 has a property on JsonTextWriter which fixes this
http://www.newtonsoft.com/json/help/html/P_Newtonsoft_Json_JsonWriter_AutoCompleteOnClose.htm

JavaScriptServices currently uses text writer in the following locations, check if these locations need to be fixed too:

https://github.com/aspnet/JavaScriptServices/blob/dev/src/Microsoft.AspNetCore.NodeServices.Sockets/SocketNodeInstance.cs#L179

kichalla changed the title ~~Fix JsonTextWriter if necessary~~ Fix JsonTextWriter usage if necessary

SteveSandersonMS closed this as completed in

6ed6b8e

Member

SteveSandersonMS commented Mar 29, 2017

Thanks - fixed!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.