eightball/PluralKit
1
0
Fork 0
mirror of https://github.com/PluralKit/PluralKit.git synced 2026-02-04 04:56:49 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

fix(bot): reset content-type header before reading content

Browse source 
Discord CDN sometimes returns content-type headers that not only break json
deserializing, but also just reading the body...
This commit is contained in:
alyssa 2024-05-11 20:54:05 +09:00
parent 8f3fa62ec3
commit f6fceffc3f
1 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

6
PluralKit.Bot/Commands/ImportExport.cs
View file

@ -45,6 +45,9 @@ public class ImportExport
try try
{ {
var response = await _client.GetAsync(url); var response = await _client.GetAsync(url);
// hacky fix for discord api returning nonsense charsets sometimes
response.Content.Headers.Remove("content-type");
response.Content.Headers.Add("content-type", "application/json; charset=UTF-8");
var content = await response.Content.ReadAsStringAsync(); var content = await response.Content.ReadAsStringAsync();
if (content == "This content is no longer available.") if (content == "This content is no longer available.")
{ {
@ -54,9 +57,6 @@ public class ImportExport
} }
if (!response.IsSuccessStatusCode) if (!response.IsSuccessStatusCode)
throw Errors.InvalidImportFile; throw Errors.InvalidImportFile;
// hacky fix for discord api returning nonsense charsets sometimes
response.Content.Headers.Remove("content-type");
response.Content.Headers.Add("content-type", "application/json; charset=UTF-8");
data = JsonConvert.DeserializeObject<JObject>( data = JsonConvert.DeserializeObject<JObject>(
content, content,
_settings _settings