[Bug 55976] New: Broken response from NIO connector in Tomcat 7.0.50

2014-01-08 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=55976

Bug ID: 55976
   Summary: Broken response from NIO connector in Tomcat 7.0.50
   Product: Tomcat 7
   Version: trunk
  Hardware: PC
Status: NEW
  Severity: normal
  Priority: P2
 Component: Connectors
  Assignee: dev@tomcat.apache.org
  Reporter: ognjen.d.blagoje...@gmail.com

Tomcat 7.0.50 (proposed for voting at the time of submitting this issue) with
Oracle Java 1.7.0_40 on Win7 64-bit, sometimes returns incorrect response when
NIO connector is used.

Start of the report, and steps to reproduce the problem are in this thread:

  http://www.mail-archive.com/dev@tomcat.apache.org/msg80175.html

-Ognjen

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



Re: [VOTE] Release Apache Tomcat 7.0.50

2014-01-08 Thread Ognjen Blagojevic

Violeta and others,

On 21.12.2013 23:41, Ognjen Blagojevic wrote:

On 21.12.2013 12:30, Violeta Georgieva wrote:

Thanks for the testing.
I'm still not convinced to stop the voting based on the frequency
(1/5000)
of the problem.


Ok.

I'm having hard time creating reproducible test case. As I test more,
the frequency of failures is even lower then I initially reported.
Sometimes it takes 5k request to create a failure, and sometimes even
500k reqests is not enough.


I analyzed the issue during past few days, and I have more info.

But since no one else reported any problem, I conclude that it might be 
something very specific to my configuration. Therefore, I am changing my 
(non-binding) vote to "+1".


I created, however, an issue in Bugzilla [1] to keep the record of the 
problem, and to publish my findings so far.


-Ognjen


[1] https://issues.apache.org/bugzilla/show_bug.cgi?id=55976

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



[Bug 55976] Broken response from NIO connector in Tomcat 7.0.50

2014-01-08 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=55976

Ognjen Blagojevic  changed:

   What|Removed |Added

 OS||All

--- Comment #1 from Ognjen Blagojevic  ---
Java client is used for smoke testing Tomcat 7.0.50. Java client fails with
following exception:


Exception in thread "main" java.lang.RuntimeException: java.io.IOException:
Invalid Http response
at webcrawler.WebCrawler.getpage(WebCrawler.java:117)
at webcrawler.WebCrawler.repeat(WebCrawler.java:191)
at webcrawler.WebCrawler.main(WebCrawler.java:30)
Caused by: java.io.IOException: Invalid Http response
at
sun.net.www.protocol.http.HttpURLConnection.getInputStream(HttpURLConnection.java:1342)
at java.net.URL.openStream(URL.java:1037)
at webcrawler.WebCrawler.getpage(WebCrawler.java:94)
... 2 more



Relevant lines from sun.net.www.protocol.http.HttURLConnection.getInputStream:


respCode = getResponseCode();
if (respCode == -1) {
disconnectInternal();
throw new IOException ("Invalid Http response");
}



So, getResponseCode returns -1. Relevant part from
sun.net.www.protocol.http.HttURLConnection.getResponseCode():


/*
 * Examine the status-line - should be formatted as per
 * section 6.1 of RFC 2616 :-
 *
 * Status-Line = HTTP-Version SP Status-Code SP Reason-Phrase
 *
 * If status line can't be parsed return -1.
 */
if (statusLine.startsWith("HTTP/1.")) {
int codePos = statusLine.indexOf(' ');
if (codePos > 0) {

int phrasePos = statusLine.indexOf(' ', codePos+1);
if (phrasePos > 0 && phrasePos < statusLine.length()) {
responseMessage = statusLine.substring(phrasePos+1);
}

// deviation from RFC 2616 - don't reject status line
// if SP Reason-Phrase is not included.
if (phrasePos < 0)
phrasePos = statusLine.length();

try {
responseCode = Integer.parseInt
(statusLine.substring(codePos+1, phrasePos));
return responseCode;
} catch (NumberFormatException e) { }
}
}
return -1;


So it seems that response from Tomcat doesn't contain string "HTTP/1." in the
first line.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



[Bug 55976] Broken response from NIO connector in Tomcat 7.0.50

2014-01-08 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=55976

--- Comment #3 from Ognjen Blagojevic  ---
Created attachment 31183
  --> https://issues.apache.org/bugzilla/attachment.cgi?id=31183&action=edit
Correct request and broken response

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



[Bug 55976] Broken response from NIO connector in Tomcat 7.0.50

2014-01-08 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=55976

--- Comment #2 from Ognjen Blagojevic  ---
Created attachment 31182
  --> https://issues.apache.org/bugzilla/attachment.cgi?id=31182&action=edit
Correct request and correct response

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



Re: svn commit: r1556071 - /tomcat/trunk/java/org/apache/tomcat/websocket/server/WsRemoteEndpointImplServer.java

2014-01-08 Thread Mark Thomas
On 06/01/2014 23:40, r...@apache.org wrote:
> Author: remm
> Date: Mon Jan  6 23:40:32 2014
> New Revision: 1556071
> 
> URL: http://svn.apache.org/r1556071
> Log:
> Avoid NPE if the write listener is called once immediately after being set, 
> even if no data was written (= the app doesn't send any message during the 
> init of the connection).
> I don't think Tomcat does it yet (this doesn't need to be ported to Tomcat 7 
> since it is not a Servlet 3.1 impl).

Tomcat only does this if the WriteListener is set via the
CoyoteOutputStream. It does not do it if HTTP upgrade is used (which
uses a lighter-weight ServletOutputStream). That looks like a bug that
needs fixing.

Mark


-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



[Bug 55976] Broken response from NIO connector in Tomcat 7.0.50

2014-01-08 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=55976

--- Comment #4 from Ognjen Blagojevic  ---
I took localhost dump using RawCap. The dump contains traffic from the smoke
testing. I then opened it with wireshark. 

In the dump, there is a number of requests for file prelude.jspf.html:

  "GET /examples/jsp/jsp2/misc/prelude.jspf.html HTTP/1.1"

Tomcat generally responds correctly, as seen in attachment 31182.

But, in one request, instead of responding with content from
"prelude.jspf.html" Tomcat responds with partial content from a different file
-- "/docs/architecture/startup/serverStartup.txt".

As one can see in attachment 31183, the response also doesn't contain any
header. That is the reason why Java client fails.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



[Bug 55976] Broken response from NIO connector in Tomcat 7.0.50

2014-01-08 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=55976

--- Comment #5 from Mark Thomas  ---
Can you provide the source code of the test client you are using so the
committers can try and reproduce this?

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



Re: svn commit: r1556071 - /tomcat/trunk/java/org/apache/tomcat/websocket/server/WsRemoteEndpointImplServer.java

2014-01-08 Thread Rémy Maucherat
2014/1/8 Mark Thomas 

> Tomcat only does this if the WriteListener is set via the
> CoyoteOutputStream. It does not do it if HTTP upgrade is used (which
> uses a lighter-weight ServletOutputStream). That looks like a bug that
> needs fixing.
>
> Yes, I saw after my commit there was code to do it for HTTP, but upgrade
uses the code in the AbstractServletOutputStream which only sets the
listener and doesn't do anything with it until there's an actual write.

Rémy


[Bug 55978] New: Initial call to onWritePossible() not made by container when using HTTP upgrade

2014-01-08 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=55978

Bug ID: 55978
   Summary: Initial call to onWritePossible() not made by
container when using HTTP upgrade
   Product: Tomcat 8
   Version: trunk
  Hardware: All
OS: All
Status: NEW
  Severity: normal
  Priority: P2
 Component: Connectors
  Assignee: dev@tomcat.apache.org
  Reporter: ma...@apache.org

When using a WriteListener and non-blocking IO with the Servlet async API, the
container makes the first call to onWritePossible() as per the spec.

When using a WriteListener and non-blocking IO with the Servlet HTTP upgrade
API the container does not make the initial call to onWritePossible(). When
fixing this bug it is worth checking the initial call to onDataAvailable() as
well.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



[Bug 55976] Broken response from NIO connector in Tomcat 7.0.50

2014-01-08 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=55976

--- Comment #6 from Ognjen Blagojevic  ---
Created attachment 31184
  --> https://issues.apache.org/bugzilla/attachment.cgi?id=31184&action=edit
Java test class

Here is Java class used for testing.

You should call it with one argument - starting URL (e.g.
"http://localhost:8080/";)

At the first phase, it crawls all pages starting from the root. During the
crawl, Java stack traces for 401s and 404s will be printed on the console, and
may be ignored.

At the second phase, it will repeatedly read all collected URLs, 500 times in a
row. In total, it reads around 140k pages. 

If there is an error during the test, exception will be thrown, e.g:


Round: 154 / 500
Round: 155 / 500
Round: 156 / 500
Exception in thread "main" java.lang.RuntimeException: java.io.IOException:
Invalid Http response
at webcrawler.WebCrawler.getpage(WebCrawler.java:117)
at webcrawler.WebCrawler.repeat(WebCrawler.java:191)
at webcrawler.WebCrawler.main(WebCrawler.java:30)
Caused by: java.io.IOException: Invalid Http response
at
sun.net.www.protocol.http.HttpURLConnection.getInputStream(HttpURLConnection.java:1342)
at java.net.URL.openStream(URL.java:1037)
at webcrawler.WebCrawler.getpage(WebCrawler.java:94)
... 2 more


Sometimes exception is thrown after 5k requests, sometimes only after running
this test several times.

-Ognjen

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



Re: [VOTE] Release Apache Tomcat 7.0.50

2014-01-08 Thread Violeta Georgieva
Hi Ognjen,


2014/1/8 Ognjen Blagojevic 
>
> Violeta and others,
>
>
> On 21.12.2013 23:41, Ognjen Blagojevic wrote:
>>
>> On 21.12.2013 12:30, Violeta Georgieva wrote:
>>>
>>> Thanks for the testing.
>>> I'm still not convinced to stop the voting based on the frequency
>>> (1/5000)
>>> of the problem.
>>
>>
>> Ok.
>>
>> I'm having hard time creating reproducible test case. As I test more,
>> the frequency of failures is even lower then I initially reported.
>> Sometimes it takes 5k request to create a failure, and sometimes even
>> 500k reqests is not enough.
>
>
> I analyzed the issue during past few days, and I have more info.
>
> But since no one else reported any problem, I conclude that it might be
something very specific to my configuration. Therefore, I am changing my
(non-binding) vote to "+1".
>
> I created, however, an issue in Bugzilla [1] to keep the record of the
problem, and to publish my findings so far.
>
> -Ognjen
>
>
> [1] https://issues.apache.org/bugzilla/show_bug.cgi?id=55976
>


Thanks
Violeta


Re: [VOTE][RESULT] Release Apache Tomcat 7.0.50

2014-01-08 Thread Violeta Georgieva
Hi,


2013/12/20 Violeta Georgieva 
>
> The proposed Apache Tomcat 7.0.50 release is now available for voting.
>
> It can be obtained from:
> https://dist.apache.org/repos/dist/dev/tomcat/tomcat-7/v7.0.50/
> The Maven staging repo is:
> https://repository.apache.org/content/repositories/orgapachetomcat-004/
> The svn tag is:
> http://svn.apache.org/repos/asf/tomcat/tc7.0.x/tags/TOMCAT_7_0_50/
>
> The proposed 7.0.50 release is:
> [ ] Broken - do not release
> [ ] Stable - go ahead and release as 7.0.50 Stable
>
> Regards
> Violeta


The votes cast were

+1 (binding): rjung, markt, olamy, violetagg, jfarcand, kkolinko
+1 (non-binding): Ognjen Blagojevic

No other votes were cast.

With six binding +1 votes and one non-binding +1 vote this vote passes.

I'll upload the release to the mirrors and announce it once the mirrors are
OK.

Regards
Violeta


svn commit: r1556505 - in /tomcat/tc7.0.x/trunk: build.properties.default res/maven/mvn.properties.default

2014-01-08 Thread violetagg
Author: violetagg
Date: Wed Jan  8 11:47:23 2014
New Revision: 1556505

URL: http://svn.apache.org/r1556505
Log:
Prep for next version

Modified:
tomcat/tc7.0.x/trunk/build.properties.default
tomcat/tc7.0.x/trunk/res/maven/mvn.properties.default

Modified: tomcat/tc7.0.x/trunk/build.properties.default
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/build.properties.default?rev=1556505&r1=1556504&r2=1556505&view=diff
==
--- tomcat/tc7.0.x/trunk/build.properties.default (original)
+++ tomcat/tc7.0.x/trunk/build.properties.default Wed Jan  8 11:47:23 2014
@@ -27,7 +27,7 @@
 # - Version Control Flags -
 version.major=7
 version.minor=0
-version.build=50
+version.build=51
 version.patch=0
 version.suffix=-dev
 

Modified: tomcat/tc7.0.x/trunk/res/maven/mvn.properties.default
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/res/maven/mvn.properties.default?rev=1556505&r1=1556504&r2=1556505&view=diff
==
--- tomcat/tc7.0.x/trunk/res/maven/mvn.properties.default (original)
+++ tomcat/tc7.0.x/trunk/res/maven/mvn.properties.default Wed Jan  8 11:47:23 
2014
@@ -35,7 +35,7 @@ maven.asf.release.repo.url=https://repos
 maven.asf.release.repo.repositoryId=apache.releases
 
 # Release version info
-maven.asf.release.deploy.version=7.0.50
+maven.asf.release.deploy.version=7.0.51
 
 #Where do we load the libraries from
 tomcat.lib.path=../../output/build/lib



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r1556526 - in /tomcat/tc6.0.x/trunk: ./ java/org/apache/jasper/compiler/ java/org/apache/jasper/runtime/ test/org/apache/jasper/compiler/ webapps/docs/

2014-01-08 Thread markt
Author: markt
Date: Wed Jan  8 13:22:40 2014
New Revision: 1556526

URL: http://svn.apache.org/r1556526
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=55198
Correctly escape XML output from tagx files

Added:
tomcat/tc6.0.x/trunk/test/org/apache/jasper/compiler/TestELParser.java   
(with props)
Modified:
tomcat/tc6.0.x/trunk/   (props changed)
tomcat/tc6.0.x/trunk/STATUS.txt
tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/ELParser.java
tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/Generator.java
tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/Validator.java
tomcat/tc6.0.x/trunk/java/org/apache/jasper/runtime/PageContextImpl.java
tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml

Propchange: tomcat/tc6.0.x/trunk/
--
  Merged /tomcat/trunk:r1500062,1539157,1539173
  Merged /tomcat/tc7.0.x/trunk:r1500065,1539176-1539177

Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=1556526&r1=1556525&r2=1556526&view=diff
==
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Wed Jan  8 13:22:40 2014
@@ -32,14 +32,6 @@ PATCHES PROPOSED TO BACKPORT:
   [ New proposals should be added at the end of the list ]
 
 
-* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=55198
-  Correctly escape XML output from tagx files.
-  http://svn.apache.org/r1500065 (excluding tests) - original fix
-  http://svn.apache.org/r1539176 (excluding tests) - required for following fix
-  http://svn.apache.org/r1539177 (excluding tests) - fixes regression in 
original
-  +1: markt, schultz, remm, jboynes
-  -1:
-
 * Fix issue with Manager app and other apps that use i18n in the UI when a
   request that specifies an Accept-Language of English ahead of French, Spanish
   or Japanese.

Modified: tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/ELParser.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/ELParser.java?rev=1556526&r1=1556525&r2=1556526&view=diff
==
--- tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/ELParser.java 
(original)
+++ tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/ELParser.java Wed Jan  
8 13:22:40 2014
@@ -17,6 +17,12 @@
 
 package org.apache.jasper.compiler;
 
+import org.apache.jasper.JasperException;
+import org.apache.jasper.compiler.ELNode.ELText;
+import org.apache.jasper.compiler.ELNode.Function;
+import org.apache.jasper.compiler.ELNode.Root;
+import org.apache.jasper.compiler.ELNode.Text;
+
 /**
  * This class implements a parser for EL expressions.
  * 
@@ -106,6 +112,7 @@ public class ELParser {
 // Output whatever is in buffer
 if (buf.length() > 0) {
 ELexpr.add(new ELNode.ELText(buf.toString()));
+buf = new StringBuffer();
 }
 if (!parseFunction()) {
 ELexpr.add(new ELNode.ELText(curToken.toString()));
@@ -130,8 +137,8 @@ public class ELParser {
 }
 String s1 = null; // Function prefix
 String s2 = curToken.toString(); // Function name
-int mark = getIndex();
 if (hasNext()) {
+int mark = getIndex();
 Token t = nextToken();
 if (t.toChar() == ':') {
 if (hasNext()) {
@@ -149,8 +156,8 @@ public class ELParser {
 ELexpr.add(new ELNode.Function(s1, s2));
 return true;
 }
+setIndex(mark);
 }
-setIndex(mark);
 return false;
 }
 
@@ -389,4 +396,42 @@ public class ELParser {
 public char getType() {
 return type;
 }
+
+
+protected static class TextBuilder extends ELNode.Visitor {
+
+protected StringBuilder output = new StringBuilder();
+
+public String getText() {
+return output.toString();
+}
+
+@Override
+public void visit(Root n) throws JasperException {
+output.append(n.getType());
+output.append('{');
+n.getExpression().visit(this);
+output.append('}');
+}
+
+@Override
+public void visit(Function n) throws JasperException {
+if (n.getPrefix() != null) {
+output.append(n.getPrefix());
+output.append(':');
+}
+output.append(n.getName());
+output.append('(');
+}
+
+@Override
+public void visit(Text n) throws JasperException {
+output.append(n.getText());
+}
+
+@Override
+public void visit(ELText n) throws JasperException {
+output.append(n.getText());
+}
+}
 }

Modified: tomcat/tc6.

[Bug 55198] Quote entity in html element attribute evaluated in tagx if attribute contains EL expression

2014-01-08 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=55198

Mark Thomas  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

--- Comment #7 from Mark Thomas  ---
This has been fixed in 6.0.x and will be included in 6.0.38 onwards.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r1556539 - in /tomcat/tc6.0.x/trunk: STATUS.txt java/org/apache/tomcat/util/res/StringManager.java webapps/docs/changelog.xml

2014-01-08 Thread markt
Author: markt
Date: Wed Jan  8 14:10:38 2014
New Revision: 1556539

URL: http://svn.apache.org/r1556539
Log:
Fix issue with Manager app and other apps that use i18n in the UI when a 
request that specifies an Accept-Language of English ahead of French, Spanish 
or Japanese.
Port all the other improvements to the StringManager from trunk as well

Modified:
tomcat/tc6.0.x/trunk/STATUS.txt
tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/res/StringManager.java
tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml

Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=1556539&r1=1556538&r2=1556539&view=diff
==
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Wed Jan  8 14:10:38 2014
@@ -32,15 +32,6 @@ PATCHES PROPOSED TO BACKPORT:
   [ New proposals should be added at the end of the list ]
 
 
-* Fix issue with Manager app and other apps that use i18n in the UI when a
-  request that specifies an Accept-Language of English ahead of French, Spanish
-  or Japanese.
-  Port all the other improvements to the StringManager from trunk as well
-  http://people.apache.org/~markt/patches/2013-12-17-webapp-locale-tc6.patch
-  +1: markt, remm, jboynes
-  +1: kkolinko (a typo in changelog: s/associated/associate/)
-  -1:
-
 * Add support for limiting the size of chunk extensions when using chunked
   encoding.
   
http://people.apache.org/~markt/patches/2013-12-17-chunk-extensions-tc6-v2.patch

Modified: 
tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/res/StringManager.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/res/StringManager.java?rev=1556539&r1=1556538&r2=1556539&view=diff
==
--- tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/res/StringManager.java 
(original)
+++ tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/res/StringManager.java Wed 
Jan  8 14:10:38 2014
@@ -18,8 +18,11 @@
 package org.apache.tomcat.util.res;
 
 import java.text.MessageFormat;
+import java.util.Enumeration;
 import java.util.Hashtable;
+import java.util.LinkedHashMap;
 import java.util.Locale;
+import java.util.Map;
 import java.util.MissingResourceException;
 import java.util.ResourceBundle;
 
@@ -52,12 +55,16 @@ import java.util.ResourceBundle;
 
 public class StringManager {
 
+// Locale.ROOT isn't available until Java 6
+private static final Locale LOCALE_ROOT = new Locale("", "", "");
+
+private static int LOCALE_CACHE_SIZE = 10;
+
 /**
  * The ResourceBundle for this StringManager.
  */
-
-private ResourceBundle bundle;
-private Locale locale;
+private final ResourceBundle bundle;
+private final Locale locale;
 
 /**
  * Creates a new StringManager for a given package. This is a
@@ -67,22 +74,36 @@ public class StringManager {
  *
  * @param packageName Name of package to create StringManager for.
  */
-
-private StringManager(String packageName) {
-   this( packageName, Locale.getDefault() );
-}
-
-private StringManager(String packageName, Locale loc) {
+private StringManager(String packageName, Locale locale) {
 String bundleName = packageName + ".LocalStrings";
-bundle = ResourceBundle.getBundle(bundleName, loc);
+ResourceBundle bnd = null;
+try {
+bnd = ResourceBundle.getBundle(bundleName, locale);
+} catch( MissingResourceException ex ) {
+// Try from the current loader (that's the case for trusted apps)
+// Should only be required if using a TC5 style classloader 
structure
+// where common != shared != server
+ClassLoader cl = Thread.currentThread().getContextClassLoader();
+if( cl != null ) {
+try {
+bnd = ResourceBundle.getBundle(bundleName, locale, cl);
+} catch(MissingResourceException ex2) {
+// Ignore
+}
+}
+}  
+bundle = bnd;
 // Get the actual locale, which may be different from the requested one
-locale = bundle.getLocale();
-}
-
-private StringManager(ResourceBundle bundle )
-{
-   this.bundle=bundle;
-locale = bundle.getLocale();
+if (bundle != null) {
+Locale bundleLocale = bundle.getLocale();
+if (bundleLocale.equals(LOCALE_ROOT)) {
+this.locale = Locale.ENGLISH;
+} else {
+this.locale = bundleLocale;
+}
+} else {
+this.locale = null;
+}
 }
 
 /**
@@ -94,7 +115,6 @@ public class StringManager {
 bundle or null if not found.
 @throws IllegalArgumentException if key is null.
  */
-
 public String getString(String

svn commit: r1556540 - in /tomcat/tc6.0.x/trunk: STATUS.txt java/org/apache/coyote/Constants.java java/org/apache/coyote/http11/filters/ChunkedInputFilter.java webapps/docs/changelog.xml webapps/docs/

2014-01-08 Thread markt
Author: markt
Date: Wed Jan  8 14:15:57 2014
New Revision: 1556540

URL: http://svn.apache.org/r1556540
Log:
Add support for limiting the size of chunk extensions when using chunked 
encoding.

Modified:
tomcat/tc6.0.x/trunk/STATUS.txt
tomcat/tc6.0.x/trunk/java/org/apache/coyote/Constants.java

tomcat/tc6.0.x/trunk/java/org/apache/coyote/http11/filters/ChunkedInputFilter.java
tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml
tomcat/tc6.0.x/trunk/webapps/docs/config/systemprops.xml

Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=1556540&r1=1556539&r2=1556540&view=diff
==
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Wed Jan  8 14:15:57 2014
@@ -32,12 +32,6 @@ PATCHES PROPOSED TO BACKPORT:
   [ New proposals should be added at the end of the list ]
 
 
-* Add support for limiting the size of chunk extensions when using chunked
-  encoding.
-  
http://people.apache.org/~markt/patches/2013-12-17-chunk-extensions-tc6-v2.patch
-  +1: markt, kkolinko, remm, jboynes
-  -1:
-
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=55749
   Improve the error message when SSLEngine is disabled in AprLifecycleListener
   and SSL is configured for an APR/native connector.

Modified: tomcat/tc6.0.x/trunk/java/org/apache/coyote/Constants.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/coyote/Constants.java?rev=1556540&r1=1556539&r2=1556540&view=diff
==
--- tomcat/tc6.0.x/trunk/java/org/apache/coyote/Constants.java (original)
+++ tomcat/tc6.0.x/trunk/java/org/apache/coyote/Constants.java Wed Jan  8 
14:15:57 2014
@@ -76,4 +76,13 @@ public final class Constants {
 Integer.parseInt(System.getProperty(
 "org.apache.coyote.MAX_TRAILER_SIZE",
 "8192"));
+
+/**
+ * Limit on the total length of the extension data in
+ * a chunked HTTP request.
+ */
+public static final int MAX_EXTENSION_SIZE =
+Integer.parseInt(System.getProperty(
+"org.apache.coyote.MAX_EXTENSION_SIZE",
+"8192"));
 }

Modified: 
tomcat/tc6.0.x/trunk/java/org/apache/coyote/http11/filters/ChunkedInputFilter.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/coyote/http11/filters/ChunkedInputFilter.java?rev=1556540&r1=1556539&r2=1556540&view=diff
==
--- 
tomcat/tc6.0.x/trunk/java/org/apache/coyote/http11/filters/ChunkedInputFilter.java
 (original)
+++ 
tomcat/tc6.0.x/trunk/java/org/apache/coyote/http11/filters/ChunkedInputFilter.java
 Wed Jan  8 14:15:57 2014
@@ -114,6 +114,12 @@ public class ChunkedInputFilter implemen
 
 
 /**
+ * Size of extensions processed for this request.
+ */
+private long extensionSize;
+
+
+/**
  * Flag set to true if the next call to doRead() must parse a CRLF pair
  * before doing anything else.
  */
@@ -248,6 +254,10 @@ public class ChunkedInputFilter implemen
 endChunk = false;
 needCRLFParse = false;
 trailingHeaders.recycle();
+if (org.apache.coyote.Constants.MAX_TRAILER_SIZE > 0) {
+
trailingHeaders.setLimit(org.apache.coyote.Constants.MAX_TRAILER_SIZE);
+}
+extensionSize = 0;
 }
 
 
@@ -294,7 +304,7 @@ public class ChunkedInputFilter implemen
 int result = 0;
 boolean eol = false;
 boolean readDigit = false;
-boolean trailer = false;
+boolean extension = false;
 
 while (!eol) {
 
@@ -306,9 +316,13 @@ public class ChunkedInputFilter implemen
 if (buf[pos] == Constants.CR || buf[pos] == Constants.LF) {
 parseCRLF(false);
 eol = true;
-} else if (buf[pos] == Constants.SEMI_COLON) {
-trailer = true;
-} else if (!trailer) { 
+} else if (buf[pos] == Constants.SEMI_COLON && !extension) {
+// First semi-colon marks the start of the extension. Further
+// semi-colons may appear to separate multiple 
chunk-extensions.
+// These need to be processed as part of parsing the 
extensions.
+extension = true;
+extensionSize++;
+} else if (!extension) { 
 //don't read data after the trailer
 int charValue = HexUtils.getDec(buf[pos]);
 if (charValue != -1) {
@@ -320,6 +334,15 @@ public class ChunkedInputFilter implemen
 //in the chunked header
 return false;
 }
+} else {
+// Extension 'parsing'
+// Note that the chunk-extension is neither parsed nor
+// val

svn commit: r1556541 - in /tomcat/tc6.0.x/trunk: STATUS.txt java/org/apache/tomcat/util/net/AprEndpoint.java java/org/apache/tomcat/util/net/res/LocalStrings.properties webapps/docs/changelog.xml

2014-01-08 Thread markt
Author: markt
Date: Wed Jan  8 14:18:39 2014
New Revision: 1556541

URL: http://svn.apache.org/r1556541
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=55749
Improve the error message when SSLEngine is disabled in AprLifecycleListener 
and SSL is configured for an APR/native connector.

Modified:
tomcat/tc6.0.x/trunk/STATUS.txt
tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java

tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/net/res/LocalStrings.properties
tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml

Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=1556541&r1=1556540&r2=1556541&view=diff
==
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Wed Jan  8 14:18:39 2014
@@ -32,13 +32,6 @@ PATCHES PROPOSED TO BACKPORT:
   [ New proposals should be added at the end of the list ]
 
 
-* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=55749
-  Improve the error message when SSLEngine is disabled in AprLifecycleListener
-  and SSL is configured for an APR/native connector.
-  http://people.apache.org/~markt/patches/2013-11-12-bug55749-tc6.patch
-  +1: markt, kkolinko, remm
-  -1:
-
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=55759
   Update the Eclipse compiler to 4.3.1
   http://svn.apache.org/viewvc?view=revision&revision=1539953

Modified: tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java?rev=1556541&r1=1556540&r2=1556541&view=diff
==
--- tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java 
(original)
+++ tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java Wed 
Jan  8 14:18:39 2014
@@ -633,6 +633,12 @@ public class AprEndpoint extends Abstrac
 if (initialized)
 return;
 
+if (rootPool != 0) {
+// A previous init() call failed
+throw new IllegalStateException(
+sm.getString("endpoint.apr.previousInitFailed"));
+}
+
 // Create the root APR memory pool
 rootPool = Pool.create(0);
 // Create the pool for the server socket
@@ -776,7 +782,15 @@ public class AprEndpoint extends Abstrac
 }
 
 // Create SSL Context
-sslContext = SSLContext.make(rootPool, value, SSL.SSL_MODE_SERVER);
+try {
+sslContext = SSLContext.make(rootPool, value, 
SSL.SSL_MODE_SERVER);
+} catch (Exception e) {
+// If the sslEngine is disabled on the AprLifecycleListener
+// there will be an Exception here but there is no way to check
+// the AprLifecycleListener settings from here
+throw new Exception(
+sm.getString("endpoint.apr.failSslContextMake"), e);
+}
 
 // Set cipher order: client (default) or server
 if (SSLHonorCipherOrder) {

Modified: 
tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/net/res/LocalStrings.properties
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/net/res/LocalStrings.properties?rev=1556541&r1=1556540&r2=1556541&view=diff
==
--- 
tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/net/res/LocalStrings.properties
 (original)
+++ 
tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/net/res/LocalStrings.properties
 Wed Jan  8 14:18:39 2014
@@ -48,4 +48,6 @@ endpoint.warn.noHonorCipherOrder='Honor 
 endpoint.debug.channelCloseFail=Failed to close channel
 endpoint.debug.socketCloseFail=Failed to close socket
 endpoint.apr.noSslCertFile=Connector attribute SSLCertificateFile must be 
defined when using SSL with APR
+endpoint.apr.failSslContextMake=Unable to create SSLContext. Check that 
SSLEngine is enabled in the AprLifecycleListener, the AprLifecycleListener has 
initialised correctly and that a valid SSLProtocol has been specified
 endpoint.apr.invalidSslProtocol=An invalid value [{0}] was provided for the 
SSLProtocol attribute
+endpoint.apr.previousInitFailed=Unable to initialise endpoint. A previous call 
to init() failed. See previous log messages for details.

Modified: tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml?rev=1556541&r1=1556540&r2=1556541&view=diff
==
--- tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml Wed Jan  8 14:18:39 2014
@@ -112,6 +112,11 @@
 Add support for limiting the size of chunk extensi

[Bug 55749] Disabling SSLEngine in AprLifecycleListener leads to misleading error message at startup when SSL Connector is delcared

2014-01-08 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=55749

Mark Thomas  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

--- Comment #2 from Mark Thomas  ---
Fixed in 6.0.x and will be included in 6.0.38 onwards.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r1556557 - in /tomcat/tc6.0.x/trunk: ./ java/javax/servlet/resources/ res/ res/META-INF/ res/maven/ webapps/docs/

2014-01-08 Thread markt
Author: markt
Date: Wed Jan  8 15:06:22 2014
New Revision: 1556557

URL: http://svn.apache.org/r1556557
Log:
Add missing J2EE 5 resources

Added:
tomcat/tc6.0.x/trunk/java/javax/servlet/resources/javaee_5.xsd
  - copied unchanged from r1553650, 
tomcat/tc7.0.x/trunk/java/javax/servlet/resources/javaee_5.xsd

tomcat/tc6.0.x/trunk/java/javax/servlet/resources/javaee_web_services_1_2.xsd
  - copied unchanged from r1553650, 
tomcat/tc7.0.x/trunk/java/javax/servlet/resources/javaee_web_services_1_2.xsd

tomcat/tc6.0.x/trunk/java/javax/servlet/resources/javaee_web_services_client_1_2.xsd
  - copied unchanged from r1553650, 
tomcat/tc7.0.x/trunk/java/javax/servlet/resources/javaee_web_services_client_1_2.xsd
tomcat/tc6.0.x/trunk/res/META-INF/servlet-api.jar.license
  - copied, changed from r1553650, 
tomcat/tc7.0.x/trunk/res/META-INF/servlet-api.jar.license
tomcat/tc6.0.x/trunk/res/META-INF/servlet-api.jar.notice
  - copied, changed from r1553650, 
tomcat/tc7.0.x/trunk/res/META-INF/servlet-api.jar.notice
Modified:
tomcat/tc6.0.x/trunk/LICENSE
tomcat/tc6.0.x/trunk/NOTICE
tomcat/tc6.0.x/trunk/STATUS.txt
tomcat/tc6.0.x/trunk/build.xml
tomcat/tc6.0.x/trunk/res/INSTALLLICENSE
tomcat/tc6.0.x/trunk/res/maven/servlet-api.pom
tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml

Modified: tomcat/tc6.0.x/trunk/LICENSE
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/LICENSE?rev=1556557&r1=1556556&r2=1556557&view=diff
==
--- tomcat/tc6.0.x/trunk/LICENSE (original)
+++ tomcat/tc6.0.x/trunk/LICENSE Wed Jan  8 15:06:22 2014
@@ -704,4 +704,339 @@ by name) to the files from the LZMA comp
 subjecting your linked code to the terms of the Common Public license version
 1.0. Any modifications or additions to files from the LZMA compression module
 for NSIS, however, are subject to the terms of the Common Public License 
version
-1.0. 
\ No newline at end of file
+1.0. 
+
+
+For the following XML Schemas for Java EE Deployment Descriptors:
+ - javaee_5.xsd
+ - javaee_web_services_1_2.xsd
+ - javaee_web_services_client_1_2.xsd
+
+COMMON DEVELOPMENT AND DISTRIBUTION LICENSE (CDDL) Version 1.0
+
+1. Definitions.
+
+   1.1. Contributor. means each individual or entity that creates or 
contributes
+to the creation of Modifications.
+
+   1.2. Contributor Version. means the combination of the Original Software,
+prior Modifications used by a Contributor (if any), and the
+Modifications made by that particular Contributor.
+
+   1.3. Covered Software. means (a) the Original Software, or (b) 
Modifications,
+or (c) the combination of files containing Original Software with files
+containing Modifications, in each case including portions thereof.
+
+   1.4. Executable. means the Covered Software in any form other than Source
+Code.
+
+   1.5. Initial Developer. means the individual or entity that first makes
+Original Software available under this License.
+
+   1.6. Larger Work. means a work which combines Covered Software or portions
+thereof with code not governed by the terms of this License.
+
+   1.7. License. means this document.
+
+   1.8. Licensable. means having the right to grant, to the maximum extent
+possible, whether at the time of the initial grant or subsequently
+acquired, any and all of the rights conveyed herein.
+
+   1.9. Modifications. means the Source Code and Executable form of any of the
+following:
+
+A. Any file that results from an addition to, deletion from or
+   modification of the contents of a file containing Original Software
+   or previous Modifications;
+
+B. Any new file that contains any part of the Original Software or
+   previous Modification; or
+
+C. Any new file that is contributed or otherwise made available under
+   the terms of this License.
+
+   1.10. Original Software. means the Source Code and Executable form of
+ computer software code that is originally released under this License.
+
+   1.11. Patent Claims. means any patent claim(s), now owned or hereafter
+ acquired, including without limitation, method, process, and apparatus
+ claims, in any patent Licensable by grantor.
+
+   1.12. Source Code. means (a) the common form of computer software code in
+ which modifications are made and (b) associated documentation included
+ in or with such code.
+
+   1.13. You. (or .Your.) means an individual or a legal entity exercising
+ rights under, and complying with all of the terms of, this License. 
For
+ legal entities, .You. includes any entity which controls, is 
controlled
+ by, or is under common control with You. For purposes of this
+ definition, .control. means (a) the power, direct or indirect, to 
cause
+ the direction 

svn commit: r1556558 - in /tomcat/tc6.0.x/trunk: LICENSE res/INSTALLLICENSE

2014-01-08 Thread markt
Author: markt
Date: Wed Jan  8 15:14:01 2014
New Revision: 1556558

URL: http://svn.apache.org/r1556558
Log:
Remove trailing whitespace

Modified:
tomcat/tc6.0.x/trunk/LICENSE
tomcat/tc6.0.x/trunk/res/INSTALLLICENSE

Modified: tomcat/tc6.0.x/trunk/LICENSE
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/LICENSE?rev=1556558&r1=1556557&r2=1556558&view=diff
==
--- tomcat/tc6.0.x/trunk/LICENSE (original)
+++ tomcat/tc6.0.x/trunk/LICENSE Wed Jan  8 15:14:01 2014
@@ -704,7 +704,7 @@ by name) to the files from the LZMA comp
 subjecting your linked code to the terms of the Common Public license version
 1.0. Any modifications or additions to files from the LZMA compression module
 for NSIS, however, are subject to the terms of the Common Public License 
version
-1.0. 
+1.0.
 
 
 For the following XML Schemas for Java EE Deployment Descriptors:

Modified: tomcat/tc6.0.x/trunk/res/INSTALLLICENSE
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/res/INSTALLLICENSE?rev=1556558&r1=1556557&r2=1556558&view=diff
==
--- tomcat/tc6.0.x/trunk/res/INSTALLLICENSE (original)
+++ tomcat/tc6.0.x/trunk/res/INSTALLLICENSE Wed Jan  8 15:14:01 2014
@@ -704,7 +704,7 @@ by name) to the files from the LZMA comp
 subjecting your linked code to the terms of the Common Public license version
 1.0. Any modifications or additions to files from the LZMA compression module
 for NSIS, however, are subject to the terms of the Common Public License 
version
-1.0. 
+1.0.
 
 
 For the following XML Schemas for Java EE Deployment Descriptors:



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



[Bug 55976] Broken response from NIO connector in Tomcat 7.0.50

2014-01-08 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=55976

--- Comment #7 from Konstantin Preißer  ---
Created attachment 31185
  --> https://issues.apache.org/bugzilla/attachment.cgi?id=31185&action=edit
TCP data log

(In reply to Ognjen Blagojevic from comment #6)
> Created attachment 31184 [details]
> Java test class

Hi,

I can reproduce the problem with your test class with Tomcat 7.0.50, Tomcat
7.0.47 and Tomcat 8.0.0-RC10 on Windows 8.1 x64 with Java 1.7.0_45.


To provide an additional example of a broken HTTP response, I used a
self-written tool to intercept the TCP connections from the Java client to
Tomcat and log all data of each TCP connection channel to a file and I attached
a Zip file containing the logs of the last few TCP connections.

The logs of connections 262-267 look normal (each HTTP request is replied to
with an HTTP response), but if you look at 268 (where a single HTTP request was
sent), Tomcat did not start the reply with an HTTP response, but with some
random data chunks that seem to belong to different responses, and later sent
the actual HTTP response for the request.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



[Bug 55976] Broken response from NIO connector in Tomcat 7.0.50

2014-01-08 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=55976

--- Comment #8 from Jess Holle  ---
(In reply to Konstantin Preißer from comment #7)
> I can reproduce the problem with your test class with Tomcat 7.0.50, Tomcat
> 7.0.47 and Tomcat 8.0.0-RC10 on Windows 8.1 x64 with Java 1.7.0_45.

So that's critical information -- this isn't a regression since 7.0.47, i.e. in
the 7.0.50 release.  It might be a regression in 7.0.47, but that was already
released as stable.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



[jira] [Commented] (MTOMCAT-238) Parameter staticContextDocbase cant be used

2014-01-08 Thread Maciej Nyczka (JIRA)

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

Maciej Nyczka commented on MTOMCAT-238:
---

The same thing here. Affects also 2.2 version

> Parameter staticContextDocbase cant be used
> ---
>
> Key: MTOMCAT-238
> URL: https://issues.apache.org/jira/browse/MTOMCAT-238
> Project: Apache Tomcat Maven Plugin
>  Issue Type: Bug
>  Components: tomcat6, tomcat7
>Affects Versions: 2.0-beta-1, 2.1
>Reporter: Michal Franc
>Assignee: Olivier Lamy (*$^¨%`£)
> Fix For: moreinfo
>
>
> Attribute staticContextDocbase leads to IllegalArgumentException
> org.apache.tomcat.maven.plugin.tomcat7.run.AbstractRunMojo#createStaticContext
> is context created from container and at the and added again to server which 
> leads to java.lang.IllegalArgumentException: addChild:  Child name '/' is not 
> unique



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r1556585 - in /tomcat/tc6.0.x/trunk: ./ java/org/apache/catalina/ java/org/apache/catalina/core/ java/org/apache/catalina/startup/ java/org/apache/jasper/ java/org/apache/jasper/compiler/

2014-01-08 Thread markt
Author: markt
Date: Wed Jan  8 17:49:03 2014
New Revision: 1556585

URL: http://svn.apache.org/r1556585
Log:
Back-port some XML processing improvements (part 1)
The patch could have been smaller but I opted to align some of the code with 
Tomcat 7 (which meant re-ordering methods in places) to make it easier to 
compare 6.0.x code with 7.0.x code.
This covers:
 - always process TLDS with a namespace aware parser
 - enable validation with STRICT_SERVLET_COMPLIANCE (Servlet 2.5, SRV.14.4.3)

Modified:
tomcat/tc6.0.x/trunk/STATUS.txt
tomcat/tc6.0.x/trunk/java/org/apache/catalina/Context.java
tomcat/tc6.0.x/trunk/java/org/apache/catalina/Globals.java
tomcat/tc6.0.x/trunk/java/org/apache/catalina/core/ApplicationContext.java
tomcat/tc6.0.x/trunk/java/org/apache/catalina/core/StandardContext.java
tomcat/tc6.0.x/trunk/java/org/apache/catalina/core/mbeans-descriptors.xml
tomcat/tc6.0.x/trunk/java/org/apache/catalina/startup/TldConfig.java
tomcat/tc6.0.x/trunk/java/org/apache/jasper/Constants.java
tomcat/tc6.0.x/trunk/java/org/apache/jasper/EmbeddedServletOptions.java
tomcat/tc6.0.x/trunk/java/org/apache/jasper/JspC.java

tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/ImplicitTagLibraryInfo.java
tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/JspConfig.java
tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/JspDocumentParser.java
tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/TagLibraryInfoImpl.java
tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/TagPluginManager.java
tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/TldLocationsCache.java
tomcat/tc6.0.x/trunk/java/org/apache/jasper/servlet/JspCServletContext.java
tomcat/tc6.0.x/trunk/java/org/apache/jasper/xmlparser/ParserUtils.java
tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/digester/Digester.java
tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml
tomcat/tc6.0.x/trunk/webapps/docs/config/context.xml
tomcat/tc6.0.x/trunk/webapps/docs/config/systemprops.xml

Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=1556585&r1=1556584&r2=1556585&view=diff
==
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Wed Jan  8 17:49:03 2014
@@ -40,32 +40,6 @@ PATCHES PROPOSED TO BACKPORT:
exclude lists in TldConfig.java, TldLocationsCache.java, see r1476932.
   -1:
 
-* Back-port some XML processing improvements (part 1)
-  The patch could have been smaller but I opted to align some of the code with
-  Tomcat 7 (which meant re-ordering methods in places) to make it easier to
-  compare 6.0.x code with 7.0.x code.
-  
http://people.apache.org/~markt/patches/2013-12-20-xml-prep-part1-tc6-v2.patch
-  +1: markt, remm, jboynes
-  +1: kkolinko:
-  I do not like that you change the values of validation flags when
-  STRICT_SERVLET_COMPLIANCE is set. This was a feature required for
-  Servlet 3.0, but I do not remember the same requirement for Servlet
-  2.5.
-  I expect that with this change the bug #351040 in Eclipse IDE will
-  become visible with Tomcat 6, while it is currently observed with
-  Tomcat 7 only. It is not our policy to provide workarounds for 3rd
-  party bugs, so this is not a veto.
-https://bugs.eclipse.org/bugs/show_bug.cgi?id=351040
-  The rest of the patch is OK.
-  markt:
-  Servlet 2.5, SRV.14.4.3 requires validation for a J2EE compliant
-  implementation. The Eclipse bug is annoying but easy to workaround and
-  only present when using STRICT_SERVLET_COMPLIANCE.
-  kkolinko:
-  Thank you for the chapter number. I am changing my vote to +1. It'd
-  be better to mention the chapter number in the changelog.
-  -1:
-
 * Back-port some XML processing improvements (part 2)
   JspC passes configuration with ServletContext init params so a real
   implementation is required. This is essentially a back-port of

Modified: tomcat/tc6.0.x/trunk/java/org/apache/catalina/Context.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/catalina/Context.java?rev=1556585&r1=1556584&r2=1556585&view=diff
==
--- tomcat/tc6.0.x/trunk/java/org/apache/catalina/Context.java (original)
+++ tomcat/tc6.0.x/trunk/java/org/apache/catalina/Context.java Wed Jan  8 
17:49:03 2014
@@ -542,6 +542,82 @@ public interface Context extends Contain
 public void setWrapperClass(String wrapperClass);
 
 
+/**
+ * Will the parsing of the web.xml file for this Context be performed by a
+ * namespace aware parser?
+ *
+ * @return true if namespace awareness is enabled.
+ */
+public boolean getXmlNamespaceAware();
+
+
+/**
+ * Controls whether the parsing of the web.xml file for this Context will 
be
+ * perfor

svn commit: r1556589 - in /tomcat/tc6.0.x/trunk: STATUS.txt java/org/apache/jasper/servlet/JspCServletContext.java

2014-01-08 Thread markt
Author: markt
Date: Wed Jan  8 17:51:48 2014
New Revision: 1556589

URL: http://svn.apache.org/r1556589
Log:
Back-port some XML processing improvements (part 2)
JspC passes configuration with ServletContext init params so a real 
implementation is required. This is essentially a back-port of 
http://svn.apache.org/r1552306

Modified:
tomcat/tc6.0.x/trunk/STATUS.txt
tomcat/tc6.0.x/trunk/java/org/apache/jasper/servlet/JspCServletContext.java

Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=1556589&r1=1556588&r2=1556589&view=diff
==
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Wed Jan  8 17:51:48 2014
@@ -40,14 +40,6 @@ PATCHES PROPOSED TO BACKPORT:
exclude lists in TldConfig.java, TldLocationsCache.java, see r1476932.
   -1:
 
-* Back-port some XML processing improvements (part 2)
-  JspC passes configuration with ServletContext init params so a real
-  implementation is required. This is essentially a back-port of
-  http://svn.apache.org/r1552306
-  
http://people.apache.org/~markt/patches/2013-12-19-xml-prep-part2-tc6-v1.patch
-  +1: markt, kkolinko, remm, jboynes
-  -1:
-
 * Back-port some XML processing improvements (part 3)
   Add the new descriptor package
   Note I am generating the patches using git so the patch shows whole new files

Modified: 
tomcat/tc6.0.x/trunk/java/org/apache/jasper/servlet/JspCServletContext.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/jasper/servlet/JspCServletContext.java?rev=1556589&r1=1556588&r2=1556589&view=diff
==
--- tomcat/tc6.0.x/trunk/java/org/apache/jasper/servlet/JspCServletContext.java 
(original)
+++ tomcat/tc6.0.x/trunk/java/org/apache/jasper/servlet/JspCServletContext.java 
Wed Jan  8 17:51:48 2014
@@ -5,9 +5,9 @@
  * The ASF licenses this file to You under the Apache License, Version 2.0
  * (the "License"); you may not use this file except in compliance with
  * the License.  You may obtain a copy of the License at
- * 
+ *
  *  http://www.apache.org/licenses/LICENSE-2.0
- * 
+ *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
@@ -28,6 +28,7 @@ import java.util.HashSet;
 import java.util.Hashtable;
 import java.util.Set;
 import java.util.Vector;
+import java.util.concurrent.ConcurrentHashMap;
 
 import javax.servlet.RequestDispatcher;
 import javax.servlet.Servlet;
@@ -55,6 +56,12 @@ public class JspCServletContext implemen
 
 
 /**
+ * Servlet context initialization parameters.
+ */
+private final ConcurrentHashMap myParameters;
+
+
+/**
  * The log writer we will write log messages to.
  */
 protected PrintWriter myLogWriter;
@@ -78,6 +85,7 @@ public class JspCServletContext implemen
 public JspCServletContext(PrintWriter aLogWriter, URL aResourceBaseURL) {
 
 myAttributes = new Hashtable();
+myParameters = new ConcurrentHashMap();
 myLogWriter = aLogWriter;
 myResourceBaseURL = aResourceBaseURL;
 
@@ -137,9 +145,7 @@ public class JspCServletContext implemen
  * @param name Name of the requested parameter
  */
 public String getInitParameter(String name) {
-
-return (null);
-
+return myParameters.get(name);
 }
 
 
@@ -148,9 +154,7 @@ public class JspCServletContext implemen
  * parameters.
  */
 public Enumeration getInitParameterNames() {
-
-return (new Vector().elements());
-
+return myParameters.keys();
 }
 
 
@@ -218,8 +222,8 @@ public class JspCServletContext implemen
 }
 
 }
-
-
+
+
 /**
  * Return a request dispatcher for the specified context-relative path.
  *
@@ -438,6 +442,6 @@ public class JspCServletContext implemen
 
 
 public boolean setInitParameter(String name, String value) {
-return false;
+return myParameters.putIfAbsent(name, value) == null;
 }
 }



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r1556591 - in /tomcat/tc6.0.x/trunk: ./ STATUS.txt java/org/apache/coyote/Response.java webapps/docs/changelog.xml

2014-01-08 Thread markt
Author: markt
Date: Wed Jan  8 18:01:14 2014
New Revision: 1556591

URL: http://svn.apache.org/r1556591
Log:
Avoid possible NPE when a content type with no charset is specified
(Followup to r1548971)

Modified:
tomcat/tc6.0.x/trunk/   (props changed)
tomcat/tc6.0.x/trunk/STATUS.txt
tomcat/tc6.0.x/trunk/java/org/apache/coyote/Response.java
tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml

Propchange: tomcat/tc6.0.x/trunk/
--
  Merged /tomcat/trunk:r1552804
  Merged /tomcat/tc7.0.x/trunk:r1552805

Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=1556591&r1=1556590&r2=1556591&view=diff
==
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Wed Jan  8 18:01:14 2014
@@ -72,12 +72,6 @@ PATCHES PROPOSED TO BACKPORT:
   requires Ant >= 1.8.0).
   -1:
 
-* Avoid possible NPE when a content type with no charset is specified
-  (Followup to r1548971)
-  http://svn.apache.org/r1552805
-  +1: markt, kkolinko, remm, jboynes
-  -1:
-
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=55973
   Fix processing of XML schemas when validation is enabled in Jasper
   http://svn.apache.org/r1556377

Modified: tomcat/tc6.0.x/trunk/java/org/apache/coyote/Response.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/coyote/Response.java?rev=1556591&r1=1556590&r2=1556591&view=diff
==
--- tomcat/tc6.0.x/trunk/java/org/apache/coyote/Response.java (original)
+++ tomcat/tc6.0.x/trunk/java/org/apache/coyote/Response.java Wed Jan  8 
18:01:14 2014
@@ -478,11 +478,14 @@ public final class Response {
 
 this.contentType = m.toStringNoCharset();
 
-String charsetValue = m.getCharset().trim();
+String charsetValue = m.getCharset();
 
-if (charsetValue != null && charsetValue.length() > 0) {
-charsetSet = true;
-this.characterEncoding = charsetValue;
+if (charsetValue != null) {
+charsetValue = charsetValue.trim();
+if (charsetValue.length() > 0) {
+charsetSet = true;
+this.characterEncoding = charsetValue;
+}
 }
 }
 

Modified: tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml?rev=1556591&r1=1556590&r2=1556591&view=diff
==
--- tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml Wed Jan  8 18:01:14 2014
@@ -132,6 +132,10 @@
 in the AprLifecycleListener and SSL is configured for an APR/native
 connector. (markt)
   
+  
+Avoid possible NPE if a content type is specified without a character 
set.
+(markt)
+  
 
   
   



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r1556600 - in /tomcat/tc6.0.x/trunk: ./ java/org/apache/tomcat/util/descriptor/

2014-01-08 Thread markt
Author: markt
Date: Wed Jan  8 18:27:53 2014
New Revision: 1556600

URL: http://svn.apache.org/r1556600
Log:
Back-port some XML processing improvements (part 3)
Add the new descriptor package

Added:
tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/descriptor/
  - copied from r1553650, 
tomcat/tc7.0.x/trunk/java/org/apache/tomcat/util/descriptor/
Modified:
tomcat/tc6.0.x/trunk/STATUS.txt

tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/descriptor/DigesterFactory.java

tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/descriptor/LocalResolver.java

tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/descriptor/XmlErrorHandler.java

tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/descriptor/XmlIdentifiers.java

Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=1556600&r1=1556599&r2=1556600&view=diff
==
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Wed Jan  8 18:27:53 2014
@@ -40,19 +40,6 @@ PATCHES PROPOSED TO BACKPORT:
exclude lists in TldConfig.java, TldLocationsCache.java, see r1476932.
   -1:
 
-* Back-port some XML processing improvements (part 3)
-  Add the new descriptor package
-  Note I am generating the patches using git so the patch shows whole new files
-  being added. When actually making the commit, they will be copied from 7.0.x 
-  and amended to retain history.
-  
http://people.apache.org/~markt/patches/2013-12-19-xml-prep-part3-tc6-v1.patch
-  +1: markt, remm
-  +1: kkolinko (Looks OK. I have not tested into what jar the new package
-  goes and whether it requires an update to build.xml).
-  +1: jboynes (Looks OK, have not verified if schema references work when the
-  XSDs are still split between servlet/resources and jsp/resources).
-  -1:
-
 * Back-port some XML processing improvements (part 4)
   Add a unit test for the new descriptor
   Note I am generating the patches using git so the patch shows whole new files

Modified: 
tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/descriptor/DigesterFactory.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/descriptor/DigesterFactory.java?rev=1556600&r1=1553650&r2=1556600&view=diff
==
--- 
tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/descriptor/DigesterFactory.java
 (original)
+++ 
tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/descriptor/DigesterFactory.java
 Wed Jan  8 18:27:53 2014
@@ -80,15 +80,6 @@ public class DigesterFactory {
 addSelf(systemIds, "javaee_web_services_1_2.xsd");
 addSelf(systemIds, "javaee_web_services_client_1_2.xsd");
 
-// from JavaEE 6
-systemIds.put(XmlIdentifiers.WEB_30_XSD, idFor("web-app_3_0.xsd"));
-systemIds.put(XmlIdentifiers.WEB_FRAGMENT_30_XSD, 
idFor("web-fragment_3_0.xsd"));
-addSelf(systemIds, "web-common_3_0.xsd");
-addSelf(systemIds, "javaee_6.xsd");
-addSelf(systemIds, "jsp_2_2.xsd");
-addSelf(systemIds, "javaee_web_services_1_3.xsd");
-addSelf(systemIds, "javaee_web_services_client_1_3.xsd");
-
 SERVLET_API_PUBLIC_IDS = Collections.unmodifiableMap(publicIds);
 SERVLET_API_SYSTEM_IDS = Collections.unmodifiableMap(systemIds);
 }

Modified: 
tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/descriptor/LocalResolver.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/descriptor/LocalResolver.java?rev=1556600&r1=1553650&r2=1556600&view=diff
==
--- 
tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/descriptor/LocalResolver.java 
(original)
+++ 
tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/descriptor/LocalResolver.java 
Wed Jan  8 18:27:53 2014
@@ -61,14 +61,12 @@ public class LocalResolver implements En
 }
 
 
-@Override
 public InputSource resolveEntity(String publicId, String systemId)
 throws SAXException, IOException {
 return resolveEntity(null, publicId, null, systemId);
 }
 
 
-@Override
 public InputSource resolveEntity(String name, String publicId,
 String base, String systemId) throws SAXException, IOException {
 
@@ -138,7 +136,6 @@ public class LocalResolver implements En
 }
 
 
-@Override
 public InputSource getExternalSubset(String name, String baseURI)
 throws SAXException, IOException {
 return null;

Modified: 
tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/descriptor/XmlErrorHandler.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/descriptor/XmlErrorHandler.java?rev=1556600&r1=1553650&r2=1556600&view=diff
==
--- 
tomcat/tc6.0.x/trunk/java/org/a

svn commit: r1556601 - /tomcat/tc6.0.x/trunk/STATUS.txt

2014-01-08 Thread markt
Author: markt
Date: Wed Jan  8 18:35:35 2014
New Revision: 1556601

URL: http://svn.apache.org/r1556601
Log:
Vote

Modified:
tomcat/tc6.0.x/trunk/STATUS.txt

Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=1556601&r1=1556600&r2=1556601&view=diff
==
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Wed Jan  8 18:35:35 2014
@@ -62,7 +62,7 @@ PATCHES PROPOSED TO BACKPORT:
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=55973
   Fix processing of XML schemas when validation is enabled in Jasper
   http://svn.apache.org/r1556377
-  +1: kkolinko, jboynes
+  +1: kkolinko, jboynes, markt
   -1:
 
 



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r4065 - /dev/tomcat/tomcat-7/v7.0.50/ /release/tomcat/tomcat-7/v7.0.50/

2014-01-08 Thread violetagg
Author: violetagg
Date: Wed Jan  8 19:10:38 2014
New Revision: 4065

Log:
Release 7.0.50

Added:
release/tomcat/tomcat-7/v7.0.50/
  - copied from r4062, dev/tomcat/tomcat-7/v7.0.50/
Removed:
dev/tomcat/tomcat-7/v7.0.50/


-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r4065 - /dev/tomcat/tomcat-7/v7.0.50/ /release/tomcat/tomcat-7/v7.0.50/

2014-01-08 Thread violetagg
Author: violetagg
Date: Wed Jan  8 19:10:38 2014
New Revision: 4065

Log:
Release 7.0.50

Added:
release/tomcat/tomcat-7/v7.0.50/
  - copied from r4062, dev/tomcat/tomcat-7/v7.0.50/
Removed:
dev/tomcat/tomcat-7/v7.0.50/


-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r1556613 - /tomcat/tc6.0.x/trunk/STATUS.txt

2014-01-08 Thread markt
Author: markt
Date: Wed Jan  8 19:45:12 2014
New Revision: 1556613

URL: http://svn.apache.org/r1556613
Log:
Propose next block of XML patches

Modified:
tomcat/tc6.0.x/trunk/STATUS.txt

Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=1556613&r1=1556612&r2=1556613&view=diff
==
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Wed Jan  8 19:45:12 2014
@@ -59,6 +59,47 @@ PATCHES PROPOSED TO BACKPORT:
   requires Ant >= 1.8.0).
   -1:
 
+* Back-port from XML processing improvements (part 5)
+  Switch to a per context configuration for the XML digester
+  
http://people.apache.org/~markt/patches/2014-01-08-xml-prep-part5-tc6-v1.patch
+  +1: markt
+  -1:
+
+* Back-port from XML processing improvements (part 6)
+  Upgrade digester to use DefaultHandler2 and use LexicalHandler to detect
+  publicId.
+  Simplify web application version detection in web.xml
+  Extracted from a patch by Jeremy Boynes.
+  Essentially, this is a back-port of http://svn.apache.org/r1547931
+  Note that the patch contains a number of strictly unnecessary cosmetic 
changes
+  that have been included to make it simpler to compare the resulting Tomcat 6
+  code with the current Tomcat 7 code to validate the correctness of the
+  back-port
+  
http://people.apache.org/~markt/patches/2014-01-08-xml-prep-part6-tc6-v1.patch
+  +1: markt
+  -1:
+
+* Back-port from XML processing improvements (part 7)
+  Switch o.a.catalina classes to use the new ErrorHandler
+  Back-port of http://svn.apache.org/r1547937
+  
http://people.apache.org/~markt/patches/2014-01-08-xml-prep-part7-tc6-v1.patch
+  +1: markt
+  -1:
+
+* Back-port from XML processing improvements (part 8)
+  Switch ParserUtils to use the new ErrorHandler
+  Back-port of http://svn.apache.org/r1547947
+  
http://people.apache.org/~markt/patches/2014-01-08-xml-prep-part8-tc6-v1.patch
+  +1: markt
+  -1:
+
+* Back-port from XML processing improvements (part 9)
+  Switch JspDocumentParser to an DefaultHandler2
+  Back-port of http://svn.apache.org/r1547948
+  
http://people.apache.org/~markt/patches/2014-01-08-xml-prep-part9-tc6-v1.patch
+  +1: markt
+  -1:
+
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=55973
   Fix processing of XML schemas when validation is enabled in Jasper
   http://svn.apache.org/r1556377



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r1556614 - /tomcat/tc6.0.x/trunk/STATUS.txt

2014-01-08 Thread markt
Author: markt
Date: Wed Jan  8 19:46:09 2014
New Revision: 1556614

URL: http://svn.apache.org/r1556614
Log:
Add a note

Modified:
tomcat/tc6.0.x/trunk/STATUS.txt

Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=1556614&r1=1556613&r2=1556614&view=diff
==
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Wed Jan  8 19:46:09 2014
@@ -45,6 +45,8 @@ PATCHES PROPOSED TO BACKPORT:
   Note I am generating the patches using git so the patch shows whole new files
   being added. When actually making the commit, they will be copied from 7.0.x 
   and amended to retain history.
+  Note: Part 5 onwards do not depend on this patch so this could be ignored /
+addressed later without impacting the XML changes.
   
http://people.apache.org/~markt/patches/2013-12-19-xml-prep-part4-tc6-v1.patch
   +1: markt, remm
   +0: kkolinko:



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



[Tomcat Wiki] Update of "Cookies" by markt

2014-01-08 Thread Apache Wiki
Dear Wiki user,

You have subscribed to a wiki page or wiki category on "Tomcat Wiki" for change 
notification.

The "Cookies" page has been changed by markt:
https://wiki.apache.org/tomcat/Cookies?action=diff&rev1=22&rev2=23

Comment:
Review proposed generation changes and make a couple of alternative suggestions

  === Changes to generation of Set-Cookie header ===
   G1 Use RFC6265 format header for V0 cookies::
   :: When version == 0 always generate a RFC6265 header, raising an exception 
from addCookie if the value is invalid rather than attempting to upgrade to a 
RFC2109 header to use quoting. Application impact is that they will now fail 
fast with an error rather than inconsistent data as described in Bug 55920; 
applications that do not set invalid values will not be impacted.
+  :: '''Alternative G1a:''' Generate an RFC6265 header if possible but provide 
an option (disabled by default) to allow switching to an RFC2109 header if a 
valid RFC6265 header is not possible.
  
   G2 Use RFC2109 format header only for V1 cookies::
   :: When version == 1 always generate a RFC2109 header, raising an exception 
from addCookie if the value is invalid. This preserves existing behaviour for 
applications that use V1 cookies.
  
   G3 Stop adding quotes or escaping to values::
   :: The value supplied by the application will be validated to the relevant 
specification and will result in a IAE if it does not conform. The value will 
never be modified to add quotations or escape characters, Application impact is 
that an attempt to set an invalid value will result in an early error rather 
than inconsistent data.
+  :: '''Alternative G3a:''' Quotes and/or escaping only to be added to RFC2109 
headers. API to remain symmetric and quoting/escaping to remain transparent to 
applicatons.
  
   G4 Use UTF-8 encoding for values::
   :: The value (which is a UCS-16 Java String) will be encoded using UTF-8 
when being added to the header. Application impact is that non-ASCII characters 
will no longer cause an IAE. For V0 cookies, this is an extension to RFC6265 
required to support HTML-5. V1 cookies already allow 8-bit characters if quoted 
and this is likely to be needed to avoid an IAE as the value would still be 
validated; it would be the application's responsibility to quote the value.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



[Tomcat Wiki] Update of "Cookies" by markt

2014-01-08 Thread Apache Wiki
Dear Wiki user,

You have subscribed to a wiki page or wiki category on "Tomcat Wiki" for change 
notification.

The "Cookies" page has been changed by markt:
https://wiki.apache.org/tomcat/Cookies?action=diff&rev1=23&rev2=24

Comment:
Review proposed parsing changes and make an alternative suggestion

  
   P3 Do not throw IAE from the parser::
   :: Invalid syntax will result in a user-data log entry and cookies being 
dropped rather than throwing of an IAE. Application impact is that requests 
with an invalid Cookie header will now be dispatched to the application.
+  :: '''Alternative P3a:''' Ensure that the cookie header is available for the 
application to parse manually.
  
  === Impact of proposal on existing issues ===
  ||<:10%>'''Issue'''||Impact||

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r1556634 - /tomcat/tc6.0.x/trunk/STATUS.txt

2014-01-08 Thread markt
Author: markt
Date: Wed Jan  8 20:51:04 2014
New Revision: 1556634

URL: http://svn.apache.org/r1556634
Log:
Proposal

Modified:
tomcat/tc6.0.x/trunk/STATUS.txt

Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=1556634&r1=1556633&r2=1556634&view=diff
==
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Wed Jan  8 20:51:04 2014
@@ -108,6 +108,12 @@ PATCHES PROPOSED TO BACKPORT:
   +1: kkolinko, jboynes, markt
   -1:
 
+* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=55119
+  Avoid CVE-2013-1571 when generating Javadoc
+  http://svn.apache.org/r1495875
+  +1: markt
+  -1:
+
 
 PATCHES/ISSUES THAT ARE STALLED
 



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



[Bug 55956] Regarding Tomcat Manager Application Status

2014-01-08 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=55956

Mark Thomas  changed:

   What|Removed |Added

 Status|NEEDINFO|RESOLVED
 Resolution|--- |INVALID

--- Comment #2 from Mark Thomas  ---
This is a configuration issue. As Chris pointed out, you need to configure
either
http://tomcat.apache.org/tomcat-7.0-doc/config/valve.html#Remote_IP_Valve or
http://tomcat.apache.org/tomcat-7.0-doc/config/filter.html#Remote_IP_Filter

If you need help doing that, the users list is the place to ask.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r1556635 - /tomcat/tc6.0.x/trunk/STATUS.txt

2014-01-08 Thread markt
Author: markt
Date: Wed Jan  8 20:59:38 2014
New Revision: 1556635

URL: http://svn.apache.org/r1556635
Log:
Proposal

Modified:
tomcat/tc6.0.x/trunk/STATUS.txt

Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=1556635&r1=1556634&r2=1556635&view=diff
==
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Wed Jan  8 20:59:38 2014
@@ -114,6 +114,12 @@ PATCHES PROPOSED TO BACKPORT:
   +1: markt
   -1:
 
+* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=55671
+  Consistent naming for genStringAsCharArray
+  http://svn.apache.org/r889966
+  +1: markt
+  -1:
+
 
 PATCHES/ISSUES THAT ARE STALLED
 



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r1556653 - /tomcat/maven-plugin/trunk/pom.xml

2014-01-08 Thread olamy
Author: olamy
Date: Wed Jan  8 22:03:39 2014
New Revision: 1556653

URL: http://svn.apache.org/r1556653
Log:
tomcat 7.0.50

Modified:
tomcat/maven-plugin/trunk/pom.xml

Modified: tomcat/maven-plugin/trunk/pom.xml
URL: 
http://svn.apache.org/viewvc/tomcat/maven-plugin/trunk/pom.xml?rev=1556653&r1=1556652&r2=1556653&view=diff
==
--- tomcat/maven-plugin/trunk/pom.xml (original)
+++ tomcat/maven-plugin/trunk/pom.xml Wed Jan  8 22:03:39 2014
@@ -73,7 +73,7 @@
 2001
 
 2008
-7.0.47
+7.0.50
 
 
 



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r1556655 - /tomcat/maven-plugin/trunk/tomcat7-maven-plugin/src/site/apt/adjust-embedded-tomcat-version.apt.vm

2014-01-08 Thread olamy
Author: olamy
Date: Wed Jan  8 22:05:47 2014
New Revision: 1556655

URL: http://svn.apache.org/r1556655
Log:
doc for last version

Modified:

tomcat/maven-plugin/trunk/tomcat7-maven-plugin/src/site/apt/adjust-embedded-tomcat-version.apt.vm

Modified: 
tomcat/maven-plugin/trunk/tomcat7-maven-plugin/src/site/apt/adjust-embedded-tomcat-version.apt.vm
URL: 
http://svn.apache.org/viewvc/tomcat/maven-plugin/trunk/tomcat7-maven-plugin/src/site/apt/adjust-embedded-tomcat-version.apt.vm?rev=1556655&r1=1556654&r2=1556655&view=diff
==
--- 
tomcat/maven-plugin/trunk/tomcat7-maven-plugin/src/site/apt/adjust-embedded-tomcat-version.apt.vm
 (original)
+++ 
tomcat/maven-plugin/trunk/tomcat7-maven-plugin/src/site/apt/adjust-embedded-tomcat-version.apt.vm
 Wed Jan  8 22:05:47 2014
@@ -35,7 +35,7 @@ Adjust Tomcat Version
 
[...]
   
-   7.0.27
+   7.0.50
[...]
   
[...]



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r1556725 - in /tomcat/trunk/java/org/apache/jasper/resources: LocalStrings.properties LocalStrings_es.properties LocalStrings_ja.properties

2014-01-08 Thread kkolinko
Author: kkolinko
Date: Thu Jan  9 05:38:12 2014
New Revision: 1556725

URL: http://svn.apache.org/r1556725
Log:
Followup to r889966:
The genStrAsCharArray init param in Jasper has been renamed to 
genStringAsCharArray in Tomcat 7.

Modified:
tomcat/trunk/java/org/apache/jasper/resources/LocalStrings.properties
tomcat/trunk/java/org/apache/jasper/resources/LocalStrings_es.properties
tomcat/trunk/java/org/apache/jasper/resources/LocalStrings_ja.properties

Modified: tomcat/trunk/java/org/apache/jasper/resources/LocalStrings.properties
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/jasper/resources/LocalStrings.properties?rev=1556725&r1=1556724&r2=1556725&view=diff
==
--- tomcat/trunk/java/org/apache/jasper/resources/LocalStrings.properties 
(original)
+++ tomcat/trunk/java/org/apache/jasper/resources/LocalStrings.properties Thu 
Jan  9 05:38:12 2014
@@ -119,7 +119,7 @@ jsp.warning.recompileOnFail=Warning: Inv
 jsp.warning.development=Warning: Invalid value for the initParam development. 
Will use the default value of \"true\"
 jsp.warning.fork=Warning: Invalid value for the initParam fork. Will use the 
default value of \"true\"
 jsp.warning.dumpSmap=Warning: Invalid value for the initParam dumpSmap. Will 
use the default value of \"false\"
-jsp.warning.genchararray=Warning: Invalid value for the initParam 
genStrAsCharArray. Will use the default value of \"false\"
+jsp.warning.genchararray=Warning: Invalid value for the initParam 
genStringAsCharArray. Will use the default value of \"false\"
 jsp.warning.suppressSmap=Warning: Invalid value for the initParam 
suppressSmap. Will use the default value of \"false\"
 jsp.warning.displaySourceFragment=Warning: Invalid value for the initParam 
displaySourceFragment. Will use the default value of \"true\"
 jsp.warning.maxLoadedJsps=Warning: Invalid value for the initParam 
maxLoadedJsps. Will use the default value of \"-1\"

Modified: 
tomcat/trunk/java/org/apache/jasper/resources/LocalStrings_es.properties
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/jasper/resources/LocalStrings_es.properties?rev=1556725&r1=1556724&r2=1556725&view=diff
==
--- tomcat/trunk/java/org/apache/jasper/resources/LocalStrings_es.properties 
(original)
+++ tomcat/trunk/java/org/apache/jasper/resources/LocalStrings_es.properties 
Thu Jan  9 05:38:12 2014
@@ -117,7 +117,7 @@ jsp.warning.recompileOnFail = Aviso\: Va
 jsp.warning.development = Aviso\: valor incorrecto para el initParam 
development. Se usar\u00E1 el valor por defecto de "true"
 jsp.warning.fork = Aviso\: valor incorrecto para el initParam fork. Se 
usar\u00E1 el valor por defecto de "true"
 jsp.warning.dumpSmap = Aviso\: valor incorrecto para el initParam dumpSmap. Se 
usar\u00E1 el valor por defecto de "false"
-jsp.warning.genchararray = Aviso\: valor incorrecto para el initParam 
genStrAsCharArray. Se usar\u00E1 el valor por defecto de "false"
+jsp.warning.genchararray = Aviso\: valor incorrecto para el initParam 
genStringAsCharArray. Se usar\u00E1 el valor por defecto de "false"
 jsp.warning.suppressSmap = Aviso\: valor incorrecto para el initParam 
suppressSmap. Se usar\u00E1 el valor por defecto de "false"
 jsp.warning.displaySourceFragment = Aviso\: valor incorrecto para el initParam 
displaySourceFragment. Se usar\u00E1 el valor por defecto de "verdadero"
 jsp.warning.maxLoadedJsps = Aviso\: Valor inv\u00E1lido para el initParam 
maxLoadedJsps. Usar\u00E9 el valor por defecto de "-1"

Modified: 
tomcat/trunk/java/org/apache/jasper/resources/LocalStrings_ja.properties
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/jasper/resources/LocalStrings_ja.properties?rev=1556725&r1=1556724&r2=1556725&view=diff
==
--- tomcat/trunk/java/org/apache/jasper/resources/LocalStrings_ja.properties 
(original)
+++ tomcat/trunk/java/org/apache/jasper/resources/LocalStrings_ja.properties 
Thu Jan  9 05:38:12 2014
@@ -104,7 +104,7 @@ jsp.warning.checkInterval=\u8b66\u544a: 
 jsp.warning.development=\u8b66\u544a: initParam 
development\u306e\u5024\u304c\u7121\u52b9\u3067\u3059\u3002\"true\"\u306e\u30c7\u30d5\u30a9\u30eb\u30c8\u5024\u3092\u4f7f\u7528\u3057\u307e\u3059
 jsp.warning.fork=\u8b66\u544a: initParam 
fork\u306e\u5024\u304c\u7121\u52b9\u3067\u3059\u3002\"true\"\u306e\u30c7\u30d5\u30a9\u30eb\u30c8\u5024\u3092\u4f7f\u7528\u3057\u307e\u3059
 jsp.warning.dumpSmap=\u8b66\u544a: initParam 
dumpSmap\u306e\u5024\u304c\u7121\u52b9\u3067\u3059\u3002\"false\"\u306e\u30c7\u30d5\u30a9\u30eb\u30c8\u5024\u3092\u4f7f\u7528\u3057\u307e\u3059
-jsp.warning.genchararray=\u8b66\u544a: initParam 
genStrAsCharArray\u306e\u5024\u304c\u7121\u52b9\u3067\u3059\u3002\"false\"\u306e\u30c7\u30d5\u30a9\u30eb\u30c8\u5024\u3092\u4f7f\u7528\u3057\u307e\u3059
+jsp.warning.genchararr

[Bug 55671] Backport genStringAsCharArray patch to Tomcat 6.x branch.

2014-01-08 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=55671

--- Comment #1 from Konstantin Kolinko  ---
Steps to reproduce = ?
Where and how do you configure the JSP compiler? What does not work?


The correct init-parameter name in Tomcat 6 is "genStrAsCharArray"
and that is documented in
http://tomcat.apache.org/tomcat-6.0-doc/jasper-howto.html

(Though the second mention of it in "Production Configuration" section says
"genStringAsCharArray", a typo.)

If you use it with Ant then yes, there is an inconsistency and you would have
to use  as the setter in JspC.java is
setGenStringAsCharArray(...) with no matching getter.


I think backporting r889966 as is is wrong. I think EmbeddedServletOptions
should ask both the old and the new init-param name, for compatibility.
It also needs r1556725 and update to jasper-howto.html.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r1556730 - /tomcat/tc6.0.x/trunk/STATUS.txt

2014-01-08 Thread kkolinko
Author: kkolinko
Date: Thu Jan  9 06:26:16 2014
New Revision: 1556730

URL: http://svn.apache.org/r1556730
Log:
votes

Modified:
tomcat/tc6.0.x/trunk/STATUS.txt

Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=1556730&r1=1556729&r2=1556730&view=diff
==
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Thu Jan  9 06:26:16 2014
@@ -64,8 +64,11 @@ PATCHES PROPOSED TO BACKPORT:
 * Back-port from XML processing improvements (part 5)
   Switch to a per context configuration for the XML digester
   
http://people.apache.org/~markt/patches/2014-01-08-xml-prep-part5-tc6-v1.patch
-  +1: markt
+  +1: markt, kkolinko
   -1:
+   kkolinko: I see no need for the new non-static "createWebXmlDigester(..)"
+ method to be public. It returns 'void', so it is unlikely for it to be
+ called from outside.
 
 * Back-port from XML processing improvements (part 6)
   Upgrade digester to use DefaultHandler2 and use LexicalHandler to detect
@@ -78,14 +81,16 @@ PATCHES PROPOSED TO BACKPORT:
   code with the current Tomcat 7 code to validate the correctness of the
   back-port
   
http://people.apache.org/~markt/patches/2014-01-08-xml-prep-part6-tc6-v1.patch
-  +1: markt
+  +1: markt, kkolinko
   -1:
+   kkolinko: It would be easier to review without all those trailing
+ whitespace changes.
 
 * Back-port from XML processing improvements (part 7)
   Switch o.a.catalina classes to use the new ErrorHandler
   Back-port of http://svn.apache.org/r1547937
   
http://people.apache.org/~markt/patches/2014-01-08-xml-prep-part7-tc6-v1.patch
-  +1: markt
+  +1: markt, kkolinko
   -1:
 
 * Back-port from XML processing improvements (part 8)
@@ -93,13 +98,15 @@ PATCHES PROPOSED TO BACKPORT:
   Back-port of http://svn.apache.org/r1547947
   
http://people.apache.org/~markt/patches/2014-01-08-xml-prep-part8-tc6-v1.patch
   +1: markt
+  +1: kkolinko (would be better with a fix to BZ 55974, so that the first
+   error is reported like the old code does, not a random one)
   -1:
 
 * Back-port from XML processing improvements (part 9)
   Switch JspDocumentParser to an DefaultHandler2
   Back-port of http://svn.apache.org/r1547948
   
http://people.apache.org/~markt/patches/2014-01-08-xml-prep-part9-tc6-v1.patch
-  +1: markt
+  +1: markt, kkolinko
   -1:
 
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=55973
@@ -112,12 +119,19 @@ PATCHES PROPOSED TO BACKPORT:
   Avoid CVE-2013-1571 when generating Javadoc
   http://svn.apache.org/r1495875
   +1: markt
+  +1: kkolinko (The fix goes into dist.xml)
   -1:
 
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=55671
   Consistent naming for genStringAsCharArray
   http://svn.apache.org/r889966
   +1: markt
+  -0: kkolinko: I think "genStrAsCharArray" is the proper and
+documented name for Tomcat 6. If we change the init-param name
+for better compatibility with Tomcat 7 (a good thing, +1),
+the code should check both the new and the old param names.
+It also needs r1556725.
+https://issues.apache.org/bugzilla/show_bug.cgi?id=55671#c1
   -1:
 
 



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r1556737 - in /tomcat/site/trunk: docs/migration-7.html xdocs/migration-7.xml

2014-01-08 Thread kkolinko
Author: kkolinko
Date: Thu Jan  9 06:47:06 2014
New Revision: 1556737

URL: http://svn.apache.org/r1556737
Log:
Document genStrAsCharArray -> genStringAsCharArray rename in Jasper r889966

Modified:
tomcat/site/trunk/docs/migration-7.html
tomcat/site/trunk/xdocs/migration-7.xml

Modified: tomcat/site/trunk/docs/migration-7.html
URL: 
http://svn.apache.org/viewvc/tomcat/site/trunk/docs/migration-7.html?rev=1556737&r1=1556736&r2=1556737&view=diff
==
--- tomcat/site/trunk/docs/migration-7.html (original)
+++ tomcat/site/trunk/docs/migration-7.html Thu Jan  9 06:47:06 2014
@@ -256,6 +256,9 @@
 
 Internal APIs
 
+
+JSP compiler
+
 
 
 
@@ -928,6 +931,23 @@ of Apache Tomcat.
 
 
 
+  
+
+JSP compiler
+
+
+  
+Initialization parameter of JspServlet that controls
+  one of performance optimizations has been renamed from
+  genStrAsCharArray to genStringAsCharArray
+  and is now consistent with the name of related attribute in Jasper task
+  for Apache Ant.
+  
+
+  
+
+
+
 
 
 Upgrading 7.0.x

Modified: tomcat/site/trunk/xdocs/migration-7.xml
URL: 
http://svn.apache.org/viewvc/tomcat/site/trunk/xdocs/migration-7.xml?rev=1556737&r1=1556736&r2=1556737&view=diff
==
--- tomcat/site/trunk/xdocs/migration-7.xml (original)
+++ tomcat/site/trunk/xdocs/migration-7.xml Thu Jan  9 06:47:06 2014
@@ -449,6 +449,17 @@ of Apache Tomcat.
 
   
 
+  
+
+  Initialization parameter of JspServlet that controls
+  one of performance optimizations has been renamed from
+  genStrAsCharArray to genStringAsCharArray
+  and is now consistent with the name of related attribute in Jasper task
+  for Apache Ant.
+  
+
+  
+
 
 
 



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r1556741 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/jasper/resources/LocalStrings.properties java/org/apache/jasper/resources/LocalStrings_es.properties java/org/apache/jasper/resource

2014-01-08 Thread kkolinko
Author: kkolinko
Date: Thu Jan  9 06:57:28 2014
New Revision: 1556741

URL: http://svn.apache.org/r1556741
Log:
Merged r1556725 from tomcat/trunk:
https://issues.apache.org/bugzilla/show_bug.cgi?id=55671
Followup to r889966:
Fix a typo in log messages: genStrAsCharArray init param in Jasper has been 
renamed to genStringAsCharArray in Tomcat 7.

Modified:
tomcat/tc7.0.x/trunk/   (props changed)

tomcat/tc7.0.x/trunk/java/org/apache/jasper/resources/LocalStrings.properties

tomcat/tc7.0.x/trunk/java/org/apache/jasper/resources/LocalStrings_es.properties

tomcat/tc7.0.x/trunk/java/org/apache/jasper/resources/LocalStrings_ja.properties
tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml

Propchange: tomcat/tc7.0.x/trunk/
--
  Merged /tomcat/trunk:r1556725

Modified: 
tomcat/tc7.0.x/trunk/java/org/apache/jasper/resources/LocalStrings.properties
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/jasper/resources/LocalStrings.properties?rev=1556741&r1=1556740&r2=1556741&view=diff
==
--- 
tomcat/tc7.0.x/trunk/java/org/apache/jasper/resources/LocalStrings.properties 
(original)
+++ 
tomcat/tc7.0.x/trunk/java/org/apache/jasper/resources/LocalStrings.properties 
Thu Jan  9 06:57:28 2014
@@ -173,7 +173,7 @@ jsp.warning.development=Warning: Invalid
 jsp.warning.fork=Warning: Invalid value for the initParam fork. Will use the 
default value of \"true\"
 jsp.warning.reloading=Warning: Invalid value for the initParam reloading. Will 
use the default value of \"true\"
 jsp.warning.dumpSmap=Warning: Invalid value for the initParam dumpSmap. Will 
use the default value of \"false\"
-jsp.warning.genchararray=Warning: Invalid value for the initParam 
genStrAsCharArray. Will use the default value of \"false\"
+jsp.warning.genchararray=Warning: Invalid value for the initParam 
genStringAsCharArray. Will use the default value of \"false\"
 jsp.warning.suppressSmap=Warning: Invalid value for the initParam 
suppressSmap. Will use the default value of \"false\"
 jsp.warning.displaySourceFragment=Warning: Invalid value for the initParam 
displaySourceFragment. Will use the default value of \"true\"
 jsp.warning.maxLoadedJsps=Warning: Invalid value for the initParam 
maxLoadedJsps. Will use the default value of \"-1\"

Modified: 
tomcat/tc7.0.x/trunk/java/org/apache/jasper/resources/LocalStrings_es.properties
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/jasper/resources/LocalStrings_es.properties?rev=1556741&r1=1556740&r2=1556741&view=diff
==
--- 
tomcat/tc7.0.x/trunk/java/org/apache/jasper/resources/LocalStrings_es.properties
 (original)
+++ 
tomcat/tc7.0.x/trunk/java/org/apache/jasper/resources/LocalStrings_es.properties
 Thu Jan  9 06:57:28 2014
@@ -169,7 +169,7 @@ jsp.warning.development = Aviso\: valor 
 jsp.warning.fork = Aviso\: valor incorrecto para el initParam fork. Se 
usar\u00E1 el valor por defecto de "true"
 jsp.warning.reloading = Aviso\: valor incorrecto para el initParam reloading. 
Se usar\u00E1 el valor por defecto de "true"
 jsp.warning.dumpSmap = Aviso\: valor incorrecto para el initParam dumpSmap. Se 
usar\u00E1 el valor por defecto de "false"
-jsp.warning.genchararray = Aviso\: valor incorrecto para el initParam 
genStrAsCharArray. Se usar\u00E1 el valor por defecto de "false"
+jsp.warning.genchararray = Aviso\: valor incorrecto para el initParam 
genStringAsCharArray. Se usar\u00E1 el valor por defecto de "false"
 jsp.warning.suppressSmap = Aviso\: valor incorrecto para el initParam 
suppressSmap. Se usar\u00E1 el valor por defecto de "false"
 jsp.warning.displaySourceFragment = Aviso\: valor incorrecto para el initParam 
displaySourceFragment. Se usar\u00E1 el valor por defecto de "verdadero"
 jsp.warning.maxLoadedJsps = Aviso\: Valor inv\u00E1lido para el initParam 
maxLoadedJsps. Usar\u00E9 el valor por defecto de "-1"

Modified: 
tomcat/tc7.0.x/trunk/java/org/apache/jasper/resources/LocalStrings_ja.properties
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/jasper/resources/LocalStrings_ja.properties?rev=1556741&r1=1556740&r2=1556741&view=diff
==
--- 
tomcat/tc7.0.x/trunk/java/org/apache/jasper/resources/LocalStrings_ja.properties
 (original)
+++ 
tomcat/tc7.0.x/trunk/java/org/apache/jasper/resources/LocalStrings_ja.properties
 Thu Jan  9 06:57:28 2014
@@ -155,7 +155,7 @@ jsp.warning.development=\u8b66\u544a: in
 jsp.warning.fork=\u8b66\u544a: initParam 
fork\u306e\u5024\u304c\u7121\u52b9\u3067\u3059\u3002\"true\"\u306e\u30c7\u30d5\u30a9\u30eb\u30c8\u5024\u3092\u4f7f\u7528\u3057\u307e\u3059
 jsp.warning.reloading=\u8b66\u544a: initParam 
reloading\u306e\u5024\u304c\u7121\u52b9\u3067\u3059\u3002\"true\"\u306e\u30c7\u30d5\u3