olingo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Christian Amend (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (OLINGO-1021) Unable to consume batch request with missing CRLF in request line
Date Thu, 23 Mar 2017 13:01:41 GMT

    [ https://issues.apache.org/jira/browse/OLINGO-1021?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15938237#comment-15938237

Christian Amend commented on OLINGO-1021:

Then I would make this change as I don`t want to change the default behavior that we currently

> Unable to consume batch request with missing CRLF in request line
> -----------------------------------------------------------------
>                 Key: OLINGO-1021
>                 URL: https://issues.apache.org/jira/browse/OLINGO-1021
>             Project: Olingo
>          Issue Type: Bug
>          Components: odata2-core
>    Affects Versions: V2 2.0.7
>            Reporter: Dmitry Tretyakov
>            Priority: Critical
>              Labels: patch
>             Fix For: V2 2.0.9
>         Attachments: 0002-OLINGO-1021-Add-ability-to-use-non-strict-batch-pars.patch
> Currently {{BatchParser}} throws BatchException(MISSING_BLANK_LINE) when request method
does not end with double CRLFs and this case reproduced in the test {{BatchRequestParserTest#testGetRequestMissingCRLF}}
> This behavior makes odata2-core incompatible with all implementations of .net OData clients
based on {{System.Data.Services.Client}} because it use only one CRLF at the end of request
> http://referencesource.microsoft.com/#System.Data.Services.Client/Client/System/Data/Services/Client/DataServiceContext.cs,3277289aacaeb96c
> which results in the following request body:
> {code}
> --batch_86108a65-4dc2-4c0c-bd92-384be755c83a
> Content-Type: application/http
> Content-Transfer-Encoding: binary
> GET https://www.nuget.org/api/v2/Packages HTTP/1.1
> --batch_86108a65-4dc2-4c0c-bd92-384be755c83a--
> {code}
> I see that {{BatchParser}}'s constructor allows to use non strict flag, but at the moment
it's impossible to set due to hardcoded value in ProviderFacadeImpl#parseBatchRequest method.

This message was sent by Atlassian JIRA

View raw message