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

Add Target and Details properties to ODataError #76

Closed
lencharest opened this issue Feb 24, 2015 · 4 comments

Comments

Projects
None yet
5 participants
@lencharest
Copy link

commented Feb 24, 2015

OData v4 allows for target and details properties on an error response. The ODataError class should provide parity with the v4 error specification. Please add Target (type: String) and Details (type: IEnumerable<ODataErrorDetail>) properties, or unseal the ODataError class.

@congysu congysu added this to the 6.12 or Later milestone Mar 3, 2015

@congysu

This comment has been minimized.

Copy link
Contributor

commented Mar 3, 2015

@lencharest Thanks for reporting this issue. What is your user scenario?

@lencharest

This comment has been minimized.

Copy link
Author

commented Mar 3, 2015

My scenario is simply that I've written an OData-based service and I want to send rich(er) error responses. In practice, I'm using System.Net.Http.HttpRequestMessage.CreateErrorResponse to set the Response property on a System.Web.Http.Filters.HttpActionExecutedContext, and CreateErrorResponse takes an ODataError instance as an argument.

@lencharest

This comment has been minimized.

Copy link
Author

commented Mar 3, 2015

I should also mention that my service depends on 3rd-party OData-based services (e.g., Azure Table Storage). When my service receives an error response from one of its dependencies, I want to propagate some of the error info from the 3rd-party service by setting target and details in my error response.

@congysu congysu modified the milestones: 6.14, 6.12 Jul 24, 2015

congysu added a commit to congysu/odata.net that referenced this issue Oct 29, 2015

congysu added a commit to congysu/odata.net that referenced this issue Oct 29, 2015

@congysu congysu assigned congysu and unassigned VikingsFan Nov 13, 2015

congysu added a commit to congysu/odata.net that referenced this issue Nov 13, 2015

congysu added a commit to congysu/odata.net that referenced this issue Nov 16, 2015

OData error with target and details OData#76
Add writer and reader tests.
Update public API baseline.

congysu added a commit to congysu/odata.net that referenced this issue Nov 16, 2015

OData error with target and details OData#76
Add writer and reader tests.
Update public API baseline.

congysu added a commit to congysu/odata.net that referenced this issue Nov 16, 2015

OData error with target and details OData#76
Add writer and reader tests.
Update public API baseline.

congysu added a commit to congysu/odata.net that referenced this issue Nov 16, 2015

OData error with target and details OData#76
Add writer and reader tests.
Update public API baseline.

congysu added a commit to congysu/odata.net that referenced this issue Nov 17, 2015

OData error with target and details OData#76
Add writer and reader tests.
Update public API baseline.

congysu added a commit that referenced this issue Nov 19, 2015

OData error with target and details #76
Add writer and reader tests.
Update public API baseline.
@congysu

This comment has been minimized.

Copy link
Contributor

commented Dec 4, 2015

PR #364

@LianwMS LianwMS modified the milestones: 6.14, 6.15 Dec 21, 2015

@LaylaLiu LaylaLiu closed this Mar 15, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.