Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

FI-2783: Turn off Accept Charset Request Header #620

Merged
merged 4 commits into from
Feb 17, 2025

Conversation

vanessuniq
Copy link
Contributor

This branch turn removed the Accept Charset Header from the FHIR Client requests. I also investigated and other http client (Faraday) doesn't automatically set this header.

Signed-off-by: Vanessa Fotso <vfotso@mitre.org>
Copy link

codecov bot commented Feb 13, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.71%. Comparing base (9571dff) to head (09c9639).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #620   +/-   ##
=======================================
  Coverage   84.71%   84.71%           
=======================================
  Files         283      283           
  Lines       12401    12402    +1     
  Branches     1503     1503           
=======================================
+ Hits        10505    10506    +1     
  Misses       1888     1888           
  Partials        8        8           
Flag Coverage Δ
backend 92.33% <100.00%> (+<0.01%) ⬆️
frontend 79.20% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@vanessuniq vanessuniq merged commit 9a521f2 into main Feb 17, 2025
10 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants