SemianiakaVY/elsa-core
1
0
Fork 0
Code Issues Pull requests Actions 1 Packages Projects Releases Wiki Activity

Update JsonContentFactory to use StringContent

Browse source 
This fixes an issue that Content-Length header is not included when using JsonContent, which is problematic for certain backend APIs.
This commit is contained in:
Sipke Schoorstra 2023-11-24 20:11:28 +01:00
parent 6a8c175b6d
commit c08388ec07
1 changed files with 6 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
src/modules/Elsa.Http/ContentWriters/JsonContentFactory.cs
View file

@ -2,6 +2,7 @@ using System.Net.Http.Headers;
using System.Net.Http.Json;
using System.Net.Mime;
using System.Text;
using System.Text.Json;
namespace Elsa.Http.ContentWriters;
@ -16,10 +17,11 @@ public class JsonContentFactory : IHttpContentFactory
/// <inheritdoc />
public HttpContent CreateHttpContent(object content, string contentType)
{
if (content is string s)
return new StringContent(s, Encoding.UTF8, contentType);
var text = content as string ?? JsonSerializer.Serialize(content);
var mediaType = MediaTypeHeaderValue.Parse(contentType);
return JsonContent.Create(content, mediaType);
if (string.IsNullOrWhiteSpace(contentType))
contentType = MediaTypeNames.Application.Json;
return new StringContent(text, Encoding.UTF8, contentType);
}
}