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

Enable compression #4

Closed
tommiehansen opened this issue Apr 26, 2016 · 3 comments
Closed

Enable compression #4

tommiehansen opened this issue Apr 26, 2016 · 3 comments
Assignees
Milestone

Comments

@tommiehansen
Copy link

How do one enable compression? Can't see or find any references to it in any headers within any of the classes?

@DerMika
Copy link
Collaborator

DerMika commented Apr 26, 2016

I've never needed compression before. I assume you're talking about the \SoapClient compression option?

@DerMika DerMika added this to the 0.1 milestone Apr 26, 2016
@DerMika
Copy link
Collaborator

DerMika commented Apr 26, 2016

Let me know if the commit above fixes your problem. If I understood your question, you can now set the compression by providing 'soapClientOptions' in the Session Handler parameters.

I wrote an example here: https://github.com/amabnl/amadeus-ws-client/blob/develop/docs/samples.rst#custom-soapclient-options

I'll merge this to master soon-ish.

@DerMika DerMika self-assigned this Apr 27, 2016
@DerMika
Copy link
Collaborator

DerMika commented Jun 15, 2016

Until reported otherwise, I'll assume the development mentioned above is the solution to the problem. So I'm closing this issue.

If not, please open a new issue!

@DerMika DerMika closed this as completed Jun 15, 2016
atomy pushed a commit to mlamm/amadeus-ws-client that referenced this issue Nov 26, 2018
…N-357-ama-request-service-search-finish to master

* commit '76a7b44df05cfcd2ad048129ac5685c97ce1f809':
  VAN-357 - AMA request service / search / finish free baggage allowance mapping - fixes after review
  VAN-357 - AMA request service / search / finish free baggage allowance mapping
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants