Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Supporting gzipped data transfer between client and server #23

Open
anchita-g opened this issue Apr 6, 2023 · 2 comments
Open

Supporting gzipped data transfer between client and server #23

anchita-g opened this issue Apr 6, 2023 · 2 comments
Assignees
Milestone

Comments

@anchita-g
Copy link
Collaborator

  • Evaluate if all the components (Android FHIR SDK - FHIR Engine, Information Gateway) involved in the example for E2E sync in this repo support gzip encoding of data for transfer with the server.
  • Run a sample case of sync to validate the E2E sync's correctness.
  • It would be good to have some performance metrics to compare the effect of compressing the data for transfer on the entire sync time of resources.
@anchita-g anchita-g self-assigned this Apr 6, 2023
@anchita-g anchita-g added this to the Beta launch milestone Apr 6, 2023
@anchita-g
Copy link
Collaborator Author

anchita-g commented Apr 17, 2023

Update:
Android FHIR SDK uses OkHttp client which performs transparent gzip. It sends the required Accept-Encoding header with value gzip by default unless asked otherwise. If the server supports the encoding, it can response back with the respective header i.e. Content-Encoding = gzip. So there is no explicit support that is required to support gzip in Android FHIR SDK.

@anchita-g
Copy link
Collaborator Author

anchita-g commented Apr 17, 2023

Update:
As for FHIR Information Gateway, it does not allow relaying the Accept-Encoding header to the FHIR resource server. I have filed an issue to support gzip as an encoding.

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

No branches or pull requests

1 participant