Skip to content
This repository was archived by the owner on Dec 14, 2018. It is now read-only.

Content Negotiation need to return 400 Bad Request \ 415 unsupported media type #458

Closed
NTaylorMullen opened this issue May 10, 2014 · 4 comments
Closed

Content Negotiation need to return 400 Bad Request \ 415 unsupported media type #458

NTaylorMullen opened this issue May 10, 2014 · 4 comments
Assignees
@harshgMSFT
Labels
3 - Done bug
Milestone
1.0.0-alpha4

Comments

@NTaylorMullen
Copy link
Contributor

In the past with WebAPI, we'd throw HttpResponseException with the right status code set and something in the pipeline would interpret it correctly. We need to figure out how to do this with formatters.

| 159 | 10317 | Reported By: Pranav K |
@danroth27 danroth27 added this to the Post Alpha milestone May 13, 2014
@pranavkm pranavkm changed the title Formatters need to return 400 Bad Request \ 415 unsupported media type Content Negotiation need to return 400 Bad Request \ 415 unsupported media type May 13, 2014
@yishaigalatzer yishaigalatzer removed this from the Post Alpha milestone May 23, 2014
@yishaigalatzer yishaigalatzer mentioned this issue Jun 2, 2014
Closed
@yishaigalatzer
Copy link
Contributor

Need more discussion around is this supposed to return 406 or 415.

@yishaigalatzer
Copy link
Contributor

https://github.com/aspnet/Mvc/blob/dev/src/Microsoft.AspNet.Mvc.ModelBinding/Formatters/TempInputFormatterProvider.cs#L34

@danroth27 danroth27 modified the milestones: 6.0.0-alpha3, 1.0.0-alpha3 Jul 7, 2014
@harshgMSFT harshgMSFT mentioned this issue Jul 28, 2014
Closed
@danroth27 danroth27 modified the milestones: 6.0.0-alpha3, 6.0.0-alpha4 Aug 12, 2014
@harshgMSFT harshgMSFT mentioned this issue Aug 20, 2014
Merged
@harshgMSFT
Copy link
Contributor

#458

@harshgMSFT
Copy link
Contributor

#1046

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@harshgMSFT harshgMSFT

Labels
3 - Done bug
Projects
None yet
Milestone
1.0.0-alpha4
Development

No branches or pull requests
4 participants
@yishaigalatzer @danroth27 @NTaylorMullen @harshgMSFT