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

Use :format => "plain" for newline-delimited output #15

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

paulmakepeace
Copy link
Contributor

The OpenStack Object Storage API allows the response body format to be given via an Accept header orformat parameter. So if the latter is given then don't set the Accept header. This replaces the json: false option.

(Didn't realize this possibility in a previous PR!)

The OpenStack Object Storage API allows the response body format to be given via an Accept header or`format` parameter. So if the latter is given then don't set the Accept header. This replaces the `json: false` option.
README.md Outdated Show resolved Hide resolved
@maikonas
Copy link

#12 also introduces opts mutation, which doesn't work on constants.

following code breaks in 0.3.0

client_get_objects('container', {}, OPTIONS)
FrozenError: can't modify frozen Hash
    ../swift_client/lib/swift_client.rb:198:in `delete'

Would be nice to release this :)

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

Successfully merging this pull request may close these issues.

3 participants