Skip to content

Fix consumes MIME type for NetworkConnect#35542

Merged
thaJeztah merged 1 commit into
moby:masterfrom
masaeedu:patch-1
Nov 20, 2017
Merged

Fix consumes MIME type for NetworkConnect#35542
thaJeztah merged 1 commit into
moby:masterfrom
masaeedu:patch-1

Conversation

@masaeedu
Copy link
Copy Markdown
Contributor

- What I did
Changed the MIME type of the NetworkConnect route to the correct type.

- How I did it
The route expects application/json instead of application/octet-stream. This can be seen from the schema of the body parameter as well as from actually calling the route (it explodes if sent an application/octet-stream content type header).

- How to verify it
Try calling the route with the old and new MIME types for the Content-Type header. Only the new one will work.

- Description for the changelog

Changed the MIME type of the NetworkConnect route to the correct type.

- A picture of a cute animal (not mandatory but encouraged)
youhavedonemeabetray.jpg

This route expects `application/json`. Sending a content type header of `application/octet-stream` results in an error.

Signed-off-by: Asad Saeeduddin <[email protected]>
Copy link
Copy Markdown
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Member

@yongtang yongtang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@thaJeztah thaJeztah merged commit 4f4a027 into moby:master Nov 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants