svn commit: r1410715 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/jasper/tagplugins/jstl/core/Out.java

2012-11-17 Thread markt
Author: markt
Date: Sat Nov 17 10:10:16 2012
New Revision: 1410715

URL: http://svn.apache.org/viewvc?rev=1410715&view=rev
Log:
Follow up to r1410636
Fix test failure

Modified:
tomcat/tc7.0.x/trunk/   (props changed)
tomcat/tc7.0.x/trunk/java/org/apache/jasper/tagplugins/jstl/core/Out.java

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

Modified: 
tomcat/tc7.0.x/trunk/java/org/apache/jasper/tagplugins/jstl/core/Out.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/jasper/tagplugins/jstl/core/Out.java?rev=1410715&r1=1410714&r2=1410715&view=diff
==
--- tomcat/tc7.0.x/trunk/java/org/apache/jasper/tagplugins/jstl/core/Out.java 
(original)
+++ tomcat/tc7.0.x/trunk/java/org/apache/jasper/tagplugins/jstl/core/Out.java 
Sat Nov 17 10:10:16 2012
@@ -48,6 +48,7 @@ public final class Out implements TagPlu
 String strSkipBodyName = ctxt.getTemporaryVariableName();
 
 //according to the tag file, the value attribute is mandatory.
+ctxt.generateImport("java.io.Reader");
 ctxt.generateJavaSource("Object " + strObjectName + "=");
 ctxt.generateAttribute("value");
 ctxt.generateJavaSource(";");



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



svn commit: r1410717 - /tomcat/tc7.0.x/trunk/build.properties.default

2012-11-17 Thread markt
Author: markt
Date: Sat Nov 17 10:13:58 2012
New Revision: 1410717

URL: http://svn.apache.org/viewvc?rev=1410717&view=rev
Log:
Restore -dev in version for trunk build

Modified:
tomcat/tc7.0.x/trunk/build.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=1410717&r1=1410716&r2=1410717&view=diff
==
--- tomcat/tc7.0.x/trunk/build.properties.default (original)
+++ tomcat/tc7.0.x/trunk/build.properties.default Sat Nov 17 10:13:58 2012
@@ -29,7 +29,7 @@ version.major=7
 version.minor=0
 version.build=33
 version.patch=0
-version.suffix=
+version.suffix=-dev
 
 # - Build control flags -
 # Note enabling validation uses Checkstyle which is LGPL licensed



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



svn commit: r1410718 - in /tomcat/tc7.0.x/tags/TOMCAT_7_0_33: ./ build.properties.default

2012-11-17 Thread markt
Author: markt
Date: Sat Nov 17 10:16:22 2012
New Revision: 1410718

URL: http://svn.apache.org/viewvc?rev=1410718&view=rev
Log:
Tag 7.0.33

Added:
tomcat/tc7.0.x/tags/TOMCAT_7_0_33/   (props changed)
  - copied from r1410717, tomcat/tc7.0.x/trunk/
Modified:
tomcat/tc7.0.x/tags/TOMCAT_7_0_33/build.properties.default

Propchange: tomcat/tc7.0.x/tags/TOMCAT_7_0_33/
--
bugtraq:append = false

Propchange: tomcat/tc7.0.x/tags/TOMCAT_7_0_33/
--
bugtraq:label = Bugzilla ID (optional)

Propchange: tomcat/tc7.0.x/tags/TOMCAT_7_0_33/
--
--- bugtraq:message (added)
+++ bugtraq:message Sat Nov 17 10:16:22 2012
@@ -0,0 +1 @@
+Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=%BUGID%

Propchange: tomcat/tc7.0.x/tags/TOMCAT_7_0_33/
--
bugtraq:number = true

Propchange: tomcat/tc7.0.x/tags/TOMCAT_7_0_33/
--
bugtraq:url = https://issues.apache.org/bugzilla/show_bug.cgi?id=%BUGID%

Propchange: tomcat/tc7.0.x/tags/TOMCAT_7_0_33/
--
bugtraq:warnifnoissue = false

Propchange: tomcat/tc7.0.x/tags/TOMCAT_7_0_33/
--
--- svn:ignore (added)
+++ svn:ignore Sat Nov 17 10:16:22 2012
@@ -0,0 +1,7 @@
+.*
+build.properties
+logs
+nbproject
+output
+work
+*.iml

Propchange: tomcat/tc7.0.x/tags/TOMCAT_7_0_33/
--
--- svn:mergeinfo (added)
+++ svn:mergeinfo Sat Nov 17 10:16:22 2012
@@ -0,0 +1 @@
+/tomcat/trunk:1156115-1157160,1157162-1157859,1157862-1157942,1157945-1160347,1160349-1163716,1163718-1166689,1166691-1174340,1174342-1175596,1175598-1175611,1175613-1175932,1175934-1177783,1177785-1177980,1178006-1180720,1180722-1183094,1183096-1187753,1187755,1187775,1187801,1187806,1187809,1187826-1188312,1188314-1188401,1188646-1188840,1188842-1190176,1190178-1195223,1195225-1195953,1195955,1195957-1201238,1201240-1203345,1203347-1206623,1206625-1208046,1208073,1208096,1208114,1208145,1208772,1209194-1212125,1212127-1220291,1220293,1220295-1221321,1221323-1222328,1222332-1222401,1222405-1222795,1222850-1222950,1222969-1225326,1225328-1225463,1225465,1225627,1225629-1226534,1226536-1228908,1228911-1228923,1228927-1229532,1229534-1230766,1230768-1231625,1231627-1233414,1233419-1235207,1235209-1237425,1237427,1237429-1237977,1237981,1237985,1237995,1238070,1238073,1239024-1239048,1239050-1239062,1239135,1239256,1239258-1239485,1239785-1240046,1240101,1240106,1240109,1240112
 
,1240114,1240116,1240118,1240121,1240329,1240474-1240850,1240857,1241087,1241160,1241408-1241822,1241908-1241909,1241912-1242110,1242371-1292130,1292134-1292458,1292464-1292670,1292672-1292776,1292780-1293392,1293397-1297017,1297019-1297963,1297965-1299820,1300108,1300111-1300460,1300520-1300948,1300997,1301006,1301280,1302332,1302348,1302608-1302610,1302649,1302837,1303138,1303163,1303338,1303521,1303587,1303698,1303803,1303852,1304011,1304035,1304037,1304135,1304249,1304253,1304260,1304271,1304275,1304468,1304895,1304930-1304932,1305194,1305943,1305965,1306556,1306579-1306580,1307084,1307310,1307511-1307512,1307579,1307591,1307597,1310636,1310639-1310640,1310642,1310701,1311212,1311995,1327617,1327670,1331766,1333161,1333173,1333827,1334787,1335026,1335257,1335547,1335692,1335711,1335731,1336515,1336813,1336864,1336868,1336884,1337419,1337426,1337546,1337572,1337591-1337595,1337643,1337707,1337719,1337734,1337741,1337745,1338151-1338154,1338178,1342027,1342029,1342315,1342
 
320,1342476,1342498,1342503,1342717,1342795,1342805,1343044-1343046,1343335,1343394,1343400,1343629,1343708,1343718,1343895,1344063,1344068,1344250,1344266,1344515,1344528,1344612,1344629,1344725,1344868,1344890,1344893,1344896,1344901,1345020,1345029,1345039,1345287-1345290,1345294,1345309,1345325,1345357,1345367,1345579-1345580,1345582,1345688,1345699,1345704,1345731-1345732,1345737,1345744,1345752,1345754,1345779,1345781,1345846,1346107,1346376,1346404,1346510,1346514,1346519,1346581,1346635,1346644,1346683,1346794,1346885,1346932,1347034,1347047,1347087,1347108-1347109,1347583,1347737,1348105,1348357,1348398,1348425,1348461-1348495,1348498,1348752,1348762,1348772,1348776,1348859,1348968,1348973,1348989,1349007,1349237,1349298,1349317,1349410,1349473,1349539,1349879,1349887,1349893,1349922,1349984,1350124,1350241,1350243,1350294-1350295,1350299,1350864,1350900,1351010,1351054,1351056,1351068,1351134-1351135,1351148,1351259,1351604,1351636-1351640,1351991,1351993,1352011,1
 
352056,1352059,1352661,1352663,1352788,1352799,1353087,1353125,1353240,1353261,1353414,1353468,13

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

2012-11-17 Thread markt
Author: markt
Date: Sat Nov 17 10:18:33 2012
New Revision: 1410719

URL: http://svn.apache.org/viewvc?rev=1410719&view=rev
Log:
Prep for next release

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=1410719&r1=1410718&r2=1410719&view=diff
==
--- tomcat/tc7.0.x/trunk/build.properties.default (original)
+++ tomcat/tc7.0.x/trunk/build.properties.default Sat Nov 17 10:18:33 2012
@@ -27,7 +27,7 @@
 # - Version Control Flags -
 version.major=7
 version.minor=0
-version.build=33
+version.build=34
 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=1410719&r1=1410718&r2=1410719&view=diff
==
--- tomcat/tc7.0.x/trunk/res/maven/mvn.properties.default (original)
+++ tomcat/tc7.0.x/trunk/res/maven/mvn.properties.default Sat Nov 17 10:18:33 
2012
@@ -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.33
+maven.asf.release.deploy.version=7.0.34
 
 #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



Re: Tomcat 7.0.33 progress report

2012-11-17 Thread Mark Thomas
On 16/11/2012 17:02, Mark Thomas wrote:
> I'll shout if I get stuck although the fastest way to find my errors
> seems to be writing an e-mail to the dev list as shortly after I send
> it, I spot the silly mistake that invalidates most of what I just wrote.

As it was in this case.

The TCKs all passed as well as the unit tests but the unit test failure
in trunk indicated an error that existed in 7.0.x and trunk. This has
been fixed and although it shouldn't impact the results of the unit
tests or the TCKs I am re-running them before I upload the release
candidate for voting.

We should have an RC for voting ready by Monday at the latest.

Mark


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



Re: Using comments.apache.org for our live docs

2012-11-17 Thread Rainer Jung
Thanks Konstantin. Will work over the weekend on your valuable remarks.

On 08.11.2012 23:25, Konstantin Kolinko wrote:
> 2012/11/8 Rainer Jung :
>> Cross posting intentionally, because our long time users list supporters
>> might want to comment as well.
>>
>> A few months ago a new Web Server committer, Daniel Gruno, suggested to use
>> a commenting system as part of the online documentation. He wanted to
>> include the disqus system. Some of his fellow committers were not very glad
>> with using an external system for the users comments and he sat down and
>> wrote an ASF commenting system. It is now running as an ASF service under
>> comments.apache.org.
>>
>> It allows users to add comments to documentation pages. Comments without
>> URLs and HTML tags are going live immediately without moderation, the other
>> ones need moderation first.
>>
>> We are using it in the web server project since a few months and we observe
>> close to no spam. Comment activity isn't to high, about 1 comments per day.
>> Some of those are not actually docs comments and they are responded by
>> referring the users to the users list. Some of them are really useful
>> because they help to clarify and improve documentation. In the meantime, the
>> trafficserver project also uses the feature.
>>
>> The comments are not meant to stay forever. Important content should be
>> integrated into the docs.
>>
>> Technically the commenting is done by adding a few lines of html and inline
>> JavaScript to each page, which then calls comments.apache.org. For the
>> Tomcat docs this can be done by adding those items to the XSL stylesheet
>> used to generate the HTML pages.
>>
>> I prepared a simple demo at:
>>
>> http://people.apache.org/~rjung/tomcat-docs-comments/tomcat-8.0-docs/
>>
>> It would be nice if you would have a look and we would discuss, whether we
>> find it useful or not. The patch for build.xml and the xsl that I applied to
>> build the comment enabled docs can be found at
>>
>> http://people.apache.org/~rjung/patches/tc-trunk-comments.patch
>>
>> A final version would include a reference to tomcat.apache.org instead of
>> people.apache.org/... The JavaScript checks the host header in order to
>> disable the feature if the docs are running on a different server, e.g.
>> inside a localhost Tomcat etc.
>>
> 
> Nice.
> 
> Several notes:
> 
>> A final version would include a reference to tomcat.apache.org
> 
> 1. I think it needs to also allow tomcat.[eu|us].apache.org mirrors
> and ci.apache.org where nightly builds of documentation are published.
> 
> 2. I think that the comments should be be hidden when the document is
> being printed.
> 
> 3. Regarding the "Comments" section header and notice
> 
> I think it would be better
> a) to have this section more distinct from the rest of the page
> (formatted as something "external" to the page itself), and
> 
> b) to write proper introduction to the comments feature somewhere one
> (formatted as a proper chapter/section of the documentation),
> e,g, in the "Introduction" chapter, or maybe on the main tomcat.apache.org 
> site.
> 
> The short notice section can have a link to this introduction.
> 
> 
> Regarding the notice section, or rather the introduction to the
> feature if we write one,
> I would like to see the following:
> 1) Maybe do not mention the IRC channel.
> 2) Maybe mention how the comments are used. (Copyrights, AL)
> 3) Maybe mention who sees the comments (Those who subscribe to receive
> them. They are not forwarded to the public mailing list).
> 
> Looking at httpd.a.o,
> - the comments section header there spans the whole page width.
> - the "Available Languages" line is above it.
> - the "notice" is distinct from the rest of text by using a red border
> - documentation and comments style is more consistent. They use the same font
> 
> 
> 4. Looking at httpd.a.o, I noticed a nice feature:
> http://httpd.apache.org/docs/2.2/configuring.html
> http://httpd.apache.org/docs/2.4/configuring.html
> 
> The "2.2" page has comments, the "2.4" does not. The following footer
> is added to the "2.4" page:
> "The 2.2 branch of the documentation has comments available for this
> page. Click here to view them."
> 
> 
> 5. It does not work well when I browse the main site through https.
> 
> It works, but most of the links back and forth redirect to the http
> version of it.
> Examples:
> a) The " Click here to view them." link mentioned above
> b) The "View" link on a comment in the list of comments on comments.apache.org
> 
> 
> 6. It is not clear what is lifecycle of a comment.
> 
> I see that when I log in then there is a link above each comment that
> allows to mark it as "Resolved". When (and who) is removing resolved
> comments?
> E.g. someone is supposed to do a manual sweep once the next minor
> Tomcat version is released and its updated documentation is published?
> The dashboard GUI is not very friendly for such a task.
> 
> Some message boards have a feature where a 

Making the Mapper type safe

2012-11-17 Thread Rainer Jung
Hi,

I did some experiments about more flexible version mapping for contextes
a while back and was a bit unhappy about the type unsafetyness of the
Mapper. Internally it only uses plain Objects and the code using it has
to cast everything.

There is a patch available at

http://people.apache.org/~rjung/patches/mapper-typesafe-trunk.patch

that makes the Mapper type safe. The only changes outside of the mapper
package is the removal of the casts. The TestMapper class had to be
adjusted to use Host, Context and Wrapper objects instead of Strings.

The test suite runs without errors.

One change is potentially problematic: MapperListener previously
registered a new context using the Container returned by
context.getParent(), ignoring whether it is a Host or something else. I
had to cast to Host, because registering a Context for any type of
Container is no longer possible. I doubt that previously it would have
really worked, because all code using the container returned later by
the mapper would have assumed it to be a Host.

The Context interface mentions in comments, that the parent might be
something else than a Host. Does anyone know a good example? Would it
have worked? What would the expected behaviour of MapperListener have
been (where would it have registered that context in the mapper)?

Regards,

Rainer

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



[GUMP@vmgump]: Project tomcat-taglibs-standard (in module tomcat-taglibs) failed

2012-11-17 Thread Gump
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-taglibs-standard has an issue affecting its community 
integration.
This issue affects 2 projects,
 and has been outstanding for 189 runs.
The current state of this project is 'Failed', with reason 'Build Failed'.
For reference only, the following projects are affected by this:
- tomcat-taglibs-standard :  Standard Taglib
- tomcat-taglibs-standard-install :  JSP Taglibs


Full details are available at:

http://vmgump.apache.org/gump/public/tomcat-taglibs/tomcat-taglibs-standard/index.html

That said, some information snippets are provided here.

The following annotations (debug/informational/warning/error messages) were 
provided:
 -INFO- Optional dependency httpunit failed with reason build failed
 -DEBUG- (Apache Gump generated) Apache Maven Settings in: 
/srv/gump/public/workspace/tomcat-taglibs/standard/gump_mvn_settings.xml
 -INFO- Failed with reason build failed
 -DEBUG- Maven POM in: 
/srv/gump/public/workspace/tomcat-taglibs/standard/pom.xml
 -INFO- Failed to extract fallback artifacts from Gump Repository



The following work was performed:
http://vmgump.apache.org/gump/public/tomcat-taglibs/tomcat-taglibs-standard/gump_work/build_tomcat-taglibs_tomcat-taglibs-standard.html
Work Name: build_tomcat-taglibs_tomcat-taglibs-standard (Type: Build)
Work ended in a state of : Failed
Elapsed: 23 secs
Command Line: /opt/maven2/bin/mvn --batch-mode -DskipTests=true --settings 
/srv/gump/public/workspace/tomcat-taglibs/standard/gump_mvn_settings.xml 
install 
[Working Directory: /srv/gump/public/workspace/tomcat-taglibs/standard]
M2_HOME: /opt/maven2
-
[INFO] [compiler:compile {execution: default-compile}]
[INFO] Nothing to compile - all classes are up to date
[debug] execute contextualize
[INFO] [resources:testResources {execution: default-testResources}]
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory 
/srv/gump/public/workspace/tomcat-taglibs/standard/spec/src/test/resources
[INFO] Copying 3 resources
[INFO] [compiler:testCompile {execution: default-testCompile}]
[INFO] No sources to compile
[INFO] [surefire:test {execution: default-test}]
[INFO] Tests are skipped.
[INFO] [bundle:bundle {execution: default-bundle}]
[INFO] [install:install {execution: default-install}]
[INFO] Installing 
/srv/gump/public/workspace/tomcat-taglibs/standard/spec/target/taglibs-standard-spec-1.2-SNAPSHOT.jar
 to 
/srv/gump/public/workspace/mvnlocalrepo/shared/org/apache/taglibs/taglibs-standard-spec/1.2-SNAPSHOT/taglibs-standard-spec-1.2-SNAPSHOT.jar
[INFO] [bundle:install {execution: default-install}]
[INFO] Parsing 
file:/srv/gump/public/workspace/mvnlocalrepo/shared/repository.xml
[INFO] Installing 
org/apache/taglibs/taglibs-standard-spec/1.2-SNAPSHOT/taglibs-standard-spec-1.2-SNAPSHOT.jar
[INFO] Writing OBR metadata
[INFO] 
[INFO] Building JSTL Implementation
[INFO]task-segment: [install]
[INFO] 
[INFO] [remote-resources:process {execution: default}]
[INFO] snapshot org.apache.taglibs:taglibs-standard-spec:1.2-SNAPSHOT: checking 
for updates from apache.snapshots
[debug] execute contextualize
[INFO] [resources:resources {execution: default-resources}]
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 14 resources
[INFO] Copying 3 resources
[INFO] [compiler:compile {execution: default-compile}]
[INFO] Compiling 96 source files to 
/srv/gump/public/workspace/tomcat-taglibs/standard/impl/target/classes
[INFO] -
[ERROR] COMPILATION ERROR : 
[INFO] -
[ERROR] 
/srv/gump/public/workspace/tomcat-taglibs/standard/impl/src/main/java/org/apache/taglibs/standard/tag/common/sql/DataSourceWrapper.java:[38,7]
 error: DataSourceWrapper is not abstract and does not override abstract method 
getParentLogger() in CommonDataSource
[INFO] 1 error
[INFO] -
[INFO] 
[ERROR] BUILD FAILURE
[INFO] 
[INFO] Compilation failure
/srv/gump/public/workspace/tomcat-taglibs/standard/impl/src/main/java/org/apache/taglibs/standard/tag/common/sql/DataSourceWrapper.java:[38,7]
 error: DataSourceWrapper is not abstract and does not override abstract method 
getParentLogger() in CommonDataSource

[INFO] 
[INFO] For more information, run Maven with the -e switch
[INFO] --

Re: Making the Mapper type safe

2012-11-17 Thread Mark Thomas
On 17/11/2012 13:05, Rainer Jung wrote:
> Hi,
> 
> I did some experiments about more flexible version mapping for contextes
> a while back and was a bit unhappy about the type unsafetyness of the
> Mapper. Internally it only uses plain Objects and the code using it has
> to cast everything.
> 
> There is a patch available at
> 
> http://people.apache.org/~rjung/patches/mapper-typesafe-trunk.patch

There is a lot to take in there. I am generally in agreement but find
myself rather overwhelmed by the patch. May I suggest you go ahead with
just the renaming of Host->MappedHost etc and commit that to trunk
(should be simple to do in an IDE) and then prepare a patch that makes
the mapper type safe.

If it would be easier for me to make the name changes and then you could
just svn up I'm happy to do so. Let me know.

> that makes the Mapper type safe. The only changes outside of the mapper
> package is the removal of the casts. The TestMapper class had to be
> adjusted to use Host, Context and Wrapper objects instead of Strings.
> 
> The test suite runs without errors.
> 
> One change is potentially problematic: MapperListener previously
> registered a new context using the Container returned by
> context.getParent(), ignoring whether it is a Host or something else. I
> had to cast to Host, because registering a Context for any type of
> Container is no longer possible. I doubt that previously it would have
> really worked, because all code using the container returned later by
> the mapper would have assumed it to be a Host.
> 
> The Context interface mentions in comments, that the parent might be
> something else than a Host. Does anyone know a good example?

I can't think of a single example.

> Would it have worked?

No. I see multiple places where the result would be an NPE if
Context->Host->Engine is not in place.

> What would the expected behaviour of MapperListener have
> been (where would it have registered that context in the mapper)?

I doubt the start up processes would have got that far.

I suspect that the initial design allowed for a different container
hierarchy but over time the Wrapper->Context->Host->Engine hierarchy has
become embedded in numerous places in the code. I see no reason to
unpick that so lets make it explicit and override getParent() for
Wrapper, Context and Host for Tomcat 8. It will enable us to remove lots
of casting and we may even find a few obscure bugs.

Mark


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



svn commit: r1410732 - /tomcat/trunk/java/org/apache/tomcat/util/modeler/NotificationInfo.java

2012-11-17 Thread markt
Author: markt
Date: Sat Nov 17 14:53:03 2012
New Revision: 1410732

URL: http://svn.apache.org/viewvc?rev=1410732&view=rev
Log:
Fix a FindBugs issue (incorrect synchronization)

Modified:
tomcat/trunk/java/org/apache/tomcat/util/modeler/NotificationInfo.java

Modified: tomcat/trunk/java/org/apache/tomcat/util/modeler/NotificationInfo.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/modeler/NotificationInfo.java?rev=1410732&r1=1410731&r2=1410732&view=diff
==
--- tomcat/trunk/java/org/apache/tomcat/util/modeler/NotificationInfo.java 
(original)
+++ tomcat/trunk/java/org/apache/tomcat/util/modeler/NotificationInfo.java Sat 
Nov 17 14:53:03 2012
@@ -14,14 +14,14 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-
-
 package org.apache.tomcat.util.modeler;
 
+import java.util.concurrent.locks.Lock;
+import java.util.concurrent.locks.ReadWriteLock;
+import java.util.concurrent.locks.ReentrantReadWriteLock;
 
 import javax.management.MBeanNotificationInfo;
 
-
 /**
  * Internal configuration information for a Notification
  * descriptor.
@@ -29,8 +29,8 @@ import javax.management.MBeanNotificatio
  * @author Craig R. McClanahan
  * @version $Id$
  */
-
 public class NotificationInfo extends FeatureInfo {
+
 static final long serialVersionUID = -6319885418912650856L;
 
 // - Instance Variables
@@ -42,6 +42,7 @@ public class NotificationInfo extends Fe
  */
 transient MBeanNotificationInfo info = null;
 protected String notifTypes[] = new String[0];
+protected ReadWriteLock notifTypesLock = new ReentrantReadWriteLock();
 
 // - Properties
 
@@ -74,7 +75,13 @@ public class NotificationInfo extends Fe
  * The set of notification types for this MBean.
  */
 public String[] getNotifTypes() {
-return (this.notifTypes);
+Lock readLock = notifTypesLock.readLock();
+try {
+readLock.lock();
+return this.notifTypes;
+} finally {
+readLock.unlock();
+}
 }
 
 
@@ -88,14 +95,18 @@ public class NotificationInfo extends Fe
  */
 public void addNotifType(String notifType) {
 
-synchronized (notifTypes) {
+Lock writeLock = notifTypesLock.writeLock();
+try {
+writeLock.lock();
+
 String results[] = new String[notifTypes.length + 1];
 System.arraycopy(notifTypes, 0, results, 0, notifTypes.length);
 results[notifTypes.length] = notifType;
 notifTypes = results;
 this.info = null;
+} finally {
+writeLock.unlock();
 }
-
 }
 
 
@@ -107,7 +118,7 @@ public class NotificationInfo extends Fe
 
 // Return our cached information (if any)
 if (info != null)
-return (info);
+return info;
 
 // Create and return a new information object
 info = new MBeanNotificationInfo
@@ -115,7 +126,7 @@ public class NotificationInfo extends Fe
 //Descriptor descriptor = info.getDescriptor();
 //addFields(descriptor);
 //info.setDescriptor(descriptor);
-return (info);
+return info;
 
 }
 
@@ -132,11 +143,14 @@ public class NotificationInfo extends Fe
 sb.append(", description=");
 sb.append(description);
 sb.append(", notifTypes=");
-sb.append(notifTypes.length);
+Lock readLock = notifTypesLock.readLock();
+try {
+readLock.lock();
+sb.append(notifTypes.length);
+} finally {
+readLock.unlock();
+}
 sb.append("]");
 return (sb.toString());
-
 }
-
-
 }



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



svn commit: r1410733 - /tomcat/trunk/res/findbugs/filter-false-positives.xml

2012-11-17 Thread markt
Author: markt
Date: Sat Nov 17 14:53:20 2012
New Revision: 1410733

URL: http://svn.apache.org/viewvc?rev=1410733&view=rev
Log:
Silence another false positive

Modified:
tomcat/trunk/res/findbugs/filter-false-positives.xml

Modified: tomcat/trunk/res/findbugs/filter-false-positives.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/res/findbugs/filter-false-positives.xml?rev=1410733&r1=1410732&r2=1410733&view=diff
==
--- tomcat/trunk/res/findbugs/filter-false-positives.xml (original)
+++ tomcat/trunk/res/findbugs/filter-false-positives.xml Sat Nov 17 14:53:20 
2012
@@ -278,6 +278,12 @@
 
   
   
+
+
+
+
+  
+  
 
 
 



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



svn commit: r1410740 - /tomcat/trunk/java/org/apache/tomcat/util/modeler/OperationInfo.java

2012-11-17 Thread markt
Author: markt
Date: Sat Nov 17 15:20:08 2012
New Revision: 1410740

URL: http://svn.apache.org/viewvc?rev=1410740&view=rev
Log:
Fix a FindBugs issue (incorrect synchronization)

Modified:
tomcat/trunk/java/org/apache/tomcat/util/modeler/OperationInfo.java

Modified: tomcat/trunk/java/org/apache/tomcat/util/modeler/OperationInfo.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/modeler/OperationInfo.java?rev=1410740&r1=1410739&r2=1410740&view=diff
==
--- tomcat/trunk/java/org/apache/tomcat/util/modeler/OperationInfo.java 
(original)
+++ tomcat/trunk/java/org/apache/tomcat/util/modeler/OperationInfo.java Sat Nov 
17 15:20:08 2012
@@ -14,17 +14,16 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-
-
 package org.apache.tomcat.util.modeler;
 
-
 import java.util.Locale;
+import java.util.concurrent.locks.Lock;
+import java.util.concurrent.locks.ReadWriteLock;
+import java.util.concurrent.locks.ReentrantReadWriteLock;
 
 import javax.management.MBeanOperationInfo;
 import javax.management.MBeanParameterInfo;
 
-
 /**
  * Internal configuration information for an Operation
  * descriptor.
@@ -32,23 +31,24 @@ import javax.management.MBeanParameterIn
  * @author Craig R. McClanahan
  */
 public class OperationInfo extends FeatureInfo {
+
 static final long serialVersionUID = 4418342922072614875L;
-// --- Constructors
 
+// --- Constructors
 
 /**
  * Standard zero-arguments constructor.
  */
 public OperationInfo() {
-
 super();
-
 }
 
+
 // - Instance Variables
 
 protected String impact = "UNKNOWN";
 protected String role = "operation";
+protected ReadWriteLock parametersLock = new ReentrantReadWriteLock();
 protected ParameterInfo parameters[] = new ParameterInfo[0];
 
 
@@ -59,7 +59,7 @@ public class OperationInfo extends Featu
  * string value "ACTION", "ACTION_INFO", "INFO", or "UNKNOWN".
  */
 public String getImpact() {
-return (this.impact);
+return this.impact;
 }
 
 public void setImpact(String impact) {
@@ -75,7 +75,7 @@ public class OperationInfo extends Featu
  * "constructor").
  */
 public String getRole() {
-return (this.role);
+return this.role;
 }
 
 public void setRole(String role) {
@@ -102,7 +102,13 @@ public class OperationInfo extends Featu
  * The set of parameters for this operation.
  */
 public ParameterInfo[] getSignature() {
-return (this.parameters);
+Lock readLock = parametersLock.readLock();
+try {
+readLock.lock();
+return this.parameters;
+} finally {
+readLock.unlock();
+}
 }
 
 // - Public Methods
@@ -115,14 +121,17 @@ public class OperationInfo extends Featu
  */
 public void addParameter(ParameterInfo parameter) {
 
-synchronized (parameters) {
+Lock writeLock = parametersLock.writeLock();
+try {
+writeLock.lock();
 ParameterInfo results[] = new ParameterInfo[parameters.length + 1];
 System.arraycopy(parameters, 0, results, 0, parameters.length);
 results[parameters.length] = parameter;
 parameters = results;
 this.info = null;
+} finally {
+writeLock.unlock();
 }
-
 }
 
 



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



buildbot success in ASF Buildbot on tomcat-trunk

2012-11-17 Thread buildbot
The Buildbot has detected a restored build on builder tomcat-trunk while 
building ASF Buildbot.
Full details are available at:
 http://ci.apache.org/builders/tomcat-trunk/builds/3574

Buildbot URL: http://ci.apache.org/

Buildslave for this Build: bb-vm_ubuntu

Build Reason: scheduler
Build Source Stamp: [branch tomcat/trunk] 1410733
Blamelist: markt

Build succeeded!

sincerely,
 -The Buildbot





svn commit: r1410742 - /tomcat/trunk/java/org/apache/jasper/compiler/ELParser.java

2012-11-17 Thread markt
Author: markt
Date: Sat Nov 17 15:53:34 2012
New Revision: 1410742

URL: http://svn.apache.org/viewvc?rev=1410742&view=rev
Log:
Refactor since returning -1 makes no sense with chars. The old code worked but 
it was a little odd (and FindBugs didn't like it).

Modified:
tomcat/trunk/java/org/apache/jasper/compiler/ELParser.java

Modified: tomcat/trunk/java/org/apache/jasper/compiler/ELParser.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/jasper/compiler/ELParser.java?rev=1410742&r1=1410741&r2=1410742&view=diff
==
--- tomcat/trunk/java/org/apache/jasper/compiler/ELParser.java (original)
+++ tomcat/trunk/java/org/apache/jasper/compiler/ELParser.java Sat Nov 17 
15:53:34 2012
@@ -240,8 +240,9 @@ public class ELParser {
 if (Character.isJavaIdentifierStart(ch)) {
 StringBuilder buf = new StringBuilder();
 buf.append(ch);
-while ((ch = peekChar()) != -1
-&& Character.isJavaIdentifierPart(ch)) {
+while (index < expression.length() &&
+Character.isJavaIdentifierPart(
+ch = expression.charAt(index))) {
 buf.append(ch);
 nextChar();
 }
@@ -307,13 +308,6 @@ public class ELParser {
 return expression.charAt(index++);
 }
 
-private char peekChar() {
-if (index >= expression.length()) {
-return (char) -1;
-}
-return expression.charAt(index);
-}
-
 private int getIndex() {
 return index;
 }



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



svn commit: r1410752 - in /tomcat/trunk: java/org/apache/catalina/mapper/Mapper.java test/org/apache/catalina/mapper/TestMapper.java

2012-11-17 Thread rjung
Author: rjung
Date: Sat Nov 17 17:12:18 2012
New Revision: 1410752

URL: http://svn.apache.org/viewvc?rev=1410752&view=rev
Log:
Type-Safety for the Mapper - Part 1.

Rename inner classes Host, Context and Wrapper
to MappedHost, MappedContext and MappedWrapper.

Modified:
tomcat/trunk/java/org/apache/catalina/mapper/Mapper.java
tomcat/trunk/test/org/apache/catalina/mapper/TestMapper.java

Modified: tomcat/trunk/java/org/apache/catalina/mapper/Mapper.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/mapper/Mapper.java?rev=1410752&r1=1410751&r2=1410752&view=diff
==
--- tomcat/trunk/java/org/apache/catalina/mapper/Mapper.java (original)
+++ tomcat/trunk/java/org/apache/catalina/mapper/Mapper.java Sat Nov 17 
17:12:18 2012
@@ -47,7 +47,7 @@ public final class Mapper {
 /**
  * Array containing the virtual hosts definitions.
  */
-protected Host[] hosts = new Host[0];
+protected MappedHost[] hosts = new MappedHost[0];
 
 
 /**
@@ -83,8 +83,8 @@ public final class Mapper {
  */
 public synchronized void addHost(String name, String[] aliases,
  Object host) {
-Host[] newHosts = new Host[hosts.length + 1];
-Host newHost = new Host();
+MappedHost[] newHosts = new MappedHost[hosts.length + 1];
+MappedHost newHost = new MappedHost();
 ContextList contextList = new ContextList();
 newHost.name = name;
 newHost.contextList = contextList;
@@ -93,8 +93,8 @@ public final class Mapper {
 hosts = newHosts;
 }
 for (int i = 0; i < aliases.length; i++) {
-newHosts = new Host[hosts.length + 1];
-newHost = new Host();
+newHosts = new MappedHost[hosts.length + 1];
+newHost = new MappedHost();
 newHost.name = aliases[i];
 newHost.contextList = contextList;
 newHost.object = host;
@@ -117,14 +117,14 @@ public final class Mapper {
 return;
 }
 Object host = hosts[pos].object;
-Host[] newHosts = new Host[hosts.length - 1];
+MappedHost[] newHosts = new MappedHost[hosts.length - 1];
 if (removeMap(hosts, newHosts, name)) {
 hosts = newHosts;
 }
 // Remove all aliases (they will map to the same host object)
 for (int i = 0; i < newHosts.length; i++) {
 if (newHosts[i].object == host) {
-Host[] newHosts2 = new Host[hosts.length - 1];
+MappedHost[] newHosts2 = new MappedHost[hosts.length - 1];
 if (removeMap(hosts, newHosts2, newHosts[i].name)) {
 hosts = newHosts2;
 }
@@ -144,10 +144,10 @@ public final class Mapper {
 // just in case...
 return;
 }
-Host realHost = hosts[pos];
+MappedHost realHost = hosts[pos];
 
-Host[] newHosts = new Host[hosts.length + 1];
-Host newHost = new Host();
+MappedHost[] newHosts = new MappedHost[hosts.length + 1];
+MappedHost newHost = new MappedHost();
 newHost.name = alias;
 newHost.contextList = realHost.contextList;
 newHost.object = realHost.object;
@@ -166,7 +166,7 @@ public final class Mapper {
 if (pos < 0) {
 return;
 }
-Host[] newHosts = new Host[hosts.length - 1];
+MappedHost[] newHosts = new MappedHost[hosts.length - 1];
 if (removeMap(hosts, newHosts, alias)) {
 hosts = newHosts;
 }
@@ -189,7 +189,7 @@ public final class Mapper {
 String version, Object context, String[] welcomeResources,
 WebResourceRoot resources) {
 
-Host[] hosts = this.hosts;
+MappedHost[] hosts = this.hosts;
 int pos = find(hosts, hostName);
 if( pos <0 ) {
 addHost(hostName, new String[0], host);
@@ -199,27 +199,27 @@ public final class Mapper {
 if (pos < 0) {
 log.error("No host found: " + hostName);
 }
-Host mappedHost = hosts[pos];
+MappedHost mappedHost = hosts[pos];
 if (mappedHost.name.equals(hostName)) {
 int slashCount = slashCount(path);
 synchronized (mappedHost) {
-Context[] contexts = mappedHost.contextList.contexts;
+MappedContext[] contexts = mappedHost.contextList.contexts;
 // Update nesting
 if (slashCount > mappedHost.contextList.nesting) {
 mappedHost.contextList.nesting = slashCount;
 }
 int pos2 = find(contexts, path);
 if (pos2 < 0 || !path.equals(contexts[pos2].name)) {
-Context newContext = new Context();
+MappedContext newContext = new MappedContext();
 newContext.name = 

Re: Making the Mapper type safe

2012-11-17 Thread Rainer Jung
On 17.11.2012 15:35, Mark Thomas wrote:
> On 17/11/2012 13:05, Rainer Jung wrote:
>> Hi,
>>
>> I did some experiments about more flexible version mapping for contextes
>> a while back and was a bit unhappy about the type unsafetyness of the
>> Mapper. Internally it only uses plain Objects and the code using it has
>> to cast everything.
>>
>> There is a patch available at
>>
>> http://people.apache.org/~rjung/patches/mapper-typesafe-trunk.patch
> 
> There is a lot to take in there. I am generally in agreement but find
> myself rather overwhelmed by the patch. May I suggest you go ahead with
> just the renaming of Host->MappedHost etc and commit that to trunk
> (should be simple to do in an IDE) and then prepare a patch that makes
> the mapper type safe.

Renaming committed as r1410752.

Remaining patch (half the size, biggest part in TestMapper, plus the
generics in MapElement):

http://people.apache.org/~rjung/patches/mapper-typesafe-trunk-v2.patch

> If it would be easier for me to make the name changes and then you could
> just svn up I'm happy to do so. Let me know.

Thanks, it was easy for me as well :)

>> One change is potentially problematic: MapperListener previously
>> registered a new context using the Container returned by
>> context.getParent(), ignoring whether it is a Host or something else. I
>> had to cast to Host, because registering a Context for any type of
>> Container is no longer possible. I doubt that previously it would have
>> really worked, because all code using the container returned later by
>> the mapper would have assumed it to be a Host.
>>
>> The Context interface mentions in comments, that the parent might be
>> something else than a Host. Does anyone know a good example?
> 
> I can't think of a single example.
> 
>> Would it have worked?
> 
> No. I see multiple places where the result would be an NPE if
> Context->Host->Engine is not in place.
> 
>> What would the expected behaviour of MapperListener have
>> been (where would it have registered that context in the mapper)?
> 
> I doubt the start up processes would have got that far.

OK

> I suspect that the initial design allowed for a different container
> hierarchy but over time the Wrapper->Context->Host->Engine hierarchy has
> become embedded in numerous places in the code. I see no reason to
> unpick that so lets make it explicit and override getParent() for
> Wrapper, Context and Host for Tomcat 8. It will enable us to remove lots
> of casting and we may even find a few obscure bugs.

Good, so that could be a next step after the remaining mapper
type-safety changes.

Regards,

Rainer

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



AccessLogValve test failure (timezone)

2012-11-17 Thread Rainer Jung
Currently the ALV tests fail for me because of a hard coded timezone in
the test. We can either copy some logic from ALV into the test class, or
make the private static method getTimeZone(() protected and call it from
the test class. A patch for the latter approach is available at:

http://people.apache.org/~rjung/patches/test-alv-timezone.patch

If we wanted to copy in, the code basically boils down to

timezone = TimeZone.getDefault();
if (timezone.inDaylightTime(date)) {
int offset = timezone.getDSTSavings();
return calculateTimeZoneOffset(timezone.getRawOffset() + offset);
} else {
return calculateTimeZoneOffset(timezone.getRawOffset());
}

and calculateTimeZoneOffset() is another 20 lines private static method
that formats the timezone offset.

I'd go with making the getTimeZone(() method protected. Of course we
then do not test the timezone formatting code, only the caching, like we
do now.

Regards,

Rainer

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



Re: Making the Mapper type safe

2012-11-17 Thread Mark Thomas
On 17/11/2012 17:22, Rainer Jung wrote:
> On 17.11.2012 15:35, Mark Thomas wrote:
>> On 17/11/2012 13:05, Rainer Jung wrote:
>>> Hi,
>>>
>>> I did some experiments about more flexible version mapping for contextes
>>> a while back and was a bit unhappy about the type unsafetyness of the
>>> Mapper. Internally it only uses plain Objects and the code using it has
>>> to cast everything.
>>>
>>> There is a patch available at
>>>
>>> http://people.apache.org/~rjung/patches/mapper-typesafe-trunk.patch
>>
>> There is a lot to take in there. I am generally in agreement but find
>> myself rather overwhelmed by the patch. May I suggest you go ahead with
>> just the renaming of Host->MappedHost etc and commit that to trunk
>> (should be simple to do in an IDE) and then prepare a patch that makes
>> the mapper type safe.
> 
> Renaming committed as r1410752.

Thanks. Much easier to review.

> Remaining patch (half the size, biggest part in TestMapper, plus the
> generics in MapElement):
> 
> http://people.apache.org/~rjung/patches/mapper-typesafe-trunk-v2.patch

+1. Looks good to me.

>> I suspect that the initial design allowed for a different container
>> hierarchy but over time the Wrapper->Context->Host->Engine hierarchy has
>> become embedded in numerous places in the code. I see no reason to
>> unpick that so lets make it explicit and override getParent() for
>> Wrapper, Context and Host for Tomcat 8. It will enable us to remove lots
>> of casting and we may even find a few obscure bugs.
> 
> Good, so that could be a next step after the remaining mapper
> type-safety changes.

+1

Mark


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



[Tomcat Wiki] Update of "PoweredBy" by Neha

2012-11-17 Thread Apache Wiki
Dear Wiki user,

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

The "PoweredBy" page has been changed by Neha:
http://wiki.apache.org/tomcat/PoweredBy?action=diff&rev1=446&rev2=447

  
  === Merkaweb.com (Web Hosting Spain) ===
  {{http://www.merkaweb.com/images/logo2.png}} 
[[http://www.merkaweb.com|Alojamiento Web]] - Spanish based web hosting company 
supporting Tomcat in all cPanel web hosting plans. Professional web hosting 
services since 2003.
+ 
+ === Milesweb.com (Web Hosting India) ===
+ [[http://www.milesweb.com/hosting-faqs/what-is-apache-tomcat-hosting/|Apache 
Tomcat Hosting]] - India based web hosting company supporting Tomcat in all 
cPanel web hosting plans.
  
  === Pickaweb - UK Web Hosting ===
  
[[http://www.pickaweb.co.uk|{{http://www.pickaweb.co.uk/images/logo.png|http://www.pickaweb.co.uk}}]]
 [[http://www.pickaweb.co.uk|UK Web Hosting]] - Supporting Tomcat in all of our 
web hosting plans. Professional UK based web hosting services since 2000. 24x7 
Support via chat, phone & email.

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



svn commit: r1410763 - /tomcat/trunk/test/org/apache/catalina/valves/TestAccessLogValve.java

2012-11-17 Thread rjung
Author: rjung
Date: Sat Nov 17 19:25:18 2012
New Revision: 1410763

URL: http://svn.apache.org/viewvc?rev=1410763&view=rev
Log:
Fix time zone formatting in ALV test.
Test failed in non-GMT time zone.

Modified:
tomcat/trunk/test/org/apache/catalina/valves/TestAccessLogValve.java

Modified: tomcat/trunk/test/org/apache/catalina/valves/TestAccessLogValve.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/valves/TestAccessLogValve.java?rev=1410763&r1=1410762&r2=1410763&view=diff
==
--- tomcat/trunk/test/org/apache/catalina/valves/TestAccessLogValve.java 
(original)
+++ tomcat/trunk/test/org/apache/catalina/valves/TestAccessLogValve.java Sat 
Nov 17 19:25:18 2012
@@ -34,7 +34,7 @@ public class TestAccessLogValve {
 final int cacheSize = 10;
 
 SimpleDateFormat sdf =
-new SimpleDateFormat("dd/MMM/:HH:mm:ss", Locale.US);
+new SimpleDateFormat("[dd/MMM/:HH:mm:ss Z]", Locale.US);
 sdf.setTimeZone(TimeZone.getDefault());
 
 AccessLogValve.DateFormatCache dfc =
@@ -89,6 +89,7 @@ public class TestAccessLogValve {
 }
 
 private String generateExpected(SimpleDateFormat sdf, long secs) {
-return "[" + sdf.format(new Date(secs * 1000)) + " +]";
+System.out.println("generateExpected: " + sdf.format(new Date(secs * 
1000)));
+return sdf.format(new Date(secs * 1000));
 }
 }



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



Re: AccessLogValve test failure (timezone)

2012-11-17 Thread Rainer Jung
On 17.11.2012 18:53, Rainer Jung wrote:
> Currently the ALV tests fail for me because of a hard coded timezone in
> the test. We can either copy some logic from ALV into the test class, or
> make the private static method getTimeZone(() protected and call it from
> the test class. A patch for the latter approach is available at:
> 
> http://people.apache.org/~rjung/patches/test-alv-timezone.patch
> 
> If we wanted to copy in, the code basically boils down to
> 
> timezone = TimeZone.getDefault();
> if (timezone.inDaylightTime(date)) {
> int offset = timezone.getDSTSavings();
> return calculateTimeZoneOffset(timezone.getRawOffset() + offset);
> } else {
> return calculateTimeZoneOffset(timezone.getRawOffset());
> }
> 
> and calculateTimeZoneOffset() is another 20 lines private static method
> that formats the timezone offset.
> 
> I'd go with making the getTimeZone(() method protected. Of course we
> then do not test the timezone formatting code, only the caching, like we
> do now.

Rubbish. Failure fixed in a simpler way by r1410763.

Regards,

Rainer


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



svn commit: r1410764 - in /tomcat/trunk: build.properties.default java/org/apache/jasper/compiler/JDTCompiler.java

2012-11-17 Thread markt
Author: markt
Date: Sat Nov 17 19:27:33 2012
New Revision: 1410764

URL: http://svn.apache.org/viewvc?rev=1410764&view=rev
Log:
Update JDT complier to 4.2.1
Fixes Findbugs warning

Modified:
tomcat/trunk/build.properties.default
tomcat/trunk/java/org/apache/jasper/compiler/JDTCompiler.java

Modified: tomcat/trunk/build.properties.default
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/build.properties.default?rev=1410764&r1=1410763&r2=1410764&view=diff
==
--- tomcat/trunk/build.properties.default (original)
+++ tomcat/trunk/build.properties.default Sat Nov 17 19:27:33 2012
@@ -122,14 +122,14 @@ wsdl4j-lib.home=${base.path}/wsdl4j-${ws
 
wsdl4j-lib.loc=${base-maven.loc}/wsdl4j/wsdl4j/${wsdl4j-lib.version}/wsdl4j-${wsdl4j-lib.version}.jar
 wsdl4j-lib.jar=${wsdl4j-lib.home}/wsdl4j-${wsdl4j-lib.version}.jar
 
-# - Eclipse JDT, version 3.2 or later -
-jdt.version=3.7.2
-jdt.release=R-3.7.2-201202080800
+# - Eclipse JDT, version 3.2 or later -#
+jdt.version=4.2.1
+jdt.release=R-4.2.1-201209141800
 jdt.home=${base.path}/ecj-${jdt.version}
 jdt.jar=${jdt.home}/ecj-${jdt.version}.jar
 # The download will be moved to the archive area eventually. We are taking 
care of that in advance.
-jdt.loc.1=http://archive.eclipse.org/eclipse/downloads/drops/${jdt.release}/ecj-${jdt.version}.jar
-jdt.loc.2=http://download.eclipse.org/eclipse/downloads/drops/${jdt.release}/ecj-${jdt.version}.jar
+jdt.loc.1=http://archive.eclipse.org/eclipse/downloads/drops4/${jdt.release}/ecj-${jdt.version}.jar
+jdt.loc.2=http://download.eclipse.org/eclipse/downloads/drops4/${jdt.release}/ecj-${jdt.version}.jar
 
 #  NPN support
 npn.version=8.1.2.v20120308

Modified: tomcat/trunk/java/org/apache/jasper/compiler/JDTCompiler.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/jasper/compiler/JDTCompiler.java?rev=1410764&r1=1410763&r2=1410764&view=diff
==
--- tomcat/trunk/java/org/apache/jasper/compiler/JDTCompiler.java (original)
+++ tomcat/trunk/java/org/apache/jasper/compiler/JDTCompiler.java Sat Nov 17 
19:27:33 2012
@@ -165,8 +165,7 @@ public class JDTCompiler extends org.apa
 return result;
 }
 
-@SuppressWarnings("unused") // New method added to interface in
-// later JDT versions
+@Override
 public boolean ignoreOptionalProblems() {
 return false;
 }



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



Re: svn commit: r1410764 - in /tomcat/trunk: build.properties.default java/org/apache/jasper/compiler/JDTCompiler.java

2012-11-17 Thread Mark Thomas
On 17/11/2012 19:27, ma...@apache.org wrote:
> Author: markt
> Date: Sat Nov 17 19:27:33 2012
> New Revision: 1410764
> 
> URL: http://svn.apache.org/viewvc?rev=1410764&view=rev
> Log:
> Update JDT complier to 4.2.1

Thoughts on back-porting this to 7.0.x?

Mark



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



svn commit: r1410766 - /tomcat/trunk/test/org/apache/catalina/valves/TestAccessLogValve.java

2012-11-17 Thread rjung
Author: rjung
Date: Sat Nov 17 19:47:59 2012
New Revision: 1410766

URL: http://svn.apache.org/viewvc?rev=1410766&view=rev
Log:
Sorry, committed debug code in r1410763.

Modified:
tomcat/trunk/test/org/apache/catalina/valves/TestAccessLogValve.java

Modified: tomcat/trunk/test/org/apache/catalina/valves/TestAccessLogValve.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/valves/TestAccessLogValve.java?rev=1410766&r1=1410765&r2=1410766&view=diff
==
--- tomcat/trunk/test/org/apache/catalina/valves/TestAccessLogValve.java 
(original)
+++ tomcat/trunk/test/org/apache/catalina/valves/TestAccessLogValve.java Sat 
Nov 17 19:47:59 2012
@@ -89,7 +89,6 @@ public class TestAccessLogValve {
 }
 
 private String generateExpected(SimpleDateFormat sdf, long secs) {
-System.out.println("generateExpected: " + sdf.format(new Date(secs * 
1000)));
 return sdf.format(new Date(secs * 1000));
 }
 }



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



svn commit: r1410770 - in /tomcat/tc7.0.x/trunk: ./ test/org/apache/catalina/valves/TestAccessLogValve.java webapps/docs/changelog.xml

2012-11-17 Thread rjung
Author: rjung
Date: Sat Nov 17 19:54:35 2012
New Revision: 1410770

URL: http://svn.apache.org/viewvc?rev=1410770&view=rev
Log:
Fix time zone formatting in ALV test.
Test failed in non-GMT time zone.

Backport of r1410763 and r1410766 from trunk.

Modified:
tomcat/tc7.0.x/trunk/   (props changed)
tomcat/tc7.0.x/trunk/test/org/apache/catalina/valves/TestAccessLogValve.java
tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml

Propchange: tomcat/tc7.0.x/trunk/
--
  Merged /tomcat/trunk:r1410763,1410766

Modified: 
tomcat/tc7.0.x/trunk/test/org/apache/catalina/valves/TestAccessLogValve.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/test/org/apache/catalina/valves/TestAccessLogValve.java?rev=1410770&r1=1410769&r2=1410770&view=diff
==
--- 
tomcat/tc7.0.x/trunk/test/org/apache/catalina/valves/TestAccessLogValve.java 
(original)
+++ 
tomcat/tc7.0.x/trunk/test/org/apache/catalina/valves/TestAccessLogValve.java 
Sat Nov 17 19:54:35 2012
@@ -34,7 +34,7 @@ public class TestAccessLogValve {
 final int cacheSize = 10;
 
 SimpleDateFormat sdf =
-new SimpleDateFormat("dd/MMM/:HH:mm:ss", Locale.US);
+new SimpleDateFormat("[dd/MMM/:HH:mm:ss Z]", Locale.US);
 sdf.setTimeZone(TimeZone.getDefault());
 
 AccessLogValve.DateFormatCache dfc =
@@ -89,6 +89,6 @@ public class TestAccessLogValve {
 }
 
 private String generateExpected(SimpleDateFormat sdf, long secs) {
-return "[" + sdf.format(new Date(secs * 1000)) + " +]";
+return sdf.format(new Date(secs * 1000));
 }
 }

Modified: tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml?rev=1410770&r1=1410769&r2=1410770&view=diff
==
--- tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml Sat Nov 17 19:54:35 2012
@@ -53,6 +53,15 @@
   They eventually become mixed with the numbered issues. (I.e., numbered
   issues to not "pop up" wrt. others).
 -->
+
+  
+
+  
+Fix unit test for AccessLogValve when using non-GMT time zone. (rjung)
+  
+
+  
+
 
   
 



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



Re: svn commit: r1410764 - in /tomcat/trunk: build.properties.default java/org/apache/jasper/compiler/JDTCompiler.java

2012-11-17 Thread Konstantin Kolinko
2012/11/17 Mark Thomas :
> On 17/11/2012 19:27, ma...@apache.org wrote:
>> Author: markt
>> Date: Sat Nov 17 19:27:33 2012
>> New Revision: 1410764
>>
>> URL: http://svn.apache.org/viewvc?rev=1410764&view=rev
>> Log:
>> Update JDT complier to 4.2.1
>
> Thoughts on back-porting this to 7.0.x?
>

+1
The compiler in Eclipse 3.7.2 was crashing for me when compiling
Tomcat 8. I switched over to 4.2.1 a week ago and the problem
disappeared.   Someone else with Java 7 could be experiencing a
similar issue, so it is better to upgrade.

Best regards,
Konstantin Kolinko

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



Re: svn commit: r1408739 - in /tomcat/trunk: java/org/apache/catalina/startup/RealmRuleSet.java webapps/docs/config/systemprops.xml

2012-11-17 Thread Christopher Schultz
Mark,

Apologies for the late reply. Please see comments inline.

On 11/13/12 9:17 AM, ma...@apache.org wrote:
> Author: markt
> Date: Tue Nov 13 14:17:42 2012
> New Revision: 1408739
> 
> URL: http://svn.apache.org/viewvc?rev=1408739&view=rev
> Log:
> Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=54141
> Increase the maximum number of supported nested realm levels from 2 to 3 and 
> make the maximum configurable via a system property.
> 
> Modified:
> tomcat/trunk/java/org/apache/catalina/startup/RealmRuleSet.java
> tomcat/trunk/webapps/docs/config/systemprops.xml
> 
> Modified: tomcat/trunk/java/org/apache/catalina/startup/RealmRuleSet.java
> URL: 
> http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/startup/RealmRuleSet.java?rev=1408739&r1=1408738&r2=1408739&view=diff
> ==
> --- tomcat/trunk/java/org/apache/catalina/startup/RealmRuleSet.java (original)
> +++ tomcat/trunk/java/org/apache/catalina/startup/RealmRuleSet.java Tue Nov 
> 13 14:17:42 2012
> @@ -34,6 +34,10 @@ import org.apache.tomcat.util.digester.R
>  public class RealmRuleSet extends RuleSetBase {
>  
>  
> +private static final int MAX_NESTED_REALM_LEVELS = Integer.getInteger(
> +
> "org.apache.catalina.startup.RealmRuleSet.MAX_NESTED_REALM_LEVELS",
> +3).intValue();
> +
>  // - Instance 
> Variables
>  
>  
> @@ -83,23 +87,28 @@ public class RealmRuleSet extends RuleSe
>  @Override
>  public void addRuleInstances(Digester digester) {
>  
> -digester.addObjectCreate(prefix + "Realm",
> - null, // MUST be specified in the element,
> - "className");
> -digester.addSetProperties(prefix + "Realm");
> -digester.addSetNext(prefix + "Realm",
> -"setRealm",
> -"org.apache.catalina.Realm");
> -
> -digester.addObjectCreate(prefix + "Realm/Realm",
> - null, // MUST be specified in the element
> - "className");
> -digester.addSetProperties(prefix + "Realm/Realm");
> -digester.addSetNext(prefix + "Realm/Realm",
> -"addRealm",
> -"org.apache.catalina.Realm");
> -
> -}
> +String pattern = prefix;
>  
> +for (int i = 0; i < MAX_NESTED_REALM_LEVELS; i++) {
>  
> +if (i > 0) {
> +pattern += "/";
> +}
> +pattern += "Realm";
> +
> +digester.addObjectCreate(pattern,
> + null, // MUST be specified in the 
> element,
> + "className");
> +digester.addSetProperties(pattern);
> +if (i == 0) {
> +digester.addSetNext(pattern,
> +"setRealm",
> +"org.apache.catalina.Realm");
> +} else {
> +digester.addSetNext(pattern,
> +"addRealm",
> +"org.apache.catalina.Realm");
> +}
> +}
> +}

The code above might be a bit more straightforward if you did "step 1"
(prefix/Realm + setRealm) above the loop and then loop for each "nested"
realm.

Also, I think most users would expect setting MAX_NESTED_REALM_LEVELS
to, say, 3, would allow Realm/Realm/Realm while I believe the code above
will only allow Realm/Realm. If you want Realm/Realm/Realm, you have to
set MAX_NESTED_REALM_LEVELS to 4 because of the strictly-less-than that
you have in the loop control statement.

Thanks,
-chris



signature.asc
Description: OpenPGP digital signature


Re: Delete catalina.out

2012-11-17 Thread Christopher Schultz
Josh,

On 11/15/12 2:45 PM, Josh Gooding wrote:
> That is the tomcat default log file.  Nothing server wise will happen if
> you delete if that is your concern.  It just removes that particular log
> file.  I believe that you have to either be root and/or have the server
> stopped to remove the file however.

On a *NIX system, neither of the above statements are true: you may
delete the file while a process holds a file handle to the file (the
file will no longer be accessible to any other process -- at least under
its old name) without harm.

I'm not sure what will happen on win32. You'll either fail to delete the
file or get the *NIX-style behavior.

Once deleted, the file will be re-created when Tomcat next launches, as
long as the Tomcat process has "create" privileges for the
CATALINA_BASE/logs directory.

-chris



signature.asc
Description: OpenPGP digital signature


Re: svn commit: r1408739 - in /tomcat/trunk: java/org/apache/catalina/startup/RealmRuleSet.java webapps/docs/config/systemprops.xml

2012-11-17 Thread Christopher Schultz
Mark,

On 11/17/12 8:04 PM, Christopher Schultz wrote:
> Apologies for the late reply. Please see comments inline.
> 
> On 11/13/12 9:17 AM, ma...@apache.org wrote:
>> Author: markt
>> Date: Tue Nov 13 14:17:42 2012
>> New Revision: 1408739
>>
>> URL: http://svn.apache.org/viewvc?rev=1408739&view=rev
>> Log:
>> Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=54141
>> Increase the maximum number of supported nested realm levels from 2 to 3 and 
>> make the maximum configurable via a system property.
>>
>> Modified:
>> tomcat/trunk/java/org/apache/catalina/startup/RealmRuleSet.java
>> tomcat/trunk/webapps/docs/config/systemprops.xml
>>
>> Modified: tomcat/trunk/java/org/apache/catalina/startup/RealmRuleSet.java
>> URL: 
>> http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/startup/RealmRuleSet.java?rev=1408739&r1=1408738&r2=1408739&view=diff
>> ==
>> --- tomcat/trunk/java/org/apache/catalina/startup/RealmRuleSet.java 
>> (original)
>> +++ tomcat/trunk/java/org/apache/catalina/startup/RealmRuleSet.java Tue Nov 
>> 13 14:17:42 2012
>> @@ -34,6 +34,10 @@ import org.apache.tomcat.util.digester.R
>>  public class RealmRuleSet extends RuleSetBase {
>>  
>>  
>> +private static final int MAX_NESTED_REALM_LEVELS = Integer.getInteger(
>> +
>> "org.apache.catalina.startup.RealmRuleSet.MAX_NESTED_REALM_LEVELS",
>> +3).intValue();
>> +
>>  // - Instance 
>> Variables
>>  
>>  
>> @@ -83,23 +87,28 @@ public class RealmRuleSet extends RuleSe
>>  @Override
>>  public void addRuleInstances(Digester digester) {
>>  
>> -digester.addObjectCreate(prefix + "Realm",
>> - null, // MUST be specified in the element,
>> - "className");
>> -digester.addSetProperties(prefix + "Realm");
>> -digester.addSetNext(prefix + "Realm",
>> -"setRealm",
>> -"org.apache.catalina.Realm");
>> -
>> -digester.addObjectCreate(prefix + "Realm/Realm",
>> - null, // MUST be specified in the element
>> - "className");
>> -digester.addSetProperties(prefix + "Realm/Realm");
>> -digester.addSetNext(prefix + "Realm/Realm",
>> -"addRealm",
>> -"org.apache.catalina.Realm");
>> -
>> -}
>> +String pattern = prefix;
>>  
>> +for (int i = 0; i < MAX_NESTED_REALM_LEVELS; i++) {
>>  
>> +if (i > 0) {
>> +pattern += "/";
>> +}
>> +pattern += "Realm";
>> +
>> +digester.addObjectCreate(pattern,
>> + null, // MUST be specified in the 
>> element,
>> + "className");
>> +digester.addSetProperties(pattern);
>> +if (i == 0) {
>> +digester.addSetNext(pattern,
>> +"setRealm",
>> +"org.apache.catalina.Realm");
>> +} else {
>> +digester.addSetNext(pattern,
>> +"addRealm",
>> +"org.apache.catalina.Realm");
>> +}
>> +}
>> +}
> 
> The code above might be a bit more straightforward if you did "step 1"
> (prefix/Realm + setRealm) above the loop and then loop for each "nested"
> realm.
> 
> Also, I think most users would expect setting MAX_NESTED_REALM_LEVELS
> to, say, 3, would allow Realm/Realm/Realm while I believe the code above
> will only allow Realm/Realm. If you want Realm/Realm/Realm, you have to
> set MAX_NESTED_REALM_LEVELS to 4 because of the strictly-less-than that
> you have in the loop control statement.

That last part is, of course, nonsense. That's what I get for reading
code while quite exhausted.

I still think that the code would be more straightforward without a
special-case when i=0.

-chris



signature.asc
Description: OpenPGP digital signature


Bug report for Tomcat 6 [2012/11/18]

2012-11-17 Thread bugzilla
+---+
| Bugzilla Bug ID   |
| +-+
| | Status: UNC=Unconfirmed NEW=New ASS=Assigned|
| | OPN=ReopenedVER=Verified(Skipped Closed/Resolved)   |
| |   +-+
| |   | Severity: BLK=Blocker CRI=Critical  REG=Regression  MAJ=Major   |
| |   |   MIN=Minor   NOR=NormalENH=Enhancement TRV=Trivial |
| |   |   +-+
| |   |   | Date Posted |
| |   |   |  +--+
| |   |   |  | Description  |
| |   |   |  |  |
|41679|New|Enh|2007-02-22|SemaphoreValve should be able to filter on url pat|
|41883|Ass|Enh|2007-03-18|use abstract wrapper instead of plain X509Certific|
|43001|New|Enh|2007-07-30|JspC lacks setMappedFile and setDie for use in Ant|
|43400|New|Enh|2007-09-14|enum support for tag libs |
|43548|Opn|Enh|2007-10-04|xml schema for tomcat-users.xml   |
|43682|New|Enh|2007-10-23|JULI: web-inf/classes/logging.properties to suppor|
|43742|New|Enh|2007-10-30|.tag compiles  performed one at a time -- extremel|
|43979|New|Enh|2007-11-27|Add abstraction for Java and Classfile output |
|44199|New|Enh|2008-01-10|expose current backlog queue size |
|44225|New|Enh|2008-01-14|SSL connector tries to load the private keystore f|
|44294|New|Enh|2008-01-25|Support for EL functions with varargs |
|44645|New|Enh|2008-03-20|[Patch] JNDIRealm - Doesn't support JNDI "java.nam|
|44787|New|Enh|2008-04-09|provide more error context on "java.lang.IllegalSt|
|44818|New|Enh|2008-04-13|tomcat hangs with GET when content-length is defin|
|45014|New|Enh|2008-05-15|Request and Response classes should have wrappers |
|45282|New|Enh|2008-06-25|NioReceiver doesn't close cleanly, leaving sockets|
|45428|New|Enh|2008-07-18|warn if the tomcat stop doesn't complete  |
|45832|New|Enh|2008-09-18|add DIGEST authentication support to Ant tasks|
|45878|New|Enh|2008-09-24|Generated jars do not contain proper manifests or |
|45879|Opn|Enh|2008-09-24|Windows installer fails to install NOTICE and RELE|
|45931|Opn|Enh|2008-10-01|trimSpaces incorrectly modifies output|
|46173|New|Enh|2008-11-09|Small patch for manager app: Setting an optional c|
|46263|New|Enh|2008-11-21|Tomcat reloading of context.xml does not update do|
|46284|New|Enh|2008-11-24|Add flag to DeltaManager that blocks processing cl|
|46350|New|Enh|2008-12-05|Maven repository should contain source bundles|
|46727|New|Enh|2009-02-17|DefaultServlet - serving multiple encodings   |
|46902|New|Enh|2009-03-24|LoginValve to bypass restrictions of j_security_ch|
|47214|New|Enh|2009-05-17|Inner classes that are explicitly referenced - sho|
|47242|New|Enh|2009-05-22|request for AJP command line client   |
|47281|New|Enh|2009-05-28|Efficiency of the JDBCStore   |
|47407|New|Enh|2009-06-23|HttpSessionListener doesn't operate in the session|
|47467|New|Enh|2009-07-02|Deployment of the war file by URL when contextpath|
|47834|New|Enh|2009-09-14|TldConfig throws Exception when exploring unpacked|
|47919|New|Enh|2009-09-30|Log Tomcat & Java environment variables in additio|
|48358|Opn|Enh|2009-12-09|JSP-unloading reloaded|
|48543|New|Enh|2010-01-14|[Patch] More flexibility in specifying -Dcatalina.|
|48672|New|Enh|2010-02-03|Tomcat Virtual Host Manager (/host-manager) have b|
|48674|New|Enh|2010-02-03|Tomcat Virtual Host Manager application doesn't pe|
|48743|New|Enh|2010-02-15|Make the SLEEP variable in catalina.sh settable fr|
|48899|New|Enh|2010-03-12|Guess URI charset should solve lot of problems|
|48922|New|Enh|2010-03-16|org.apache.catalina.connector.Request clone static|
|48928|New|Enh|2010-03-17|An alternative solution to preloading classes when|
|49161|New|Enh|2010-04-21|Unknown Publisher when installing tomcat 6.0.26   |
|49176|Opn|Enh|2010-04-23|Jasper in Dev Mode Is Memory Inefficient  |
|49464|New|Enh|2010-06-18|DefaultServlet and CharacterEncoding  |
|49531|New|Enh|2010-06-30|singlesignon failover not working on DeltaManager/|
|49804|New|Enh|2010-08-23|Allow Embedded.redirectStreams value to be configu|
|49939|New|Enh|2010-09-16|Expose a method via JMX which empties the webapp f|
|49943|New|Enh|2010-09-16|Logging (via juli) does not reread configuration c|
|50285|New|Enh|2010-11-17|Standard HTTP and AJP connectors silently ignore a|
|50288|New|Enh|2010-11-17|Uploading a war file that already exists should au|
|50692|

Bug report for Tomcat 7 [2012/11/18]

2012-11-17 Thread bugzilla
+---+
| Bugzilla Bug ID   |
| +-+
| | Status: UNC=Unconfirmed NEW=New ASS=Assigned|
| | OPN=ReopenedVER=Verified(Skipped Closed/Resolved)   |
| |   +-+
| |   | Severity: BLK=Blocker CRI=Critical  REG=Regression  MAJ=Major   |
| |   |   MIN=Minor   NOR=NormalENH=Enhancement TRV=Trivial |
| |   |   +-+
| |   |   | Date Posted |
| |   |   |  +--+
| |   |   |  | Description  |
| |   |   |  |  |
|16579|New|Enh|2003-01-30|documentation page layout/style breaks wrapping to|
|18500|New|Enh|2003-03-30|Host aliases to match by regular expression   |
|28039|Opn|Enh|2004-03-30|Cluster Support for SingleSignOn  |
|40728|Inf|Enh|2006-10-11|Catalina MBeans use non-serializable classes  |
|40881|Opn|Enh|2006-11-02|Unable to receive message through  TCP channel -> |
|41007|Opn|Enh|2006-11-20|Can't define customized 503 error page|
|43866|New|Enh|2007-11-14|add support for session attribute propagation with|
|43925|Opn|Enh|2007-11-21|org.apache.jasper.runtime.BodyContentImpl causing |
|44216|New|Enh|2008-01-11|Don't reuse session ID even if emptySessionPath=tr|
|48550|Inf|Enh|2010-01-14|Update examples and default server.xml to use UTF-|
|49395|New|Enh|2010-06-06|manager.findLeaks : display the date when the leak|
|49589|New|Enh|2010-07-12|Tag handlers with constant attribute values are al|
|49785|New|Enh|2010-08-19|Enabling TLS for JNDIRealm|
|49821|New|Enh|2010-08-25|Tomcat CLI|
|50019|New|Enh|2010-09-28|Adding JNDI "lookup-name" support In XML and Resou|
|50175|New|Enh|2010-10-28|Enhance memory leak detection by selectively apply|
|50234|New|Enh|2010-11-08|JspC use servlet 3.0 features |
|50504|New|Enh|2010-12-21|Allow setting query string character set trough re|
|50670|New|Enh|2011-01-27|Tribes | RpcChannel | Add option to specify extern|
|51195|New|Enh|2011-05-13|"Find leaks" reports a false positive memory/class|
|51294|Opn|Enh|2011-05-30|Since 7.0.12 do not work option unpackWARs=true fo|
|51423|Inf|Enh|2011-06-23|[Patch] to add a path and a version parameters to |
|51463|New|Enh|2011-07-01|Tomcat.setBaseDir  (package org.apache.catalina.st|
|51496|New|Enh|2011-07-11|NSIS - Warn that duplicate service name will resul|
|51497|New|Enh|2011-07-11|Use canonical IPv6 text representation in logs|
|51526|New|Enh|2011-07-18|Process web application context config with embedd|
|51587|New|Enh|2011-07-29|Implement status and uptime commands  |
|51953|New|Enh|2011-10-04|Proposal: netmask filtering valve and filter  |
|52092|New|Enh|2011-10-26|Please make AsyncFileHandler and OneLineFormatter |
|52235|New|Enh|2011-11-23|Please do a bit of SEO tuning for the web site|
|52236|New|Enh|2011-11-23|Idea: support 'overlays' shaped like Maven overlay|
|52323|New|Enh|2011-12-13|Cobertura test code coverage support for build.xml|
|52381|New|Enh|2011-12-22|Please add OSGi metadata  |
|52448|New|Enh|2012-01-11|Cache jar indexes in WebappClassLoader to speed up|
|52489|New|Enh|2012-01-19|Enhancement request for code signing of war files |
|52558|New|Enh|2012-01-30|CometConnectionManagerValve is adding non-serializ|
|52688|New|Enh|2012-02-16|Add ability to remove old access log files|
|52751|Opn|Enh|2012-02-23|Optimized configuration of the system info display|
|52952|New|Enh|2012-03-20|Improve ExtensionValidator handling for embedded s|
|53085|New|Enh|2012-04-16|[perf] [concurrency] DefaultInstanceManager.annota|
|53387|New|Enh|2012-06-08|SSI: Allow to use $1 to get result of regular expr|
|53411|Opn|Enh|2012-06-13|NullPointerException in org.apache.tomcat.util.buf|
|53469|Inf|Nor|2012-06-26|possible bug in Response.normalize(CharChunk cc)  |
|53492|New|Enh|2012-07-01|Make JspC shell multithreaded |
|53553|New|Enh|2012-07-16|[PATCH] Deploy uploaded WAR with context.xml from |
|53602|New|Enh|2012-07-25|Support for HTTP status code 451  |
|53620|New|Enh|2012-07-30|[juli] delay opening a file until something gets l|
|53665|New|Enh|2012-08-06|Minor JNDI Howto document enhancement concerning m|
|53776|New|Enh|2012-08-24|Multitenancy support for JDBCRealm|
|53777|New|Enh|2012-08-24|Ability to bundle JAAS Configuration in Webappp   |
|53869|Inf|Enh|2012-09-13|Performance tuning solution to resolve too many ca|
|54013|

Bug report for Taglibs [2012/11/18]

2012-11-17 Thread bugzilla
+---+
| Bugzilla Bug ID   |
| +-+
| | Status: UNC=Unconfirmed NEW=New ASS=Assigned|
| | OPN=ReopenedVER=Verified(Skipped Closed/Resolved)   |
| |   +-+
| |   | Severity: BLK=Blocker CRI=Critical  REG=Regression  MAJ=Major   |
| |   |   MIN=Minor   NOR=NormalENH=Enhancement TRV=Trivial |
| |   |   +-+
| |   |   | Date Posted |
| |   |   |  +--+
| |   |   |  | Description  |
| |   |   |  |  |
|38193|Ass|Enh|2006-01-09|[RDC] BuiltIn Grammar support for Field   |
|38600|Ass|Enh|2006-02-10|[RDC] Enable RDCs to be used in X+V markup (X+RDC)|
|42413|New|Enh|2007-05-14|[PATCH] Log Taglib enhancements   |
|46052|New|Nor|2008-10-21|SetLocaleSupport is slow to initialize when many l|
|48333|New|Enh|2009-12-02|TLD generator |
+-+---+---+--+--+
| Total5 bugs   |
+---+

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



Bug report for Tomcat Connectors [2012/11/18]

2012-11-17 Thread bugzilla
+---+
| Bugzilla Bug ID   |
| +-+
| | Status: UNC=Unconfirmed NEW=New ASS=Assigned|
| | OPN=ReopenedVER=Verified(Skipped Closed/Resolved)   |
| |   +-+
| |   | Severity: BLK=Blocker CRI=Critical  REG=Regression  MAJ=Major   |
| |   |   MIN=Minor   NOR=NormalENH=Enhancement TRV=Trivial |
| |   |   +-+
| |   |   | Date Posted |
| |   |   |  +--+
| |   |   |  | Description  |
| |   |   |  |  |
|34526|Opn|Nor|2005-04-19|Truncated content in decompressed requests from mo|
|35959|Opn|Enh|2005-08-01|mod_jk not independant of UseCanonicalName|
|43303|New|Enh|2007-09-04|Versioning under Windows not reported by many conn|
|43968|Inf|Enh|2007-11-26|[patch] support ipv6 with mod_jk  |
|44290|Inf|Nor|2008-01-24|mod_jk/1.2.26: retry is not useful for an importan|
|44349|Inf|Maj|2008-02-04|mod_jk/1.2.26 module does not read worker.status.s|
|44379|New|Enh|2008-02-07|convert the output of strftime into UTF-8 |
|44454|New|Nor|2008-02-19|busy count reported in mod_jk inflated, causes inc|
|44571|New|Enh|2008-03-10|Limits busy per worker to a threshold |
|45063|New|Nor|2008-05-22|JK-1.2.26 IIS ISAPI filter issue when running diff|
|45313|New|Nor|2008-06-30|mod_jk 1.2.26 & apache 2.2.9 static compiled on so|
|46337|New|Nor|2008-12-04|real worker name is wrong |
|46676|New|Enh|2009-02-09|Configurable test request for Watchdog thread |
|46767|New|Enh|2009-02-25|mod_jk to send DECLINED in case no fail-over tomca|
|47327|New|Enh|2009-06-07|remote_user not logged in apache logfile  |
|47617|Inf|Enh|2009-07-31|include time spent doing ajp_get_endpoint() in err|
|47678|New|Cri|2009-08-11|Unable to allocate shared memory when using isapi_|
|47714|New|Cri|2009-08-20|Reponse mixed between users   |
|47750|New|Maj|2009-08-27|Loss of worker settings when changing via jkstatus|
|47795|New|Maj|2009-09-07|service sticky_session not being set correctly wit|
|47840|Inf|Min|2009-09-14|A broken worker name is written in the log file.  |
|48191|New|Maj|2009-11-13|Problem with mod_jk 1.2.28 - Can not render up the|
|48460|New|Nor|2009-12-30|mod_proxy_ajp document has three misleading portio|
|48490|New|Nor|2010-01-05|Changing a node to stopped in uriworkermap.propert|
|48513|New|Enh|2010-01-09|IIS Quick setup instructions  |
|48564|New|Nor|2010-01-18|Unable to turn off retries for LB worker  |
|48830|New|Nor|2010-03-01|IIS shutdown blocked in endpoint service when serv|
|48891|Opn|Enh|2010-03-11|Missing EOL-style settings in tomcat/jk/trunk |
|49035|New|Maj|2010-04-01|data lost when post a multipart/form-data form|
|49063|New|Enh|2010-04-07|Please add JkStripSession status in jk-status work|
|49135|New|Enh|2010-04-16|SPDY Connector for The Tomcat |
|49469|New|Enh|2010-06-19|Workers status page has negative number of connect|
|49732|Opn|Nor|2010-08-10|reply_timeout can't wait forever. |
|49822|New|Enh|2010-08-25|Add hash lb worker method |
|49903|New|Enh|2010-09-09|Make workers file reloadable  |
|50186|New|Nor|2010-10-31|Wrong documentation of connection_pool_timeout / c|
|52334|New|Maj|2011-12-14|recover_time is not properly used |
|52483|New|Enh|2012-01-18|Print JkOptions's options in log file and jkstatus|
|52651|New|Nor|2012-02-13|JKSHMFile size limitation |
|53324|Opn|Nor|2012-05-30|Starting with mod_jk 1.2.35 I cannot modify worker|
|53542|New|Min|2012-07-13|Spelling mistake on 503 service unavailable page  |
|53762|New|Nor|2012-08-22|JK status manager: mass nodes handling doesn't wor|
|53883|New|Maj|2012-09-17|isapi_redirect v 1.2.37 crashes w3wp.exe  on the p|
|53977|New|Maj|2012-10-07|32bits isapi connector cannot work in wow64 mode  |
|54027|New|Cri|2012-10-18|isapi send request to outside address instead of i|
|54112|Opn|Blk|2012-11-07|ISAPI redirector not working when IIS recycles|
|54117|New|Maj|2012-11-08|access violation exception in isapi_redirect.dll  |
+-+---+---+--+--+
| Total   47 bugs   |
+---+

-
To unsubscribe

Bug report for Tomcat Modules [2012/11/18]

2012-11-17 Thread bugzilla
+---+
| Bugzilla Bug ID   |
| +-+
| | Status: UNC=Unconfirmed NEW=New ASS=Assigned|
| | OPN=ReopenedVER=Verified(Skipped Closed/Resolved)   |
| |   +-+
| |   | Severity: BLK=Blocker CRI=Critical  REG=Regression  MAJ=Major   |
| |   |   MIN=Minor   NOR=NormalENH=Enhancement TRV=Trivial |
| |   |   +-+
| |   |   | Date Posted |
| |   |   |  +--+
| |   |   |  | Description  |
| |   |   |  |  |
|48240|New|Nor|2009-11-19|Tomcat-Lite missing @Override markers |
|48268|New|Nor|2009-11-23|Patch to fix generics in tomcat-lite  |
|48861|New|Nor|2010-03-04|Files without AL headers  |
|49685|New|Nor|2010-08-02|Unsafe synchronization in class ManagedBean   |
|49686|New|Nor|2010-08-02|Using an instance lock to protect static shared da|
|50571|Inf|Nor|2011-01-11|Tomcat 7 JDBC connection pool exception enhancemen|
|51595|Inf|Nor|2011-08-01|org.apache.tomcat.jdbc.pool.jmx.ConnectionPool sho|
|51879|Inf|Enh|2011-09-22|Improve access to Native Connection Methods   |
|52024|Inf|Enh|2011-10-13|Custom interceptor to support automatic failover o|
|53088|Opn|Min|2012-04-17|Give PoolCleaner TimerTask a better name  |
|53198|New|Cri|2012-05-07|'driverClassName' Data Source Property Being Manda|
|53199|Inf|Enh|2012-05-07|Refactor ConnectionPool to use ScheduledExecutorSe|
|53200|New|Enh|2012-05-07|Be able to use SlowQueryReport without reporting f|
|53565|New|Min|2012-07-18|deprecated|
|53770|New|Enh|2012-08-23|tomcat-pool: always log validation query syntax er|
|53853|New|Nor|2012-09-11|Can tomcat-jdbc consider Thread#getContextClassLoa|
|53905|New|Nor|2012-09-19|Connection pool not reusing connections.  |
|53968|New|Nor|2012-10-04|Database password is exposed via JMX in getDbPrope|
|54116|New|Cri|2012-11-07|Deadlocks with mysql driver   |
|54150|New|Nor|2012-11-15|SlowQueryReportJmx mbean deregistration failure du|
+-+---+---+--+--+
| Total   20 bugs   |
+---+

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



Bug report for Tomcat Native [2012/11/18]

2012-11-17 Thread bugzilla
+---+
| Bugzilla Bug ID   |
| +-+
| | Status: UNC=Unconfirmed NEW=New ASS=Assigned|
| | OPN=ReopenedVER=Verified(Skipped Closed/Resolved)   |
| |   +-+
| |   | Severity: BLK=Blocker CRI=Critical  REG=Regression  MAJ=Major   |
| |   |   MIN=Minor   NOR=NormalENH=Enhancement TRV=Trivial |
| |   |   +-+
| |   |   | Date Posted |
| |   |   |  +--+
| |   |   |  | Description  |
| |   |   |  |  |
|45392|New|Nor|2008-07-14|No OCSP support for client SSL verification   |
|46179|Opn|Maj|2008-11-10|apr ssl client authentication |
|48655|Inf|Nor|2010-02-02|Active multipart downloads prevent tomcat shutdown|
|49038|Inf|Nor|2010-04-02|Crash in tcnative |
|51655|New|Nor|2011-08-12|Index page does not say what native does  |
|51813|New|Cri|2011-09-14|Tomcat randomly crashes with [libtcnative-1.so.1+0|
|52153|New|Maj|2011-11-08|periodic JVM crash (access violation) on buffer fl|
|52231|New|Nor|2011-11-23|Ant Tasks need to reflect changes in manager comma|
|52319|New|Maj|2011-12-12|Tomcat 6 crashes with [libapr-1.so.0+0x196da]  sig|
|52627|New|Min|2012-02-08|Segmentation fault in org.apache.tomcat.jni.File.i|
|52856|New|Maj|2012-03-08|High CPU load when the underlying socket error in |
|53110|New|Cri|2012-04-20|Access Violation Error while creating SHM |
|53605|New|Nor|2012-07-26|use tcnative-1.1.24 Tomcat shutdown still crash   |
|53847|Inf|Nor|2012-09-10|High CPU usage in tomcat native 1.22+ |
|53937|New|Reg|2012-09-26|Double call to apr_pool_destroy() if OCSP checking|
|53940|New|Enh|2012-09-27|Added support for new CRL loading after expiration|
|53952|New|Nor|2012-10-02|Add support for TLS 1.1 and 1.2   |
|54064|New|Nor|2012-10-29|tomcat native randomly very slow with high CPU usa|
|54085|New|Nor|2012-11-01|ssl_socket_recv sometimes loops infinitely with no|
+-+---+---+--+--+
| Total   19 bugs   |
+---+

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