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

ASF GitHub Bot commented on WW-4558:
------------------------------------

GitHub user victorsosa reopened a pull request:

    https://github.com/apache/struts/pull/87

    WW-4558 contentType override ignored for JSONInterceptor

    From now on:
    
    The "accept" request header parameter must be "application/json" or 
"application/json-rpc"
    
    Also the default encoding is "UTF-8"

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/victorsosa/struts ww-4558

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/struts/pull/87.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #87
    
----
commit e05f9f01804bc64ace15bfc573ef483da583e5a7
Author: victor sosa <[email protected]>
Date:   2016-01-21T12:00:54Z

    Merge pull request #3 from apache/master
    
    update pull

commit 778eec3b94f283290f186c8caffd10f03576fa27
Author: victor sosa <[email protected]>
Date:   2016-01-26T12:38:50Z

    Merge pull request #4 from apache/master
    
    update pull

commit 46bd92fe202041c76704b96d11a0b8f8e83d56de
Author: victor sosa <[email protected]>
Date:   2016-01-29T21:54:27Z

    Merge pull request #5 from apache/master
    
    update pull

commit a2c5bc835ec06483af6175fc0abf67714fb28711
Author: victor sosa <[email protected]>
Date:   2016-02-04T17:12:22Z

    Merge pull request #6 from apache/master
    
    pull update

commit fab687a07e8d873c82465b29188100debf61097f
Author: victorsosa <[email protected]>
Date:   2016-02-15T12:28:01Z

    fix patch WW-4558
    
    contentType override ignored for JSONInterceptor
    
    From now on:
    
    The "accept" request header parameter must be "application/json" or
    "application/json-rpc"
    
    Also the default encoding is "UTF-8"

commit 0d4905038119f614f021af5738154079b30bca24
Author: victorsosa <[email protected]>
Date:   2016-02-15T12:28:01Z

    fix patch WW-4558
    
    contentType override ignored for JSONInterceptor
    
    use of accept parameter st by the interceptor params
    
    
    Signed-off-by: victorsosa <[email protected]>

commit bab451c6d80ee9b2685570d797e505768ff8bbe4
Author: victorsosa <[email protected]>
Date:   2016-02-15T14:34:15Z

    Merge branch 'ww-4558' of github.com:victorsosa/struts into ww-4558
    
    # Conflicts:
    #   plugins/json/src/main/java/org/apache/struts2/json/JSONInterceptor.java

commit 2fada1ec05e61dfb0b3e43ab447a7673d689c684
Author: victorsosa <[email protected]>
Date:   2016-02-15T14:34:15Z

    Merge branch 'ww-4558' of github.com:victorsosa/struts into ww-4558
    
    # Conflicts:
    #   plugins/json/src/main/java/org/apache/struts2/json/JSONInterceptor.java

commit 8b8135af79550bf732d5aaa98ee7d3d5116e4b4b
Author: victorsosa <[email protected]>
Date:   2016-02-15T14:37:26Z

    Merge branch 'ww-4558' of github.com:victorsosa/struts into ww-4558
    
    # Conflicts:
    #   plugins/json/src/main/java/org/apache/struts2/json/JSONInterceptor.java

----


> contentType override ignored for JSONInterceptor
> ------------------------------------------------
>
>                 Key: WW-4558
>                 URL: https://issues.apache.org/jira/browse/WW-4558
>             Project: Struts 2
>          Issue Type: Bug
>          Components: Plugin - JSON
>    Affects Versions: 2.3.24
>            Reporter: Jasper Rosenberg
>            Priority: Minor
>             Fix For: 2.5
>
>
> JSONInterceptor takes a contentType parameter, but as far as I can tell does 
> nothing with it.  I would think a reasonable way to use it would be to have 
> it used if available, rather than the content type from the request so that 
> the interceptor can be used when receiving a JSON post from a third party 
> that does not set the correct content type.
> PR https://github.com/apache/struts/pull/87



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

Reply via email to