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

ODataLib doesn't write relative URIs in OData batch operations #271

Closed
object opened this Issue Jul 17, 2015 · 1 comment

Comments

Projects
None yet
4 participants
@object

object commented Jul 17, 2015

I received a change request regarding my library that generates OData requests using ODataLib. The library supports OData batch operations, and each operation is written using absolute URI - this is what ODataLib does.

Scenario when this doesn't work is a bit tricky and described here (object/Simple.OData.Client#122). There is a security layer and/or a load balancer in front of the processing server and the base URI for the OData request will no longer match the URI for each batch request in the body.

The developers tried to rewrite the generated batch payload, and as soon as they changed each operation's URI into a relative one, the batch request worked.

I tried to tweak ODataLib but didn't get the same result: it always writes the absolute URI for batch operations.

@congysu

This comment has been minimized.

Contributor

congysu commented Aug 7, 2015

See if can use client hooks to change the request uri: http://odata.github.io/odata.net/#04-06-use-client-hooks-in-odata-client

@LaylaLiu LaylaLiu added this to the 7.0 milestone Apr 1, 2016

@LaylaLiu LaylaLiu added in-progress and removed ready labels Apr 26, 2016

TomDu added a commit to TomDu/odata.net that referenced this issue May 10, 2016

TomDu added a commit to TomDu/odata.net that referenced this issue May 10, 2016

TomDu added a commit to TomDu/odata.net that referenced this issue May 10, 2016

TomDu added a commit to TomDu/odata.net that referenced this issue May 10, 2016

TomDu added a commit to TomDu/odata.net that referenced this issue May 10, 2016

TomDu added a commit to TomDu/odata.net that referenced this issue May 10, 2016

@TomDu TomDu added docs and removed in-progress labels May 10, 2016

@TomDu TomDu closed this Aug 1, 2016

@TomDu TomDu removed the docs label Aug 1, 2016

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