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

config/show should be application/json #5335

Closed
hacdias opened this issue Aug 3, 2018 · 0 comments
Closed

config/show should be application/json #5335

hacdias opened this issue Aug 3, 2018 · 0 comments
Labels
kind/bug A bug in existing code (including security flaws)

Comments

@hacdias
Copy link
Member

hacdias commented Aug 3, 2018

Right now, go-ipfs HTTP API is responding to config?arg=key with:

Content-Type: application/json

But for config/show, we have:

Content-Type: text/plain

Since it's a JSON object, doesn't it make sense to change the Content Type to application/json on config/show?

Ref.: ipfs-inactive/js-ipfs-http-client#822

@Stebalien Stebalien added the kind/bug A bug in existing code (including security flaws) label Aug 6, 2018
Stebalien added a commit that referenced this issue Aug 6, 2018
This doesn't change there CLI output, it just means that the server responds
with a JSON output by default.

fixes #5335

License: MIT
Signed-off-by: Steven Allen <steven@stebalien.com>
Stebalien added a commit that referenced this issue May 30, 2019
This doesn't change there CLI output, it just means that the server responds
with a JSON output by default.

fixes #5335

License: MIT
Signed-off-by: Steven Allen <steven@stebalien.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug A bug in existing code (including security flaws)
Projects
None yet
Development

No branches or pull requests

2 participants