You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@stanbol.apache.org by "Florent ANDRE (JIRA)" <ji...@apache.org> on 2014/06/23 14:20:25 UTC

[jira] [Comment Edited] (STANBOL-1350) HttpQueryHeader filter consumes content if Content-Type is set to "application/x-www-form-urlencoded"

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

Florent ANDRE edited comment on STANBOL-1350 at 6/23/14 12:20 PM:
------------------------------------------------------------------

this patch contains some others modifications : 

******
commons/namespaceprefix/service/src/main/java/org/apache/stanbol/commons/namespaceprefix/NamespacePrefixService.java

=> add  a method to the interface to import mapping from inputstreams

******
entityhub/jersey/pom.xml

=> move all stanbol's module dependencies to a unified (0.12.1-SNAPSHOT) one.
This was done throw maven property... Seems to remember that it's not really like here... but found it a really easy and clean way to stop the hassle on version numbering inside each module...

*****
entityhub/jersey/src/test/java/org/apache/stanbol/entityhub/jersey/writers/QueryResultsToRDFTest.java

=> Test case. 
This also show a commented potential problem when serializing RDFQueryResultsList to Json (that don't append when serializing QueryResultList.

****
entityhub/model/clerezza/src/main/java/org/apache/stanbol/entityhub/model/clerezza/RdfRepresentation.java

=> contains the fix : 
     public TripleCollection getRdfGraph(){
-        return graphNode.getGraph();
+       return graphNode.getNodeContext();
     }

=> and a toJson method to make the toString result more human readable.


was (Author: flo4sengines):
this patch contains some others modifications : 

******
commons/namespaceprefix/service/src/main/java/org/apache/stanbol/commons/namespaceprefix/NamespacePrefixService.java

=> add  a method to the interface to import mapping from inputstreams

******
entityhub/jersey/pom.xml

=> move all stanbol's module dependencies to a unified (0.12.1-SNAPSHOT) one.
This was done throw maven property... Seems to remember that it's not really like here... but found it a really easy and clean way to stop the hassle on version numbering inside each module...

*****
entityhub/jersey/src/test/java/org/apache/stanbol/entityhub/jersey/writers/QueryResultsToRDFTest.java

=> Test case. 
This also show a commented potential problem when serializing RDFQueryResultsList to Json (that don't append when serializing QueryResultList.

entityhub/model/clerezza/src/main/java/org/apache/stanbol/entityhub/model/clerezza/RdfRepresentation.java

=> contains the fix : 
     public TripleCollection getRdfGraph(){
-        return graphNode.getGraph();
+       return graphNode.getNodeContext();
     }

=> and a toJson method to make the toString result more human readable.

> HttpQueryHeader filter consumes content if Content-Type is set to "application/x-www-form-urlencoded"
> -----------------------------------------------------------------------------------------------------
>
>                 Key: STANBOL-1350
>                 URL: https://issues.apache.org/jira/browse/STANBOL-1350
>             Project: Stanbol
>          Issue Type: Bug
>          Components: Commons
>    Affects Versions: 0.12.0
>            Reporter: Rupert Westenthaler
>            Assignee: Rupert Westenthaler
>             Fix For: 1.0.0, 0.12.1
>
>         Attachments: STANBOL-1350.patch
>
>
> The QueryHeadersFilter does call `request.getParameter*(..)` methods on the ServletRequest. This will cause the content to be consumed in case the Content-Type is set to "application/x-www-form-urlencoded".
> This is clearly not intended as this filter is only expected to process query parameters.



--
This message was sent by Atlassian JIRA
(v6.2#6252)