Skip to content

gzip/deflate response through Proxy is not getting decompressed before consumed. #1583

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.

Sign up for GitHub

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

Jump to bottom
Closed
mallik-soupati opened this issue Oct 4, 2018 · 4 comments
Closed

gzip/deflate response through Proxy is not getting decompressed before consumed. #1583

mallik-soupati opened this issue Oct 4, 2018 · 4 comments
Assignees
@slandelle
Labels
Defect
Milestone
2.5.4

Comments

@mallik-soupati
Copy link

mallik-soupati commented Oct 4, 2018 •
edited
Loading

I have a situation where :
Browser ----> (proxy app:AsyncHttpClient) ----> Proxy ----> Origin Server( web site)
Ex: URL: https://example.com/startPage

While I can view the web pages correctly when accessed without Proxy; the browser shows (junk) gzip data when accessed the same pages through Proxy.

My proxy configuration doesn't unpack the gzip/deflate response received from Origin. I think the AsyncHttpClient should decompress it, before the response is consumed downstream.

Could you please look into it, or suggest any workaround. Please let me know if you need more information.

thanks,
@mallik-soupati mallik-soupati changed the title gzip/deflate response from Proxy is not getting decompressed before gzip/deflate response through Proxy is not getting decompressed before consumed. Oct 4, 2018
@slandelle slandelle self-assigned this Oct 5, 2018
@slandelle slandelle added this to the 2.5.4 milestone Oct 5, 2018
@slandelle
Copy link
Contributor

Fixed, thanks for reporting!

@mallik-soupati
Copy link
Author

mallik-soupati commented Oct 5, 2018 •
edited
Loading

Greatly appreciate your quickly addressing this issue, I needed it desperately. Is it possible to release this fix ? 2.5.4?

Apologies for commenting on the closed ticket.

@slandelle
Copy link
Contributor

Trying to, but Sonatype is broken (again)...

@mallik-soupati
Copy link
Author

Thanks, I will check back.

anexplore added a commit to anexplore/async-http-client that referenced this issue Oct 10, 2018
@anexplore
Merge branch 'master' of https://github.com/AsyncHttpClient/async-htt…
2dd9016 
…p-client

* 'master' of https://github.com/AsyncHttpClient/async-http-client:
  [maven-release-plugin] prepare for next development iteration
  [maven-release-plugin] prepare release async-http-client-project-2.5.4
  Upgrade netty 4.1.30.Final
  HttpContentDecompressor is misplaced after upgrading pipeline for HTTP tunnelling, close AsyncHttpClient#1583
  Fix regression introduced in d596056, close AsyncHttpClient#1274
  [maven-release-plugin] prepare for next development iteration
  [maven-release-plugin] prepare release async-http-client-project-2.5.3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@slandelle slandelle

Labels
Defect
Projects
None yet
Milestone
2.5.4
Development

No branches or pull requests
2 participants
@slandelle @mallik-soupati