[GitHub] tomcat pull request: now json is fashion

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 enabl

[GitHub] tomcat pull request: now json is fashion

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 --

[GitHub] tomcat pull request: now json is fashion

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 sub

[GitHub] tomcat pull request: now json is fashion

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

[GitHub] tomcat pull request: now json is fashion

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

[GitHub] tomcat pull request: now json is fashion

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 doe

[GitHub] tomcat pull request: now json is fashion

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 statemen

[GitHub] tomcat pull request: now json is fashion

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 appr

[GitHub] tomcat pull request: now json is fashion

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":"as

[GitHub] tomcat pull request: now json is fashion

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.

[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 c