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

Giacomo Pati commented on OLINGO-480:
-------------------------------------

Hi Michael

As I've wrote in the Jira description, according to the javadocs it might
not be the right place. I've asked myself, why we need our own decoder if
java.net.URLDecode could do?

Ciao & thanks





-- 
Giacomo


> good old '+' as a encoding for a space is not handled in Decoder class
> ----------------------------------------------------------------------
>
>                 Key: OLINGO-480
>                 URL: https://issues.apache.org/jira/browse/OLINGO-480
>             Project: Olingo
>          Issue Type: Bug
>          Components: odata2-core
>    Affects Versions: V2 2.0.1
>            Reporter: Giacomo Pati
>            Assignee: Michael Bolz
>             Fix For: V2 2.0.2
>
>         Attachments: 0001-Change-Decoder-class-to-respect-also-as-space.patch
>
>
> A request like wget 
> http://localhost:8080/SalesOrderProcessing.svc/SalesOrderItems?$filter=Delivered+eq+true
>  should be accepted as commonly a '+' is encoding a space. The Decoder class 
> so far only decodes percent-encoded values and leaves the plus sign as is. 
> The applied patch adds this decoding to the Decoder class (maybe it's not the 
> right place to put it but I was not able to find any other place)



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

Reply via email to