Skip to content

Fix batch failed with relative url & Content-Transfer-Encoding have duplicate binary#7

Open
sleidor wants to merge 5 commits intopanoramicdata:mainfrom
sleidor:main
Open

Fix batch failed with relative url & Content-Transfer-Encoding have duplicate binary#7
sleidor wants to merge 5 commits intopanoramicdata:mainfrom
sleidor:main

Conversation

@sleidor
Copy link

@sleidor sleidor commented Dec 25, 2025

Exception with relative URLs

  • Add check for IsAbsoluteUri during HttpMessageContent;

Duplicate binary in Content-Transfer-Encoding

  • Remove duplicate headers in PanoramicData.OData.Client/ODataClient.Batch.cs, as constructor HttpMessageContent already init with that headers;

UnitTests

  • mark datetime UTC for Read_SimpleDateFormat_ParsesCorrectly(), it will failed if your local timezone is UTC + number;
  • add WriteToStreamAsync_ShouldIncludeRequiredHeaders() for checking headers in batch;

@codacy-production
Copy link

codacy-production bot commented Mar 19, 2026

Codacy's Analysis Summary

4 new issues (≤ 0 issue)
0 new security issue
4 complexity
0 duplications
More details

AI Reviewer: run a review on demand. To trigger the first review automatically, go to your organization or repository integration settings. AI can make mistakes. Always validate suggestions.

Run reviewer

TIP This summary will be updated as you push new changes. Give us feedback

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant