Fix JsonTextWriter usage if necessary #875

New Issue

fergalmoran · 2025-08-09T17:18:00Z

fergalmoran commented

2025-08-09 17:18:00 +00:00

Originally created by @kichalla on 3/28/2017

MVC had the following issue filed:
https://github.com/aspnet/Mvc/issues/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

*Originally created by @kichalla on 3/28/2017* MVC had the following issue filed: https://github.com/aspnet/Mvc/issues/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

fergalmoran closed this issue

2025-08-09 17:18:00 +00:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: github/JavaScriptServices#875