Parsing ContentType is too restrictive

Question

Parsing ContentType is too restrictive

Closed this issue 5 years ago · 4 comments

DocumentLoader::LoadDocument() tries to validate that the media type is JSON-LD. However, the check is too strict: it only passes if the content type is equal to "application/ld+json". This excludes other valid strings like "application/ld+json; charset=utf-8". Specifying the charset is very common for users who create their response messages using System.Net.Http.dll (e.g., using StringContent).

An easy fix is to replace equality check with "StartsWith()".

Answer 1 · 2020-05-04T13:39:56.000Z

This issue has been automatically marked as stale because it has not had recent activity. After 30 days from now, it will be closed if no further activity occurs. Thank you for your contributions.

Answer 2 · 2020-07-02T13:56:54.000Z

Reconsidering as part of #60

Answer 3 · 2020-07-05T11:30:27.000Z

Should be implemented in conjunction with #21

Answer 4 · 2020-10-03T13:31:49.000Z

This issue has been automatically marked as stale because it has not had recent activity. After 30 days from now, it will be closed if no further activity occurs. Thank you for your contributions.