[jira] [Commented] (OLINGO-846) Flexible URL parsing for System Query Options

["Christian Amend (JIRA)" <ji...@apache.org>]

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

Christian Amend commented on OLINGO-846:
----------------------------------------

We are currently refactoring the URI parser on this branch: https://git-wip-us.apache.org/repos/asf?p=olingo-odata4.git;a=shortlog;h=refs/heads/OLINGO-834_Filter_Parser
The goal is to get rid of the AntLR dependency. There we could make this change as well. 

The question would be how far we would go with this. Should we also accept whitespaces inside the filter statement? For example something like "$filter=intexof( X  ,   'x')     ne   -   1"?
I am not sure if there are cases where an additional whitespace changes the meaning of the filter statement. So we would need to be prepared for that.

The easiest change would be to remove any leading or trailing whitespaces but sticking to the specification within the statement.

> Flexible URL parsing for System Query Options
> ---------------------------------------------
>
>                 Key: OLINGO-846
>                 URL: https://issues.apache.org/jira/browse/OLINGO-846
>             Project: Olingo
>          Issue Type: Improvement
>          Components: odata4-server
>    Affects Versions: (Java) V4 4.0.0-beta-01
>            Reporter: Ramesh Reddy
>
> Salesforce application is generating a OData URL with leading space for "$filter" option like
>  http://host:port/schema/Entity?$filter= indexof(X, 'x') ne -1
>  
> see the leading space before "indexof"? Is this legal URL? Olingo parser fails currently with malformed URL error.
> The community discussion is here [1], I am requesting same relaxation of rules in parsing in Olingo framework as in .Net framework from Microsoft
> [1] http://markmail.org/search/list:org.apache.olingo.dev#query:list%3Aorg.apache.olingo.dev+page:1+mid:7rojwqbwjpcsvffw+state:results



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)