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

Feature Request: Ability to Specify "process: false" at Operation Level #172

Open
GuyPaddock opened this issue Sep 7, 2019 · 0 comments

Comments

@GuyPaddock
Copy link

We are working with a service that is mostly JSON but that has some resources that return binary data (images). Trying to accommodate this with the current version of Guzzle Services is inconvenient because the option as to whether or not a response should be parsed/processed is specified in the service client config instead of in the operation config.

My current workaround is that all of our requests go through a services tier/request facade. That facade uses one version of the Guzzle services client for requests that need JSON, and a different version for binary requests. But that also means we have to remember not to invoke the wrong service commands on the wrong client, OR we have to keep the configurations for the two types of clients separate.

Instead, it would be great if I could provide a config like this:

operations:
  getFile:
    httpMethod: 'GET'
    process: false
    uri: 'serve/{file_id}'
    parameters:
      file_id:
        location: 'uri'
        description: 'The unique identifier of the file being requested.'
        type: 'integer'
        required: true
        minimum: 1

The key is process: false. This is not currently supported.

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