Well in two words if you take the most common example of JAXRS 2
continuation (@Suspended) it will do it (doing the link with
servlets):
-> request 1 -> suspend async context
-> request 2 -> async context dispatch (resume)
issue is that resume is called from request 2. In other words
ContainerThr
On 11/02/2015 06:25, Pravallika Peddi wrote:
> Hi Mark,
> Attached the fix details.
Patches should be in diff -u format and attached to the bug report to
which they relate.
Mark
> I have modified AprLifecycleListener.java and Library.java to fix the
> issue. Added a new file "LibraryNotFoundErr
On 11/02/2015 07:58, Romain Manni-Bucau wrote:
> Well in two words if you take the most common example of JAXRS 2
> continuation (@Suspended) it will do it (doing the link with
> servlets):
>
> -> request 1 -> suspend async context
> -> request 2 -> async context dispatch (resume)
>
> issue is th
On 11/02/2015 00:40, Konstantin Kolinko wrote:
> 2015-02-09 13:30 GMT+03:00 :
>> Author: markt
>> Date: Mon Feb 9 10:30:40 2015
>> New Revision: 1658351
>>
>> URL: http://svn.apache.org/r1658351
>> Log:
>> Ensure only \${ and \#{ are treated as escapes for ${ and #{ rather than \$
>> and \# bein
Well it is not different and both should be allowed IMO, i just tried
to find a solution for the original bug without breaking common
usages. It should be easy to reproduce with a servlet: GET
/foo?start=true -> create an async context then GET
/foo?continuer=true -> resume without popping a new th
https://issues.apache.org/bugzilla/show_bug.cgi?id=57546
--- Comment #9 from Mark Thomas ---
See r1658734 and r1658790.
--
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-un
On 11/02/2015 09:21, Romain Manni-Bucau wrote:
> Well it is not different and both should be allowed IMO, i just tried
> to find a solution for the original bug without breaking common
> usages. It should be easy to reproduce with a servlet: GET
> /foo?start=true -> create an async context then GET
2015-02-11 10:15 GMT+01:00 Mark Thomas :
> On 11/02/2015 07:58, Romain Manni-Bucau wrote:
> > Well in two words if you take the most common example of JAXRS 2
> > continuation (@Suspended) it will do it (doing the link with
> > servlets):
> >
> > -> request 1 -> suspend async context
> > -> reques
Ok, let's look a jaxrs 2 sample and bind underlying implementation:
@Path("touch")
@ApplicationScoped
public class Endpoint {
private volatile AsyncResponse current;
@GET
public void async(@Suspended final AsyncResponse response) {
if (current == nu
https://issues.apache.org/bugzilla/show_bug.cgi?id=57021
VIN changed:
What|Removed |Added
CC||reachme.va...@gmail.com
--- Comment #2 from
Hi Mark,
I have created the patch file as you mentioned and attached it to the bug
report, i would like to confirm on one thing:
I have added two new strings to the LocalStrings.properties file which has
Locale specific strings. Some one has to help to add the new strings to the
other Locale file
https://issues.apache.org/bugzilla/show_bug.cgi?id=57021
--- Comment #3 from VIN ---
Sorry, I could not attach the log files. This portal is not allowing me to
attach the logs which are generated with the fix.
--
You are receiving this mail because:
You are the assignee for the bug.
--
On 11/02/2015 11:32, Pravallika Peddi wrote:
> Hi Mark,
>
> I have created the patch file as you mentioned and attached it to the bug
> report, i would like to confirm on one thing:
>
> I have added two new strings to the LocalStrings.properties file which has
> Locale specific strings. Some one
Author: markt
Date: Wed Feb 11 12:03:18 2015
New Revision: 1658940
URL: http://svn.apache.org/r1658940
Log:
Follow-up to r1658351.
Back-port additional patch required to complete this fix.
Modified:
tomcat/tc7.0.x/trunk/ (props changed)
tomcat/tc7.0.x/trunk/java/org/apache/jasper/compil
Thanks for your guidance in this Mark and Chris.
Hope my first fix will be accepted.
On Wed, Feb 11, 2015 at 5:16 PM, Mark Thomas wrote:
> On 11/02/2015 11:32, Pravallika Peddi wrote:
> > Hi Mark,
> >
> > I have created the patch file as you mentioned and attached it to the bug
> > report, i w
Author: markt
Date: Wed Feb 11 13:04:00 2015
New Revision: 1658948
URL: http://svn.apache.org/r1658948
Log:
Fix Javadoc error
Modified:
tomcat/trunk/java/org/apache/coyote/AsyncContextCallback.java
Modified: tomcat/trunk/java/org/apache/coyote/AsyncContextCallback.java
URL:
http://svn.apach
On 11/02/2015 09:35, Romain Manni-Bucau wrote:
> Ok, let's look a jaxrs 2 sample and bind underlying implementation:
>
> @Path("touch")
> @ApplicationScoped
> public class Endpoint {
> private volatile AsyncResponse current;
>
> @GET
> public void async(@Suspen
2015-02-11 7:40 GMT+03:00 Jeremy Boynes :
> The proposed Apache Taglibs 1.2.3 release is now available for voting.
>
> It can be obtained from:
> https://dist.apache.org/repos/dist/dev/tomcat/taglibs/taglibs-standard-1.2.3/
>
> The Maven staging repo is:
> https://repository.apache.org/content/re
that's it I think
If you want servlet code let's do it (wanted to avoid code in mail
since I never manage to format it well):
1) request 1 (start the async context) does:
// field context has type AsyncContext
context = req.startAsync(req, resp);
context.addListener(this); // not important for u
https://issues.apache.org/bugzilla/show_bug.cgi?id=57564
Christopher Schultz changed:
What|Removed |Added
Keywords||PatchAvailable
--
You are r
Author: markt
Date: Wed Feb 11 14:22:00 2015
New Revision: 1658966
URL: http://svn.apache.org/r1658966
Log:
Follow up to r1594198
Limit cases where a dispatch is not performed because processing is already on
a poller thread.
Modified:
tomcat/trunk/java/org/apache/coyote/AsyncStateMachine.ja
Author: markt
Date: Wed Feb 11 14:27:16 2015
New Revision: 1658969
URL: http://svn.apache.org/r1658969
Log:
Follow up to r1594198
Limit cases where a dispatch is not performed because processing is already on
a poller thread.
Modified:
tomcat/tc8.0.x/trunk/ (props changed)
tomcat/tc8.0
Author: markt
Date: Wed Feb 11 14:29:15 2015
New Revision: 1658970
URL: http://svn.apache.org/r1658970
Log:
Update changelog
Modified:
tomcat/tc8.0.x/trunk/webapps/docs/changelog.xml
Modified: tomcat/tc8.0.x/trunk/webapps/docs/changelog.xml
URL:
http://svn.apache.org/viewvc/tomcat/tc8.0.x/t
On 11/02/2015 13:54, Romain Manni-Bucau wrote:
> that's it I think
OK. Patch applied to trunk and 8.0.x. Let us know how you get on.
Mark
>
> If you want servlet code let's do it (wanted to avoid code in mail
> since I never manage to format it well):
>
> 1) request 1 (start the async context
https://issues.apache.org/bugzilla/show_bug.cgi?id=57021
Christopher Schultz changed:
What|Removed |Added
Keywords||PatchAvailable
--- Comment #
all green with the trunk. That you very much.
Romain Manni-Bucau
@rmannibucau
http://www.tomitribe.com
http://rmannibucau.wordpress.com
https://github.com/rmannibucau
2015-02-11 15:31 GMT+01:00 Mark Thomas :
> On 11/02/2015 13:54, Romain Manni-Bucau wrote:
>> that's it I think
>
> OK. Patch app
https://issues.apache.org/bugzilla/show_bug.cgi?id=57534
--- Comment #3 from Mark Thomas ---
You are right.
When I first looked at this I started to look at how the request content-type
header was parsed and that has some issues that would need
o.a.tomcat.util.http.parser.MediaType to solve but
https://issues.apache.org/bugzilla/show_bug.cgi?id=57021
--- Comment #5 from Mark Thomas ---
@Chris the difference with WARN and DEBUG is that DEBUG includes the stack
trace. I'd say if debug is not enabled, log at WARN, else log at DEBUG so you
never get more than one message.
Some more comment
https://issues.apache.org/bugzilla/show_bug.cgi?id=57021
--- Comment #6 from Christopher Schultz ---
(In reply to Mark Thomas from comment #5)
> @Chris the difference with WARN and DEBUG is that DEBUG includes the stack
> trace. I'd say if debug is not enabled, log at WARN, else log at DEBUG so
>
https://issues.apache.org/bugzilla/show_bug.cgi?id=57489
--- Comment #11 from Mark Thomas ---
I've fixed another issue. I don't think it will make any different but if you
could do a quick test it would be appreciated.
I've just noticed that the stack trace passes through the onOpen method of th
https://issues.apache.org/bugzilla/show_bug.cgi?id=57528
Mark Thomas changed:
What|Removed |Added
Status|NEW |NEEDINFO
--
You are receiving this
Author: markt
Date: Wed Feb 11 19:18:46 2015
New Revision: 1659043
URL: http://svn.apache.org/r1659043
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=57534
CORS Filter should only look at media type component of content type
Modified:
tomcat/trunk/java/org/apache/catalina/filters
Author: markt
Date: Wed Feb 11 19:22:42 2015
New Revision: 1659045
URL: http://svn.apache.org/r1659045
Log: (empty)
Modified:
tomcat/tc8.0.x/trunk/ (props changed)
tomcat/tc8.0.x/trunk/java/org/apache/catalina/filters/CorsFilter.java
tomcat/tc8.0.x/trunk/test/org/apache/catalina/fil
Author: markt
Date: Wed Feb 11 19:24:16 2015
New Revision: 1659046
URL: http://svn.apache.org/r1659046
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=57534
CORS Filter should only look at media type component of content type
Modified:
tomcat/tc7.0.x/trunk/ (props changed)
t
https://issues.apache.org/bugzilla/show_bug.cgi?id=57534
Mark Thomas changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://issues.apache.org/bugzilla/show_bug.cgi?id=45014
--- Comment #24 from mr.oliver.hernan...@gmail.com ---
Created attachment 32456
--> https://issues.apache.org/bugzilla/attachment.cgi?id=32456&action=edit
Maven project for a valve that dumps the full request with the body
NOTE: For our u
https://issues.apache.org/bugzilla/show_bug.cgi?id=45014
mr.oliver.hernan...@gmail.com changed:
What|Removed |Added
Attachment #32446|0 |1
is obsolet
We've had a couple of important Coyote fixes since the last release so
I'm planning on tagging 8.0.x tomorrow.
In my mind this is a slightly delayed beginning of Feb release. I plan
to get back to a roughly beginning of the month schedule next month.
Mark
https://issues.apache.org/bugzilla/show_bug.cgi?id=57540
--- Comment #7 from Mark Thomas ---
Reviewing the proposed patch:
1. The changes to tomcat/util/net/jsse/openssl/Protocol.java need to be removed
2. There looks to be the right hooks in tc-native to obtain the protocol so do
that rather t
Author: markt
Date: Wed Feb 11 19:59:58 2015
New Revision: 1659053
URL: http://svn.apache.org/r1659053
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=57564
Make JspC amenable to sub-classing
Patch provided by Jan Bartel
Modified:
tomcat/trunk/java/org/apache/jasper/JspC.java
Mod
Author: markt
Date: Wed Feb 11 20:01:23 2015
New Revision: 1659054
URL: http://svn.apache.org/r1659054
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=57564
Make JspC amenable to sub-classing
Patch provided by Jan Bartel
Modified:
tomcat/tc8.0.x/trunk/ (props changed)
tomcat
https://issues.apache.org/bugzilla/show_bug.cgi?id=57564
Mark Thomas changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://issues.apache.org/bugzilla/show_bug.cgi?id=57432
Konstantin Kolinko changed:
What|Removed |Added
Component|Library |Connectors
Version
Author: markt
Date: Wed Feb 11 20:32:57 2015
New Revision: 1659059
URL: http://svn.apache.org/r1659059
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=57432
Align SSL_OP_NO_TLSv1_1 and SSL_OP_NO_TLSv1_2 constant values with OpenSSL
(they had been swapped).
Modified:
tomcat/trunk/
Author: markt
Date: Wed Feb 11 20:36:37 2015
New Revision: 1659060
URL: http://svn.apache.org/r1659060
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=57432
Align SSL_OP_NO_TLSv1_1 and SSL_OP_NO_TLSv1_2 constant values with OpenSSL
(they had been swapped).
Modified:
tomcat/tc8.0.
https://issues.apache.org/bugzilla/show_bug.cgi?id=57432
Mark Thomas changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://issues.apache.org/bugzilla/show_bug.cgi?id=57532
Mark Thomas changed:
What|Removed |Added
Status|NEEDINFO|RESOLVED
Resolution|---
https://issues.apache.org/bugzilla/show_bug.cgi?id=57570
Bug ID: 57570
Summary: Make processing of chunked encoding trailers an opt-in
feature (align with HTTPD 2.4.12)
Product: Tomcat 8
Version: 8.0.18
Hardware: PC
https://issues.apache.org/bugzilla/show_bug.cgi?id=57570
Konstantin Kolinko changed:
What|Removed |Added
OS||All
--- Comment #1 from Konst
2015-02-11 22:18 GMT+03:00 :
> Author: markt
> Date: Wed Feb 11 19:18:46 2015
> New Revision: 1659043
>
> URL: http://svn.apache.org/r1659043
> Log:
> Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=57534
> CORS Filter should only look at media type component of content type
>
> Modified:
>
To whom it may engage...
This is an automated request, but not an unsolicited one. For
more information please visit http://gump.apache.org/nagged.html,
and/or contact the folk at gene...@gump.apache.org.
Project tomcat-tc8.0.x-test-nio2 has an issue affecting its community
integration
To whom it may engage...
This is an automated request, but not an unsolicited one. For
more information please visit http://gump.apache.org/nagged.html,
and/or contact the folk at gene...@gump.apache.org.
Project tomcat-tc8.0.x-test-apr has an issue affecting its community
integration.
https://issues.apache.org/bugzilla/show_bug.cgi?id=57573
Bug ID: 57573
Summary: Host Header Internal IP Address Disclosure
Product: Tomcat 6
Version: 6.0.4
Hardware: All
OS: Linux
Status: NEW
Severity: nor
https://issues.apache.org/bugzilla/show_bug.cgi?id=57573
Christopher Schultz changed:
What|Removed |Added
Status|NEW |NEEDINFO
--- Comment #1 from
To whom it may engage...
This is an automated request, but not an unsolicited one. For
more information please visit http://gump.apache.org/nagged.html,
and/or contact the folk at gene...@gump.apache.org.
Project tomcat-tc7.0.x-test-apr has an issue affecting its community
integration.
To whom it may engage...
This is an automated request, but not an unsolicited one. For
more information please visit http://gump.apache.org/nagged.html,
and/or contact the folk at gene...@gump.apache.org.
Project tomcat-trunk-test-apr has an issue affecting its community integration.
Th
https://issues.apache.org/bugzilla/show_bug.cgi?id=57432
--- Comment #3 from Christopher Schultz ---
(In reply to Jeff Pinner from comment #0)
> Bug 53952 added support for TLS 1.1 and TLS 1.2 and added the following
> constants (from jni/java/org/apache/tomcat/jni/SSL.java)
>
> public static fi
https://issues.apache.org/bugzilla/show_bug.cgi?id=57540
--- Comment #8 from Christopher Schultz ---
(In reply to Ralf Hauser from comment #6)
> Created attachment 32437 [details]
> apache.tomcat.tls.protocol.57540.patch
Is IOException the most natural exception type for SSLSupport.getProtocol?
> On Feb 10, 2015, at 8:40 PM, Jeremy Boynes wrote:
>
> The proposed Apache Taglibs 1.2.3 release is now available for voting.
>
> It can be obtained from:
> https://dist.apache.org/repos/dist/dev/tomcat/taglibs/taglibs-standard-1.2.3/
>
> The Maven staging repo is:
> https://repository.apache
https://issues.apache.org/bugzilla/show_bug.cgi?id=57021
--- Comment #7 from VIN ---
I updated the code as per your comments.
Generated java doc for LibraryNotFoundError.java class and added to patch.
Now the patch is of 100KB size.
I attached the updated patch.
Please review it.
--
You are rec
https://issues.apache.org/bugzilla/show_bug.cgi?id=57021
--- Comment #8 from VIN ---
Created attachment 32460
--> https://issues.apache.org/bugzilla/attachment.cgi?id=32460&action=edit
Updated patch with suggested comments
Please let me know if anything is missing.
--
You are receiving this
https://issues.apache.org/bugzilla/show_bug.cgi?id=57573
xinshouke <1599409...@qq.com> changed:
What|Removed |Added
CC||1599409...@qq.com
-
https://issues.apache.org/bugzilla/show_bug.cgi?id=57573
--- Comment #3 from xinshouke <1599409...@qq.com> ---
I had no found there is a config about IP in my server.xml. Just a 'localhost'.
I attached my server.xml in the attachment.
--
You are receiving this mail because:
You are the assignee
Hi Mark,
2015-02-11 21:40 GMT+02:00 Mark Thomas :
>
> We've had a couple of important Coyote fixes since the last release so
> I'm planning on tagging 8.0.x tomorrow.
>
> In my mind this is a slightly delayed beginning of Feb release. I plan
> to get back to a roughly beginning of the month schedu
64 matches
Mail list logo
