OFFI-149: HttpContent and ZipArchive extensions, UserReadableException

[sarahelsaig]

OFFI-149

[wAsnk]

Add this to https://github.com/Lombiq/Helpful-Libraries/blob/dev/Lombiq.HelpfulLibraries.Common/Readme.md

[wAsnk]

Add this to https://github.com/Lombiq/Helpful-Libraries/blob/dev/Lombiq.HelpfulLibraries.AspNetCore/Docs/Extensions.md

[sarahelsaig]

@wAsnk If you are satisfied with the HL and OSOCE PRs, please merge them ahead of time. This is because "Upgrade to Orchard Core 2.1.2" of OSOE-903 is to be merged in very soon (I was just reminded to do the second review pass for it) and I don't want to delay this issue with having to do another unnecessary OC upgrade in OFFI.

Copilot reviewed 5 out of 9 changed files in this pull request and generated no suggestions.

Files not reviewed (4)

• Lombiq.HelpfulLibraries.Common/Docs/Exceptions.md: Evaluated as low risk
• Lombiq.HelpfulLibraries.Common/Exceptions/UserReadableException.cs: Evaluated as low risk
• Lombiq.HelpfulLibraries.Common/Extensions/ZipArchiveExtensions.cs: Evaluated as low risk
• Lombiq.HelpfulLibraries.Common/Readme.md: Evaluated as low risk

Comments skipped due to low confidence (3)

Lombiq.HelpfulLibraries.AspNetCore/Extensions/HttpContentExtensions.cs:31

• [nitpick] The variable name 'xml' is misleading. It should be renamed to 'fileContent'.

var xml = new ByteArrayContent(content);

Lombiq.HelpfulLibraries.AspNetCore/Extensions/HttpContentExtensions.cs:54

• The 'mediaType' parameter should be checked for null before proceeding to avoid a potential null reference exception.

string mediaType = null)

Lombiq.HelpfulLibraries.OrchardCore/Mvc/OrchardControllerExtensions.cs:12

• Ensure that the LogJsonError method is covered by tests to verify its behavior, especially the serialization of route values.

using System.Text.Json;