You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ahu0605 <gi...@git.apache.org> on 2015/01/12 16:50:28 UTC
[GitHub] tomcat pull request: now json is fashion
GitHub user ahu0605 opened a pull request:
https://github.com/apache/tomcat/pull/15
now json is fashion
The class should support data json
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ahu0605/tomcat patch-1
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/tomcat/pull/15.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 #15
----
commit ca7a2e2d46298c5db6dc10e54e2209e2a5c708b0
Author: ahu0605 <zh...@gmail.com>
Date: 2015-01-12T15:50:15Z
now json is fashion
The class should support data json
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org
[GitHub] tomcat pull request: now json is fashion
Posted by martin-g <gi...@git.apache.org>.
Github user martin-g commented on the pull request:
https://github.com/apache/tomcat/pull/15#issuecomment-69742518
I'm pretty sure Jersey does something.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org
[GitHub] tomcat pull request: now json is fashion
Posted by ebourg <gi...@git.apache.org>.
Github user ebourg commented on the pull request:
https://github.com/apache/tomcat/pull/15#issuecomment-69768016
As I understand this pull request is more a request for enhancement than an actual code contribution. I guess you suggest that Tomcat implements the [W3C HTML JSON form submission](http://www.w3.org/TR/html-json-forms/) draft, correct?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org
[GitHub] tomcat pull request: now json is fashion
Posted by hazendaz <gi...@git.apache.org>.
Github user hazendaz commented on the pull request:
https://github.com/apache/tomcat/pull/15#issuecomment-69744890
Sure...what I meant here is that this patch does nothing from tomcat perspective. It's just an empty if statement. It does nothing at all.
--- Original Message ---
From: "Martin Grigorov" <no...@github.com>
Sent: January 13, 2015 8:18 AM
To: "apache/tomcat" <to...@noreply.github.com>
Cc: "Jeremy Landis" <je...@hotmail.com>
Subject: Re: [tomcat] now json is fashion (#15)
I'm pretty sure Jersey does something.
---
Reply to this email directly or view it on GitHub:
https://github.com/apache/tomcat/pull/15#issuecomment-69742518
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org
[GitHub] tomcat pull request: now json is fashion
Posted by ahu0605 <gi...@git.apache.org>.
Github user ahu0605 commented on the pull request:
https://github.com/apache/tomcat/pull/15#issuecomment-69769556
yes,from Perspective of development. it is benifit java ,it is benifit Tomcat ,our company has choice resin,if the problem keep alive, i maybe choice PHP or Python
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org
[GitHub] tomcat pull request: now json is fashion
Posted by hazendaz <gi...@git.apache.org>.
Github user hazendaz commented on the pull request:
https://github.com/apache/tomcat/pull/15#issuecomment-69741658
We run json using Jackson with jersey in tomcat. No issues. This code doesn't do anything. So unclear how it solves anything. All you did was add an empty if statement.
--- Original Message ---
From: "ahu0605" <no...@github.com>
Sent: January 12, 2015 10:09 PM
To: "apache/tomcat" <to...@noreply.github.com>
Subject: Re: [tomcat] now json is fashion (#15)
but when i push content type - json，there is some bug , 4 ex :data is json "{\"id\":\"asd\"}",the value will be a Map key, request.getPamaterMap () the key is {"id":"asd"}, value is null ,is not logic
---
Reply to this email directly or view it on GitHub:
https://github.com/apache/tomcat/pull/15#issuecomment-69689061
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org
[GitHub] tomcat pull request: now json is fashion
Posted by ahu0605 <gi...@git.apache.org>.
Github user ahu0605 commented on the pull request:
https://github.com/apache/tomcat/pull/15#issuecomment-69689061
but when i push content type - json，there is some bug , 4 ex :data is json "{\"id\":\"asd\"}",the value will be a Map key, request.getPamaterMap () the key is {"id":"asd"}, value is null ,is not logic
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org
[GitHub] tomcat pull request: now json is fashion
Posted by ahu0605 <gi...@git.apache.org>.
Github user ahu0605 commented on the pull request:
https://github.com/apache/tomcat/pull/15#issuecomment-69754848
I am very sad this bug is not be modifed, PHP had surpported content -type :JSON ,but java ...
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org
[GitHub] tomcat pull request: now json is fashion
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/tomcat/pull/15
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org
[GitHub] tomcat pull request: now json is fashion
Posted by markt-asf <gi...@git.apache.org>.
Github user markt-asf commented on the pull request:
https://github.com/apache/tomcat/pull/15#issuecomment-69624704
This pull request is declined. Even ignoring the poor quality of this pull request, supporting JSON formatted data is an application concern, not a container concern.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org
[GitHub] tomcat pull request: now json is fashion
Posted by martin-g <gi...@git.apache.org>.
Github user martin-g commented on the pull request:
https://github.com/apache/tomcat/pull/15#issuecomment-69703890
When you POST JSON data you should use http://docs.oracle.com/javaee/6/api/javax/servlet/ServletRequest.html#getReader() to consume the body and parse it as you find appropriate.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org