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

added gzip encoding support for outgoing requests #1068

Draft
wants to merge 2 commits into
base: dev-3.0.0
Choose a base branch
from

Conversation

yesoreyeram
Copy link
Collaborator

Fixes #1003 and addressed #779

How to test

Before changes

  • Select URL as source and paste https://infinity-demo.vercel.app/api/applications/billing/logs?startTime=${__from}&limit=10 as URL
  • Run the query and you will see results
  • Add a Accept-Encoding: gzip header manually
  • You will see the following error
image

After changes

  • Repeat the above steps
  • Don't need to add Accept-Encoding: gzip header manually as it is getting added by default in the backend
  • You will see the results as shown below. Also the query inspector will show you that the URL sent contain the Accept-Encoding header.
image

@yesoreyeram yesoreyeram changed the base branch from main to dev-3.0.0 December 1, 2024 23:23
@yesoreyeram yesoreyeram marked this pull request as draft December 1, 2024 23:28
@yesoreyeram yesoreyeram added this to the Version 3.0 milestone Dec 9, 2024
@yesoreyeram yesoreyeram self-assigned this Dec 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] Gzip Compression to improve retrieval performance
1 participant