svn commit: r1798419 - /tomcat/trunk/java/org/apache/catalina/startup/Tomcat.java

2017-06-12 Thread violetagg
Author: violetagg
Date: Mon Jun 12 07:03:48 2017
New Revision: 1798419

URL: http://svn.apache.org/viewvc?rev=1798419&view=rev
Log:
Remove trailing space

Modified:
tomcat/trunk/java/org/apache/catalina/startup/Tomcat.java

Modified: tomcat/trunk/java/org/apache/catalina/startup/Tomcat.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/startup/Tomcat.java?rev=1798419&r1=1798418&r2=1798419&view=diff
==
--- tomcat/trunk/java/org/apache/catalina/startup/Tomcat.java (original)
+++ tomcat/trunk/java/org/apache/catalina/startup/Tomcat.java Mon Jun 12 
07:03:48 2017
@@ -87,7 +87,7 @@ import org.apache.tomcat.util.descriptor
  * Requirements:
  * 
  *   all tomcat classes and possibly servlets are in the classpath.
- *   (for example all is in one big jar, or in eclipse CP, or in 
+ *   (for example all is in one big jar, or in eclipse CP, or in
  *any other combination)
  *
  *   we need one temporary directory for work files



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



svn commit: r1798420 - in /tomcat/tc8.5.x/trunk: ./ java/org/apache/catalina/startup/Tomcat.java

2017-06-12 Thread violetagg
Author: violetagg
Date: Mon Jun 12 07:05:16 2017
New Revision: 1798420

URL: http://svn.apache.org/viewvc?rev=1798420&view=rev
Log:
Remove trailing space

Modified:
tomcat/tc8.5.x/trunk/   (props changed)
tomcat/tc8.5.x/trunk/java/org/apache/catalina/startup/Tomcat.java

Propchange: tomcat/tc8.5.x/trunk/
--
--- svn:mergeinfo (original)
+++ svn:mergeinfo Mon Jun 12 07:05:16 2017
@@ -1 +1 @@
-/tomcat/trunk:1734785,1734799,1734845,1734928,1735041,1735044,1735480,1735577,1735597,1735599-1735600,1735615,1736145,1736162,1736209,1736280,1736297,1736299,1736489,1736646,1736703,1736836,1736849,1737104-1737105,1737112,1737117,1737119-1737120,1737155,1737157,1737192,1737280,1737339,1737632,1737664,1737715,1737748,1737785,1737834,1737860,1737903,1737959,1738005,1738007,1738014-1738015,1738018,1738022,1738039,1738043,1738059-1738060,1738147,1738149,1738174-1738175,1738261,1738589,1738623-1738625,1738643,1738816,1738850,1738855,1738946-1738948,1738953-1738954,1738979,1738982,1739079-1739081,1739087,1739113,1739153,1739172,1739176,1739191,1739474,1739726,1739762,1739775,1739814,1739817-1739818,1739975,1740131,1740324,1740465,1740495,1740508-1740509,1740520,1740535,1740707,1740803,1740810,1740969,1740980,1740991,1740997,1741015,1741033,1741036,1741058,1741060,1741080,1741147,1741159,1741164,1741173,1741181,1741190,1741197,1741202,1741208,1741213,1741221,1741225,1741232,1741409,1741501
 
,1741677,1741892,1741896,1741984,1742023,1742042,1742071,1742090,1742093,1742101,1742105,1742111,1742139,1742146,1742148,1742166,1742181,1742184,1742187,1742246,1742248-1742251,1742263-1742264,1742268,1742276,1742369,1742387,1742448,1742509-1742512,1742917,1742919,1742933,1742975-1742976,1742984,1742986,1743019,1743115,1743117,1743124-1743125,1743134,1743425,1743554,1743679,1743696-1743698,1743700-1743701,1744058,1744064-1744065,1744125,1744194,1744229,1744270,1744323,1744432,1744684,1744697,1744705,1744713,1744760,1744786,1745083,1745142-1745143,1745145,1745177,1745179-1745180,1745227,1745248,1745254,1745337,1745467,1745473,1745535,1745576,1745735,1745744,1746304,1746306-1746307,1746319,1746327,1746338,1746340-1746341,1746344,1746427,1746441,1746473,1746490,1746492,1746495-1746496,1746499-1746501,1746503-1746507,1746509,1746549,1746551,1746554,1746556,1746558,1746584,1746620,1746649,1746724,1746939,1746989,1747014,1747028,1747035,1747210,1747225,1747234,1747253,1747404,1747506,1747
 
536,1747924,1747980,1747993,1748001,1748253,1748452,1748547,1748629,1748676,1748715,1749287,1749296,1749328,1749373,1749465,1749506,1749508,1749665-1749666,1749763,1749865-1749866,1749898,1749978,1749980,1750011,1750015,1750056,1750480,1750617,1750634,1750692,1750697,1750700,1750703,1750707,1750714,1750718,1750723,1750774,1750899,1750975,1750995,1751061,1751097,1751173,1751438,1751447,1751463,1751702,1752212,1752737,1752745,1753078,1753080,1753358,1753363,1754111,1754140-1754141,1754281,1754310,1754445,1754467,1754494,1754496,1754528,1754532-1754533,1754613,1754714,1754874,1754941,1754944,1754950-1754951,1755005,1755007,1755009,1755132,1755180-1755181,1755185,1755190,1755204-1755206,1755208,1755214,1755224,1755227,1755230,1755629,1755646-1755647,1755650,1755653,1755675,1755680,1755683,1755693,1755717,1755731-1755737,1755812,1755828,1755884,1755890,1755918-1755919,1755942,1755958,1755960,1755970,1755993,1756013,1756019,1756039,1756056,1756083-1756114,1756175,1756288-1756289,1756408-1
 
756410,1756778,1756798,1756878,1756898,1756939,1757123-1757124,1757126,1757128,1757132-1757133,1757136,1757145,1757167-1757168,1757175,1757180,1757182,1757195,1757271,1757278,1757347,1757353-1757354,1757363,1757374,1757399,1757406,1757408,1757485,1757495,1757499,1757527,1757578,1757684,1757722,1757727,1757790,1757799,1757813,1757853,1757883,1757903,1757976,1757997,1758000,1758058,1758072-1758075,1758078-1758079,1758223,1758257,1758261,1758276,1758292,1758369,1758378-1758383,1758421,1758423,1758425-1758427,1758430,1758443,1758448,1758459,1758483,1758486-1758487,1758499,1758525,1758556,1758580,1758582,1758584,1758588,1758842,1759019,1759212,1759224,1759227,1759252,1759274,1759513-1759516,1759611,1759757,1759785-1759790,1760005,1760022,1760109-1760110,1760135,1760200-1760201,1760227,1760300,1760397,1760446,1760454,1760640,1760648,1761057,1761422,1761491,1761498,1761500-1761501,1761550,1761553,1761572,1761574,1761625-1761626,1761628,1761682,1761740,1761752,1762051-1762053,1762123,176216
 
8,1762172,1762182,1762201-1762202,1762204,1762208,1762288,1762296,1762324,1762348,1762353,1762362,1762374,1762492,1762503,1762505,1762541,1762608,1762710,1762753,1762766,1762769,1762944,1762947,1762953,1763167,1763179,1763232,1763259,1763271-1763272,1763276-1763277,1763319-1763320,1763370,1763372,1763375,1763377,1763393,1763412,1763430,1763450,1763462,1763505,1763511-1763512,1763516,1763518,1763520,1763529,1763559,1763565,1763568,1763574,1763619,1763634-1763635,1763718,1763786,1763798-1763799,1763810,1763813,1763815

svn commit: r1798421 - in /tomcat/tc8.0.x/trunk: ./ java/org/apache/catalina/startup/Tomcat.java

2017-06-12 Thread violetagg
Author: violetagg
Date: Mon Jun 12 07:06:22 2017
New Revision: 1798421

URL: http://svn.apache.org/viewvc?rev=1798421&view=rev
Log:
Remove trailing space

Modified:
tomcat/tc8.0.x/trunk/   (props changed)
tomcat/tc8.0.x/trunk/java/org/apache/catalina/startup/Tomcat.java

Propchange: tomcat/tc8.0.x/trunk/
--
--- svn:mergeinfo (original)
+++ svn:mergeinfo Mon Jun 12 07:06:22 2017
@@ -1,2 +1,2 @@
 
/tomcat/tc8.5.x/trunk:1735042,1737966,1743139-1743140,1744151,1747537,1747925,1748002,1754614,1754643,1762124,1762183,1762203,1763792,1772948,1777014,1779719,1782037,1782240,1782386-1782387,1785669,1786845,1788249,1788324,1788905,1789216,1789335,1791528,1791558,1796697-1796698,1797521
-/tomcat/trunk:1636524,1637156,1637176,1637188,1637331,1637684,1637695,1637890,1637892,1638720-1638725,1639653,1640010,1640083-1640084,1640088,1640275,1640322,1640347,1640361,1640365,1640403,1640410,1640652,1640655-1640658,1640688,1640700-1640883,1640903,1640976,1640978,1641000,1641026,1641038-1641039,1641051-1641052,1641058,1641064,1641300,1641369,1641374,1641380,1641486,1641634,1641656-1641692,1641704,1641707-1641718,1641720-1641722,1641735,1641981,1642233,1642280,1642554,1642564,1642595,1642606,1642668,1642679,1642697,1642699,1642766,1643002,1643045,1643054-1643055,1643066,1643121,1643128,1643206,1643209-1643210,1643216,1643249,1643270,1643283,1643309-1643310,1643323,1643365-1643366,1643370-1643371,1643465,1643474,1643536,1643570,1643634,1643649,1643651,1643654,1643675,1643731,1643733-1643734,1643761,1643766,1643814,1643937,1643963,1644017,1644169,1644201-1644203,1644321,1644323,1644516,1644523,1644529,1644535,1644730,1644768,1644784-1644785,1644790,1644793,1644815,1644884,1644886
 
,1644890,1644892,1644910,1644924,1644929-1644930,1644935,1644989,1645011,1645247,1645355,1645357-1645358,1645455,1645465,1645469,1645471,1645473,1645475,1645486-1645488,1645626,1645641,1645685,1645743,1645763,1645951-1645953,1645955,1645993,1646098-1646106,1646178,1646220,1646302,1646304,1646420,1646470-1646471,1646476,1646559,1646717-1646723,1646773,1647026,1647042,1647530,1647655,1648304,1648815,1648907,1649973,1650081,1650365,1651116,1651120,1651280,1651470,1652938,1652970,1653041,1653471,1653550,1653574,1653797,1653815-1653816,1653819,1653840,1653857,1653888,1653972,1654013,1654030,1654050,1654123,1654148,1654159,1654513,1654515,1654517,1654522,1654524,1654725,1654735,1654766,1654785,1654851-1654852,1654978,1655122-1655124,1655126-1655127,1655129-1655130,1655132-1655133,1655312,1655351,1655438,1655441,1655454,168,1656087,1656299,1656319,1656331,1656345,1656350,1656590,1656648-1656650,1656657,1657041,1657054,1657374,1657492,1657510,1657565,1657580,1657584,1657586,1657589,1657
 
592,1657607,1657609,1657682,1657907,1658207,1658734,1658781,1658790,1658799,1658802,1658804,1658833,1658840,1658966,1659043,1659053,1659059,1659174,1659184,1659188-1659189,1659216,1659263,1659293,1659304,1659306-1659307,1659382,1659384,1659428,1659471,1659486,1659505,1659516,1659521,1659524,1659559,1659562,1659803,1659806,1659814,1659833,1659862,1659905,1659919,1659948,1659967,1659983-1659984,1660060,1660074,1660077,1660133,1660168,1660331-1660332,1660353,1660358,1660924,1661386,1661770,1661867,1661972,1661990,1662200,1662308-1662309,1662548,1662614,1662696,1662736,1662985,1662988-1662989,1663264,1663277,1663298,1663534,1663562,1663676,1663715,1663754,1663768,1663772,1663781,1663893,1663995,1664143,1664163,1664174,1664301,1664317,1664347,1664657,1664659,1664710,1664863-1664864,1664866,1665085,1665292,1665559,1665653,1665661,1665672,1665694,1665697,1665736,1665779,1665976-1665977,1665980-1665981,1665985-1665986,1665989,1665998,1666004,1666008,1666013,1666017,1666024,1666116,1666386-1
 
666387,1666494,1666496,1666552,1666569,1666579,137,149,1666757,1666966,1666972,1666985,1666995,1666997,1667292,1667402,1667406,1667546,1667615,1667630,1667636,1667688,1667764,1667871,1668026,1668135,1668193,1668593,1668596,1668630,1668639,1668843,1669353,1669370,1669451,1669800,1669838,1669876,1669882,1670394,1670433,1670591,1670598-1670600,1670610,1670631,1670719,1670724,1670726,1670730,1670940,1671112,1672272,1672284,1673754,1674294,1675461,1675486,1675594,1675830,1676231,1676250-1676251,1676364,1676381,1676393,1676479,1676525,1676552,1676615,1676630,1676634,1676721,1676926,1676943,1677140,1677802,1678011,1678162,1678174,1678339,1678426-1678427,1678694,1678701,1679534,1679708,1679710,1679716,1680034,1680246,1681056,1681123,1681138,1681280,1681283,1681286,1681450,1681697,1681699,1681701,1681729,1681770,1681779,1681793,1681807,1681837-1681838,1681854,1681862,1681958,1682028,1682033,1682311,1682315,1682317,1682320,1682324,1682330,1682842,1684172,1684366,1684383,1684526-168452
 
7,1684549-1684550,1685556,1685591,1685739,1685744,1685772,1685816,1685826,1685891,1687242,1687261,1687268,1687340,1687544,1687551,1688563,1688841,1688878,165,1688896,1688901,1689345-1689346,1689357,1689656,1689675-1689677,168

[GUMP@vmgump-vm3]: Project tomcat-tc8.0.x-validate (in module tomcat-8.0.x) failed

2017-06-12 Thread Bill Barker
To whom it may engage...

This is an automated request, but not an unsolicited one. For 
more information please visit http://gump.apache.org/nagged.html, 
and/or contact the folk at gene...@gump.apache.org.

Project tomcat-tc8.0.x-validate has an issue affecting its community 
integration.
This issue affects 1 projects.
The current state of this project is 'Failed', with reason 'Build Failed'.
For reference only, the following projects are affected by this:
- tomcat-tc8.0.x-validate :  Tomcat 8.x, a web server implementing the Java 
Servlet 3.1,
...


Full details are available at:
http://vmgump-vm3.apache.org/tomcat-8.0.x/tomcat-tc8.0.x-validate/index.html

That said, some information snippets are provided here.

The following annotations (debug/informational/warning/error messages) were 
provided:
 -DEBUG- Dependency on checkstyle exists, no need to add for property 
checkstyle.jar.
 -INFO- Failed with reason build failed



The following work was performed:
http://vmgump-vm3.apache.org/tomcat-8.0.x/tomcat-tc8.0.x-validate/gump_work/build_tomcat-8.0.x_tomcat-tc8.0.x-validate.html
Work Name: build_tomcat-8.0.x_tomcat-tc8.0.x-validate (Type: Build)
Work ended in a state of : Failed
Elapsed: 21 secs
Command Line: /usr/lib/jvm/java-8-oracle/bin/java -Djava.awt.headless=true 
-Dbuild.sysclasspath=only org.apache.tools.ant.Main 
-Dgump.merge=/srv/gump/public/gump/work/merge.xml 
-Dbase.path=/srv/gump/public/workspace/tomcat-8.0.x/tomcat-build-libs 
-Dcheckstyle.jar=/srv/gump/public/workspace/checkstyle/target/checkstyle-7.8.2-SNAPSHOT.jar
 -Dexecute.validate=true validate 
[Working Directory: /srv/gump/public/workspace/tomcat-8.0.x]
CLASSPATH: 
/usr/lib/jvm/java-8-oracle/lib/tools.jar:/srv/gump/public/workspace/ant/dist/lib/ant.jar:/srv/gump/public/workspace/ant/dist/lib/ant-launcher.jar:/srv/gump/public/workspace/ant/dist/lib/ant-jmf.jar:/srv/gump/public/workspace/ant/dist/lib/ant-junit.jar:/srv/gump/public/workspace/ant/dist/lib/ant-junit4.jar:/srv/gump/public/workspace/ant/dist/lib/ant-swing.jar:/srv/gump/public/workspace/ant/dist/lib/ant-apache-resolver.jar:/srv/gump/public/workspace/ant/dist/lib/ant-apache-xalan2.jar:/srv/gump/public/workspace/xml-commons/java/build/resolver.jar:/srv/gump/public/workspace/checkstyle/target/checkstyle-7.8.2-SNAPSHOT.jar:/srv/gump/packages/antlr/antlr-3.1.3.jar:/srv/gump/public/workspace/apache-commons/beanutils/dist/commons-beanutils-20170612.jar:/srv/gump/packages/commons-collections3/commons-collections-3.2.1.jar:/srv/gump/public/workspace/commons-cli/target/commons-cli-1.5-SNAPSHOT.jar:/srv/gump/public/workspace/commons-lang-trunk/target/commons-lang3-3.6-SNAPSHOT.jar:/srv
 
/gump/public/workspace/apache-commons/logging/target/commons-logging-20170612.jar:/srv/gump/public/workspace/apache-commons/logging/target/commons-logging-api-20170612.jar:/srv/gump/public/workspace/google-guava/guava/target/guava-23.0-SNAPSHOT.jar
-
Buildfile: /srv/gump/public/workspace/tomcat-8.0.x/build.xml

build-prepare:
   [delete] Deleting directory 
/srv/gump/public/workspace/tomcat-8.0.x/output/build/temp
[mkdir] Created dir: 
/srv/gump/public/workspace/tomcat-8.0.x/output/build/temp

compile-prepare:

download-validate:

testexist:
 [echo] Testing  for 
/srv/gump/public/workspace/checkstyle/target/checkstyle-7.8.2-SNAPSHOT.jar

setproxy:

downloadfile:

validate:
[mkdir] Created dir: 
/srv/gump/public/workspace/tomcat-8.0.x/output/res/checkstyle
[checkstyle] Running Checkstyle 7.8.2-SNAPSHOT on 3028 files
[checkstyle] [ERROR] 
/srv/gump/public/workspace/tomcat-8.0.x/java/org/apache/catalina/startup/Tomcat.java:90:
 Line matches the illegal pattern '\s+$'. [RegexpSingleline]

BUILD FAILED
/srv/gump/public/workspace/tomcat-8.0.x/build.xml:552: Got 1 errors and 0 
warnings.

Total time: 21 seconds
-

To subscribe to this information via syndicated feeds:
- RSS: http://vmgump-vm3.apache.org/tomcat-8.0.x/tomcat-tc8.0.x-validate/rss.xml
- Atom: 
http://vmgump-vm3.apache.org/tomcat-8.0.x/tomcat-tc8.0.x-validate/atom.xml

== Gump Tracking Only ===
Produced by Apache Gump(TM) version 2.3.
Gump Run 20170612060006, vmgump-vm3.apache.org:vmgump:20170612060006
Gump E-mail Identifier (unique within run) #1.

--
Apache Gump
http://gump.apache.org/ [Instance: vmgump-vm3]

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



[GUMP@vmgump-vm3]: Project tomcat-trunk-validate (in module tomcat-trunk) failed

2017-06-12 Thread Bill Barker
To whom it may engage...

This is an automated request, but not an unsolicited one. For 
more information please visit http://gump.apache.org/nagged.html, 
and/or contact the folk at gene...@gump.apache.org.

Project tomcat-trunk-validate has an issue affecting its community integration.
This issue affects 1 projects.
The current state of this project is 'Failed', with reason 'Build Failed'.
For reference only, the following projects are affected by this:
- tomcat-trunk-validate :  Tomcat 9.x, a web server implementing the Java 
Servlet 4.0,
...


Full details are available at:
http://vmgump-vm3.apache.org/tomcat-trunk/tomcat-trunk-validate/index.html

That said, some information snippets are provided here.

The following annotations (debug/informational/warning/error messages) were 
provided:
 -DEBUG- Dependency on checkstyle exists, no need to add for property 
checkstyle.jar.
 -INFO- Failed with reason build failed



The following work was performed:
http://vmgump-vm3.apache.org/tomcat-trunk/tomcat-trunk-validate/gump_work/build_tomcat-trunk_tomcat-trunk-validate.html
Work Name: build_tomcat-trunk_tomcat-trunk-validate (Type: Build)
Work ended in a state of : Failed
Elapsed: 21 secs
Command Line: /usr/lib/jvm/java-8-oracle/bin/java -Djava.awt.headless=true 
-Dbuild.sysclasspath=only org.apache.tools.ant.Main 
-Dgump.merge=/srv/gump/public/gump/work/merge.xml 
-Dbase.path=/srv/gump/public/workspace/tomcat-trunk/tomcat-build-libs 
-Dcheckstyle.jar=/srv/gump/public/workspace/checkstyle/target/checkstyle-7.8.2-SNAPSHOT.jar
 -Dexecute.validate=true validate 
[Working Directory: /srv/gump/public/workspace/tomcat-trunk]
CLASSPATH: 
/usr/lib/jvm/java-8-oracle/lib/tools.jar:/srv/gump/public/workspace/ant/dist/lib/ant.jar:/srv/gump/public/workspace/ant/dist/lib/ant-launcher.jar:/srv/gump/public/workspace/ant/dist/lib/ant-jmf.jar:/srv/gump/public/workspace/ant/dist/lib/ant-junit.jar:/srv/gump/public/workspace/ant/dist/lib/ant-junit4.jar:/srv/gump/public/workspace/ant/dist/lib/ant-swing.jar:/srv/gump/public/workspace/ant/dist/lib/ant-apache-resolver.jar:/srv/gump/public/workspace/ant/dist/lib/ant-apache-xalan2.jar:/srv/gump/public/workspace/xml-commons/java/build/resolver.jar:/srv/gump/public/workspace/checkstyle/target/checkstyle-7.8.2-SNAPSHOT.jar:/srv/gump/packages/antlr/antlr-3.1.3.jar:/srv/gump/public/workspace/apache-commons/beanutils/dist/commons-beanutils-20170612.jar:/srv/gump/packages/commons-collections3/commons-collections-3.2.1.jar:/srv/gump/public/workspace/commons-cli/target/commons-cli-1.5-SNAPSHOT.jar:/srv/gump/public/workspace/commons-lang-trunk/target/commons-lang3-3.6-SNAPSHOT.jar:/srv
 
/gump/public/workspace/apache-commons/logging/target/commons-logging-20170612.jar:/srv/gump/public/workspace/apache-commons/logging/target/commons-logging-api-20170612.jar:/srv/gump/public/workspace/google-guava/guava/target/guava-23.0-SNAPSHOT.jar
-
Buildfile: /srv/gump/public/workspace/tomcat-trunk/build.xml

build-prepare:
   [delete] Deleting directory 
/srv/gump/public/workspace/tomcat-trunk/output/build/temp
[mkdir] Created dir: 
/srv/gump/public/workspace/tomcat-trunk/output/build/temp

compile-prepare:

download-validate:

testexist:
 [echo] Testing  for 
/srv/gump/public/workspace/checkstyle/target/checkstyle-7.8.2-SNAPSHOT.jar

setproxy:

downloadfile:

validate:
[mkdir] Created dir: 
/srv/gump/public/workspace/tomcat-trunk/output/res/checkstyle
[checkstyle] Running Checkstyle 7.8.2-SNAPSHOT on 3135 files
[checkstyle] [ERROR] 
/srv/gump/public/workspace/tomcat-trunk/java/org/apache/catalina/startup/Tomcat.java:90:
 Line matches the illegal pattern '\s+$'. [RegexpSingleline]

BUILD FAILED
/srv/gump/public/workspace/tomcat-trunk/build.xml:539: Got 1 errors and 0 
warnings.

Total time: 21 seconds
-

To subscribe to this information via syndicated feeds:
- RSS: http://vmgump-vm3.apache.org/tomcat-trunk/tomcat-trunk-validate/rss.xml
- Atom: http://vmgump-vm3.apache.org/tomcat-trunk/tomcat-trunk-validate/atom.xml

== Gump Tracking Only ===
Produced by Apache Gump(TM) version 2.3.
Gump Run 20170612060006, vmgump-vm3.apache.org:vmgump:20170612060006
Gump E-mail Identifier (unique within run) #2.

--
Apache Gump
http://gump.apache.org/ [Instance: vmgump-vm3]

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



Re: Things that we can do to increase contributor involvement?

2017-06-12 Thread Violeta Georgieva
2017-06-11 23:57 GMT+03:00 Coty Sutherland :
>
> On Sun, Jun 11, 2017 at 4:20 PM, Violeta Georgieva 
wrote:
> > Hi Coty,
> >
> > 2017-06-02 17:50 GMT+03:00 Coty Sutherland :
> >>
> >> On Fri, Jun 2, 2017 at 10:27 AM, Violeta Georgieva <
violet...@apache.org>
> > wrote:
> >> > Hi,
> >> >
> >> > 2017-05-31 6:07 GMT+03:00 Coty Sutherland :
> >> >>
> >> >> Hi all,
> >> >>
> >> >> I've been thinking about things that we could do for Tomcat to help
> >> >> bring in new contributors and to be more appealing to new
developers.
> >> >> Right now we have http://tomcat.apache.org/getinvolved.html which
has
> >> >> a few bullet points and links to documentation, which is a bit
verbose,
> >> >> about how to contribute to an Apache project. We also have the wiki
> >> >> (https://wiki.apache.org/tomcat/FrontPage), which mentions nothing
> >> >> about contributing. Bugzilla is a bit daunting for newcomers
(thought
> >> >> we did create the "Beginners" tag to help identify some BZs for new
> >> >> folks to work on) too. I've been looking around for some ideas on
how
> > to
> >> >> make it easier for new people to contribute after having some
> >> >> conversations with friends about contributing to Tomcat and found
some
> >> >> interesting examples other projects are using to help bring new
people
> >> >> in, such as https://wiki.gnome.org/Newcomers (which is my favorite)
> >> >> and https://fedoraproject.org/wiki/Join. Obviously Tomcat isn't as
> >> >> large of a project as those, but it does have multiple places for
> >> >> people to contribute (Documentation, Patches, FAQ, wiki, etc) which
> >> >> could use different skill sets. This site
> >> >> http://whatcanidoforfedora.org/en would be really cool to implement,
> >> >> but at the ASF level I think (Tomcat isn't complex enough to warrant
> >> >> that, is it?).
> >> >>
> >> >> Anyway, the point of this email is really just to say that we should
> >> >> take some cues from other projects and try and develop a solid entry
> >> >> ramp to help entice new developers :) What does everyone else think?
> >> >
> >> > One thing that might help from my point of view is to provide
README.md
> > and
> >> > CONTRIBUTING.md for those who are working with GitHub replications of
> > the
> >> > repository. It is convenient to have the contribution's instruction
> >> > directly in the root of the repository.
> >> > e.g.
> >> > https://github.com/apache/jmeter/blob/trunk/README.md
> >> > https://github.com/apache/flink/blob/master/README.md
> >> >
> >> >
> >> > What do you think?
> >>
> >> Oh yeah. That's a great idea! I was just catching up on the thread and
> >> was trying to think of a way a way to let github users know what
> >> committers are doing with their PRs to get them committed (a README is
> >> obvious). I think that adding some transparency there may help them
> >> understand some issues that could cause latency.
> >>
> >
> > If you didn't start with README.md I can prepare some initial version.
>
> I hadn't started yet, but I intended to. It's on my TODO list :)

Ok I'll leave it to you ;)

Violeta

> If you want, you can put something up and I'll edit it as soon as I make
> time.
>
> > Regards,
> > Violeta
> >
> >> > Regards,
> >> > Violeta
> >> >>
> >> >>
> >> >> Thanks,
> >> >> Coty
> >>
> >> -
> >> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
> >> For additional commands, e-mail: dev-h...@tomcat.apache.org
> >>
>
> -
> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
> For additional commands, e-mail: dev-h...@tomcat.apache.org
>


[Bug 60594] RFC 7230/3986 url requirement that prevents unencoded curly braces should be optional, since it breaks existing sites

2017-06-12 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=60594

--- Comment #27 from Lulseged Zerfu  ---
Hi

 Any comment if you will add '"' to allow in our request URL? Ta the end of the
day we are taking the risk.

BR
Lulseged

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



Tomcat6 bug report

2017-06-12 Thread Orlando José Luque Moraira
Hi, first time here.

Found in tomcat6, not sure if it have been solved / changed in a newer
version.

Resumee: in a URLed jSessionId controlled session, if you do, from the
client, one or more html requests to get files but without jSessionId in
the URL, tomcat create a new "general site" cookie with a new jSessionId
and this new cookie is stored in the client. After a jSessionId is set in
that "general site" cookie, the jSessionId of the cookie is used instead of
the URLed jSessionId value, which is completely ignored from now, breaking
the URLed jSessionId functionality completely.

Example of requests in a row:
furball.com<-- server: set-cookie: jSessionId = 1234
furball.com;jsessionid=1234 --> we see the cookie in the client
furball.com/other  <-- server: set-cookie: jSessionId = 5678
furball.com;jsessionid=1234 --> it ignores the 1234 and starts using the
cookie jSessionId, so the jSessionId dissapear from the URLs shown in the
page... really it is a new session :-(


Any subsequent html request is flawed in the same way until you manage to
clean that cookie.


Seems like those new cokies are stored... in like a new local
not-session-related-cookie. As said, any subsequent html request will have
its jSessionId ignored and tomcat will use the one stored in that
"not-session-related-cookie", so you lose your original session and are in
danger of corruption between tabs with different sessions.

In a embedded webbrowser, this breaks all sessions working until you
restart the embedding application.

In webbrowser applications (as firefox) suffer the same way and as far as I
saw, deleting al cookies did not solve the issue. Weird. (I have the cache
madness so much... :-S).



Please, have this been resolved in newer versions? If possible, in which
one?

Thanks!

Regards,
   Orlando


Re: Things that we can do to increase contributor involvement?

2017-06-12 Thread Coty Sutherland
On Mon, Jun 12, 2017 at 3:20 AM, Violeta Georgieva  wrote:
> 2017-06-11 23:57 GMT+03:00 Coty Sutherland :
>>
>> On Sun, Jun 11, 2017 at 4:20 PM, Violeta Georgieva 
> wrote:
>> > Hi Coty,
>> >
>> > 2017-06-02 17:50 GMT+03:00 Coty Sutherland :
>> >>
>> >> On Fri, Jun 2, 2017 at 10:27 AM, Violeta Georgieva <
> violet...@apache.org>
>> > wrote:
>> >> > Hi,
>> >> >
>> >> > 2017-05-31 6:07 GMT+03:00 Coty Sutherland :
>> >> >>
>> >> >> Hi all,
>> >> >>
>> >> >> I've been thinking about things that we could do for Tomcat to help
>> >> >> bring in new contributors and to be more appealing to new
> developers.
>> >> >> Right now we have http://tomcat.apache.org/getinvolved.html which
> has
>> >> >> a few bullet points and links to documentation, which is a bit
> verbose,
>> >> >> about how to contribute to an Apache project. We also have the wiki
>> >> >> (https://wiki.apache.org/tomcat/FrontPage), which mentions nothing
>> >> >> about contributing. Bugzilla is a bit daunting for newcomers
> (thought
>> >> >> we did create the "Beginners" tag to help identify some BZs for new
>> >> >> folks to work on) too. I've been looking around for some ideas on
> how
>> > to
>> >> >> make it easier for new people to contribute after having some
>> >> >> conversations with friends about contributing to Tomcat and found
> some
>> >> >> interesting examples other projects are using to help bring new
> people
>> >> >> in, such as https://wiki.gnome.org/Newcomers (which is my favorite)
>> >> >> and https://fedoraproject.org/wiki/Join. Obviously Tomcat isn't as
>> >> >> large of a project as those, but it does have multiple places for
>> >> >> people to contribute (Documentation, Patches, FAQ, wiki, etc) which
>> >> >> could use different skill sets. This site
>> >> >> http://whatcanidoforfedora.org/en would be really cool to implement,
>> >> >> but at the ASF level I think (Tomcat isn't complex enough to warrant
>> >> >> that, is it?).
>> >> >>
>> >> >> Anyway, the point of this email is really just to say that we should
>> >> >> take some cues from other projects and try and develop a solid entry
>> >> >> ramp to help entice new developers :) What does everyone else think?
>> >> >
>> >> > One thing that might help from my point of view is to provide
> README.md
>> > and
>> >> > CONTRIBUTING.md for those who are working with GitHub replications of
>> > the
>> >> > repository. It is convenient to have the contribution's instruction
>> >> > directly in the root of the repository.
>> >> > e.g.
>> >> > https://github.com/apache/jmeter/blob/trunk/README.md
>> >> > https://github.com/apache/flink/blob/master/README.md
>> >> >
>> >> >
>> >> > What do you think?
>> >>
>> >> Oh yeah. That's a great idea! I was just catching up on the thread and
>> >> was trying to think of a way a way to let github users know what
>> >> committers are doing with their PRs to get them committed (a README is
>> >> obvious). I think that adding some transparency there may help them
>> >> understand some issues that could cause latency.
>> >>
>> >
>> > If you didn't start with README.md I can prepare some initial version.
>>
>> I hadn't started yet, but I intended to. It's on my TODO list :)
>
> Ok I'll leave it to you ;)

Sounds good. I'm working on it now and should be committing the two
files shortly in trunk for review. After I get some positive feedback
then I'll port to 7, 8, and 8.5. I was going to try and make them
generic documents, however since the branches are all in separate git
repos anyway I'm adding some version specific information.

> Violeta
>
>> If you want, you can put something up and I'll edit it as soon as I make
>> time.
>>
>> > Regards,
>> > Violeta
>> >
>> >> > Regards,
>> >> > Violeta
>> >> >>
>> >> >>
>> >> >> Thanks,
>> >> >> Coty
>> >>
>> >> -
>> >> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
>> >> For additional commands, e-mail: dev-h...@tomcat.apache.org
>> >>
>>
>> -
>> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
>> For additional commands, e-mail: dev-h...@tomcat.apache.org
>>

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



svn commit: r1798505 - in /tomcat/trunk: CONTRIBUTING.md README.md

2017-06-12 Thread csutherl
Author: csutherl
Date: Mon Jun 12 18:37:20 2017
New Revision: 1798505

URL: http://svn.apache.org/viewvc?rev=1798505&view=rev
Log:
Add README and CONTRIBUTING documents to help users that are utilizing the 
github mirror rather than SVN.

Added:
tomcat/trunk/CONTRIBUTING.md
tomcat/trunk/README.md

Added: tomcat/trunk/CONTRIBUTING.md
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/CONTRIBUTING.md?rev=1798505&view=auto
==
--- tomcat/trunk/CONTRIBUTING.md (added)
+++ tomcat/trunk/CONTRIBUTING.md Mon Jun 12 18:37:20 2017
@@ -0,0 +1,144 @@
+# Contributing to Apache Tomcat
+
+Firstly, thanks for your interest in contributing! I hope that this will be a
+pleasant first experience for you, and that you will return to continue
+contributing.
+
+Please visit our [Get Involved page](http://tomcat.apache.org/getinvolved.html)
+for more information on how to contribute.
+
+## Code of Conduct
+
+This project and everyone participating in it is governed by the Apache
+software Foundation's
+[Code of Conduct](http://www.apache.org/foundation/policies/conduct.html). By
+participating, you are expected to adhere to this code. If you are aware of
+unacceptable behavior, please visit the
+[Reporting Guidelines 
page](http://www.apache.org/foundation/policies/conduct.html#reporting-guidelines)
+and follow the instructions there.
+
+## How Can I Contribute?
+
+Most of the contributions that we receive are code contributions, but you can
+also contribute to the documentation, wiki, etc., or simply report solid bugs
+for us to fix.
+
+### Reporting Bugs
+
+Please review our [guide](http://tomcat.apache.org/bugreport.html) on how to
+submit a bug report. This page also has links to other resources to assist
+you.
+
+### Your First Code Contribution
+
+### Trouble Deciding How to Contribute?
+
+Unsure where to begin contributing to Tomcat? You can start by taking a look at
+the issues marked 'Beginner', link below. Please note that the Beginner keyword
+is pretty new to the project, so if there aren't any issues in the filter feel
+free to ask on the [dev list](http://tomcat.apache.org/lists.html#tomcat-dev).
+
+* [Beginner 
issues](https://bz.apache.org/bugzilla/buglist.cgi?bug_status=NEW&bug_status=ASSIGNED&bug_status=REOPENED&bug_status=NEEDINFO&keywords=Beginner&keywords_type=allwords&list_id=160824&product=Tomcat%207&product=Tomcat%208&product=Tomcat%209&query_format=advanced)
 -
+issues which should only require a few lines of code, and a test or two to
+resolve.
+
+The list above shows all bugs that are marked 'Beginner' and are open in the
+currently supported Tomcat versions (7, 8, and 9).
+
+If you prefer C over Java, you may also take a look at the tomcat-native and
+Tomcat Connectors products in Bugzilla.
+
+### How to Provide Your First Patch
+
+Excited yet? This section will guide you through providing a patch to the
+committers of the project for review and acceptance.
+
+# Chose Your Method of Submission
+
+You can provide a patch in one of the following ways (in order of preference):
+
+* Patch attachment to the Bugzilla issue
+* Github Pull Request
+> **Note:** Github is a mirror of the SVN repository that Tomcat is stored in
+and therefore it can't be merged outright. Your contribution will be converted
+into an SVN patch and committed with a mention of your name for credit.
+* Email the patch to the developer list. This is not preferred, but if no bug
+is associated with the patch, or you would like a developer review, an email
+may be appropriate.
+
+# Get the Sources
+
+Now that you've chosen how you want to submit a patch, you need to get the
+source code.
+
+## Download The Source Distribution
+
+This method works if you want to submit a patch (like you would do for SVN), 
but
+the difference in using the sources distribution and a VCS is that you have to
+manually generate the patch file by using diff. If this is what you want, you
+can download the sources from the "Source Code Distributions" section of the
+[Download Page](https://tomcat.apache.org/download-90.cgi).
+
+## SVN
+
+If you have chosen to attach a patch to the Bugzilla issue (or email
+one), then you'll need to checkout the SVN version. Instructions for new
+committers to learn how to do this are found
+[here](http://www.apache.org/dev/contributors.html#svnbasics). However, in the
+interest of a fast ramp up, the short version is below. Note that the root of
+the SVN repository is
+[tomcat/trunk](http://svn.apache.org/repos/asf/tomcat/trunk),
+but you can clone specific versions too, such as
+[tc8.5.x](http://svn.apache.org/repos/asf/tomcat/tc8.5.x/trunk/) or even tags (
+[TOMCAT_8_5_15](http://svn.apache.org/repos/asf/tomcat/tc8.5.x/tags/TOMCAT_8_5_15/)).
+
+```
+$ svn co http://svn.apache.org/repos/asf/tomcat/trunk/
+```
+
+# Github
+
+For Github, it's almost the same. Chose the major version that you want (for
+now they're in different re

svn commit: r1798507 - /tomcat/trunk/java/org/apache/catalina/valves/CrawlerSessionManagerValve.java

2017-06-12 Thread violetagg
Author: violetagg
Date: Mon Jun 12 18:39:38 2017
New Revision: 1798507

URL: http://svn.apache.org/viewvc?rev=1798507&view=rev
Log:
Format the code. No functional changes.
In preparation to apply PR https://github.com/apache/tomcat70/pull/3

Modified:
tomcat/trunk/java/org/apache/catalina/valves/CrawlerSessionManagerValve.java

Modified: 
tomcat/trunk/java/org/apache/catalina/valves/CrawlerSessionManagerValve.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/valves/CrawlerSessionManagerValve.java?rev=1798507&r1=1798506&r2=1798507&view=diff
==
--- 
tomcat/trunk/java/org/apache/catalina/valves/CrawlerSessionManagerValve.java 
(original)
+++ 
tomcat/trunk/java/org/apache/catalina/valves/CrawlerSessionManagerValve.java 
Mon Jun 12 18:39:38 2017
@@ -40,19 +40,14 @@ import org.apache.juli.logging.LogFactor
  * users - regardless of whether or not they provide a session token with their
  * requests.
  */
-public class CrawlerSessionManagerValve extends ValveBase
-implements HttpSessionBindingListener {
+public class CrawlerSessionManagerValve extends ValveBase implements 
HttpSessionBindingListener {
 
-private static final Log log =
-LogFactory.getLog(CrawlerSessionManagerValve.class);
+private static final Log log = 
LogFactory.getLog(CrawlerSessionManagerValve.class);
 
-private final Map clientIpSessionId =
-new ConcurrentHashMap<>();
-private final Map sessionIdClientIp =
-new ConcurrentHashMap<>();
+private final Map clientIpSessionId = new 
ConcurrentHashMap<>();
+private final Map sessionIdClientIp = new 
ConcurrentHashMap<>();
 
-private String crawlerUserAgents =
-".*[bB]ot.*|.*Yahoo! Slurp.*|.*Feedfetcher-Google.*";
+private String crawlerUserAgents = ".*[bB]ot.*|.*Yahoo! 
Slurp.*|.*Feedfetcher-Google.*";
 private Pattern uaPattern = null;
 private int sessionInactiveInterval = 60;
 
@@ -109,7 +104,7 @@ public class CrawlerSessionManagerValve
 }
 
 
-public Map getClientIpSessionId() {
+public Map getClientIpSessionId() {
 return clientIpSessionId;
 }
 
@@ -123,17 +118,15 @@ public class CrawlerSessionManagerValve
 
 
 @Override
-public void invoke(Request request, Response response) throws IOException,
-ServletException {
+public void invoke(Request request, Response response) throws IOException, 
ServletException {
 
 boolean isBot = false;
 String sessionId = null;
 String clientIp = null;
 
 if (log.isDebugEnabled()) {
-log.debug(request.hashCode() + ": ClientIp=" +
-request.getRemoteAddr() + ", RequestedSessionId=" +
-request.getRequestedSessionId());
+log.debug(request.hashCode() + ": ClientIp=" + 
request.getRemoteAddr()
++ ", RequestedSessionId=" + 
request.getRequestedSessionId());
 }
 
 // If the incoming request has a valid session ID, no action is 
required
@@ -157,8 +150,7 @@ public class CrawlerSessionManagerValve
 isBot = true;
 
 if (log.isDebugEnabled()) {
-log.debug(request.hashCode() +
-": Bot found. UserAgent=" + uaHeader);
+log.debug(request.hashCode() + ": Bot found. 
UserAgent=" + uaHeader);
 }
 }
 }
@@ -170,8 +162,7 @@ public class CrawlerSessionManagerValve
 if (sessionId != null) {
 request.setRequestedSessionId(sessionId);
 if (log.isDebugEnabled()) {
-log.debug(request.hashCode() + ": SessionID=" +
-sessionId);
+log.debug(request.hashCode() + ": SessionID=" + 
sessionId);
 }
 }
 }
@@ -191,14 +182,13 @@ public class CrawlerSessionManagerValve
 s.setMaxInactiveInterval(sessionInactiveInterval);
 
 if (log.isDebugEnabled()) {
-log.debug(request.hashCode() +
-": New bot session. SessionID=" + s.getId());
+log.debug(request.hashCode() + ": New bot session. 
SessionID=" + s.getId());
 }
 }
 } else {
 if (log.isDebugEnabled()) {
-log.debug(request.hashCode() +
-": Bot session accessed. SessionID=" + sessionId);
+log.debug(
+request.hashCode() + ": Bot session accessed. 
SessionID=" + sessionId);
 }
 }
 }



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

svn commit: r1798509 - in /tomcat/trunk: conf/logging.properties java/org/apache/coyote/AbstractProcessor.java java/org/apache/coyote/AsyncStateMachine.java webapps/docs/changelog.xml

2017-06-12 Thread markt
Author: markt
Date: Mon Jun 12 18:42:32 2017
New Revision: 1798509

URL: http://svn.apache.org/viewvc?rev=1798509&view=rev
Log:
Make asynchronous error handling more robust. In particular ensure that 
onError() is called for any registered AsyncListeners after an I/O error on a 
non-container thread.

Modified:
tomcat/trunk/conf/logging.properties
tomcat/trunk/java/org/apache/coyote/AbstractProcessor.java
tomcat/trunk/java/org/apache/coyote/AsyncStateMachine.java
tomcat/trunk/webapps/docs/changelog.xml

Modified: tomcat/trunk/conf/logging.properties
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/conf/logging.properties?rev=1798509&r1=1798508&r2=1798509&view=diff
==
--- tomcat/trunk/conf/logging.properties (original)
+++ tomcat/trunk/conf/logging.properties Mon Jun 12 18:42:32 2017
@@ -65,6 +65,8 @@ org.apache.catalina.core.ContainerBase.[
 
 # To see debug messages for HTTP/2 handling, uncomment the following line:
 #org.apache.coyote.http2.level = FINE
+org.apache.coyote.level = FINEST
+org.apache.catalina.level = FINEST
 
 # To see debug messages for WebSocket handling, uncomment the following line:
 #org.apache.tomcat.websocket.level = FINE

Modified: tomcat/trunk/java/org/apache/coyote/AbstractProcessor.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/AbstractProcessor.java?rev=1798509&r1=1798508&r2=1798509&view=diff
==
--- tomcat/trunk/java/org/apache/coyote/AbstractProcessor.java (original)
+++ tomcat/trunk/java/org/apache/coyote/AbstractProcessor.java Mon Jun 12 
18:42:32 2017
@@ -95,6 +95,7 @@ public abstract class AbstractProcessor
 // have been completed. Dispatch to a container thread to do the
 // clean-up. Need to do it this way to ensure that all the 
necessary
 // clean-up is performed.
+asyncStateMachine.asyncMustError();
 
getLog().info(sm.getString("abstractProcessor.nonContainerThreadError"), t);
 processSocketEvent(SocketEvent.ERROR, true);
 }

Modified: tomcat/trunk/java/org/apache/coyote/AsyncStateMachine.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/AsyncStateMachine.java?rev=1798509&r1=1798508&r2=1798509&view=diff
==
--- tomcat/trunk/java/org/apache/coyote/AsyncStateMachine.java [UTF-8] 
(original)
+++ tomcat/trunk/java/org/apache/coyote/AsyncStateMachine.java [UTF-8] Mon Jun 
12 18:42:32 2017
@@ -66,38 +66,49 @@ import org.apache.tomcat.util.security.P
  *differences required to avoid race conditions during 
error
  *handling.
  * DISPATCHING  - The dispatch is being processed.
+ * MUST_ERROR   - ServletRequest.startAsync() has been called followed by 
an
+ *I/O error on a non-container thread. The main purpose of
+ *this state is to prevent additional async actions
+ *(complete(), dispatch() etc.) on the non-container 
thread.
+ *The container will perform the necessary error handling,
+ *including ensuring that the AsyncLister.onError() method
+ *is called.
  * ERROR- Something went wrong.
  *
- * |-»--|
- * |   \|/ 
/---«---«--|
- * |   |--«-E R R O 
R--«---«---|  |
- * |   |  complete() /|\/|\\ \-««---|  
|  |
- * |   |  |  |  \   |  
|  |
- * |   ||-»---|  |   \---»--|   |  
|  |
- * |   |||  |dispatch() |  
|  |
- * |   ||| \|/  ^  
|  |
- * |   |||  |--|timeout()   |   |  
|  |
- * |   || post() |  | \|/   | post()|  
|  |
- * |   |||-- | --»DISPATCHED«-- | 
--COMPLETING«-|  |  |
- * |   |||   |   /|\/|\ |   || /|\ 
/|\  |  |  |
- * |   ||||---»- | ---|  |  |startAsync()   |   timeout()|--|  
 |   |  |  |
- * |   |^^|  |   |  |   |  
 |   |  ^  |
- * |   ||||   |-- \ -|  |   complete()  |  
 |post() |  |  |
- * |   ||||   |\| /--»- | 
---COMPLETE_PENDING-»-|   ^  |  |
- * 

Re: asyncError() is not valid while in Async state DISPATCHING

2017-06-12 Thread Mark Thomas
On 09/06/17 16:26, Violeta Georgieva wrote:
> 2017-06-09 17:25 GMT+03:00 Mark Thomas :



>> I've spent some time working through the various possible combinations
>> of events and have concluded it is impossible to completely fix this
>> without imposing additional requirements on applications that the
>> specification doesn't mention.
>>
>> However, I believe that we can do better than the current
>> implementation. What I have on mind would:
>>
>> - always trigger AsyncListener.onError() for all listeners
>> - generally, process the complete() dispatch() call from the
>>   AsyncListener rather than any from the non-container thread
>> - generally, throw an ISE if complete() or dispatch() is called
>>   from the non-container thread after that thread experiences an I/O
>>error
>> - leave a small timing window where it was possible that the complete()
>>   or dispatch() from the non-container thread would be used rather than
>>   from the AsyncListener. In that case the AsyncListener would see the
>>   ISE but any remaining AsyncListener instances would still be called
>>
>> I don't see a way of doing better than this without spec changes /
>> clarifications.
>>
>> WDYT?
> 
> +1
> I'm able to test the new behavior with my real web app.

Excellent. I've committed my proposed fix. The async unit tests pass
which is generally a good sign. If this works better with your real web
application then we can look to back-port this.

Mark

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



Re: svn commit: r1798509 - in /tomcat/trunk: conf/logging.properties java/org/apache/coyote/AbstractProcessor.java java/org/apache/coyote/AsyncStateMachine.java webapps/docs/changelog.xml

2017-06-12 Thread Martin Grigorov
Hi Mark,

On Mon, Jun 12, 2017 at 8:42 PM,  wrote:

> Author: markt
> Date: Mon Jun 12 18:42:32 2017
> New Revision: 1798509
>
> URL: http://svn.apache.org/viewvc?rev=1798509&view=rev
> Log:
> Make asynchronous error handling more robust. In particular ensure that
> onError() is called for any registered AsyncListeners after an I/O error on
> a non-container thread.
>
> Modified:
> tomcat/trunk/conf/logging.properties
> tomcat/trunk/java/org/apache/coyote/AbstractProcessor.java
> tomcat/trunk/java/org/apache/coyote/AsyncStateMachine.java
> tomcat/trunk/webapps/docs/changelog.xml
>
> Modified: tomcat/trunk/conf/logging.properties
> URL: http://svn.apache.org/viewvc/tomcat/trunk/conf/logging.prope
> rties?rev=1798509&r1=1798508&r2=1798509&view=diff
> 
> ==
> --- tomcat/trunk/conf/logging.properties (original)
> +++ tomcat/trunk/conf/logging.properties Mon Jun 12 18:42:32 2017
> @@ -65,6 +65,8 @@ org.apache.catalina.core.ContainerBase.[
>
>  # To see debug messages for HTTP/2 handling, uncomment the following line:
>  #org.apache.coyote.http2.level = FINE
> +org.apache.coyote.level = FINEST
> +org.apache.catalina.level = FINEST
>

Look like debug leftovers.


>
>  # To see debug messages for WebSocket handling, uncomment the following
> line:
>  #org.apache.tomcat.websocket.level = FINE
>
> Modified: tomcat/trunk/java/org/apache/coyote/AbstractProcessor.java
> URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/co
> yote/AbstractProcessor.java?rev=1798509&r1=1798508&r2=1798509&view=diff
> 
> ==
> --- tomcat/trunk/java/org/apache/coyote/AbstractProcessor.java (original)
> +++ tomcat/trunk/java/org/apache/coyote/AbstractProcessor.java Mon Jun 12
> 18:42:32 2017
> @@ -95,6 +95,7 @@ public abstract class AbstractProcessor
>  // have been completed. Dispatch to a container thread to do
> the
>  // clean-up. Need to do it this way to ensure that all the
> necessary
>  // clean-up is performed.
> +asyncStateMachine.asyncMustError();
>  
> getLog().info(sm.getString("abstractProcessor.nonContainerThreadError"),
> t);
>  processSocketEvent(SocketEvent.ERROR, true);
>  }
>
> Modified: tomcat/trunk/java/org/apache/coyote/AsyncStateMachine.java
> URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/co
> yote/AsyncStateMachine.java?rev=1798509&r1=1798508&r2=1798509&view=diff
> 
> ==
> --- tomcat/trunk/java/org/apache/coyote/AsyncStateMachine.java [UTF-8]
> (original)
> +++ tomcat/trunk/java/org/apache/coyote/AsyncStateMachine.java [UTF-8]
> Mon Jun 12 18:42:32 2017
> @@ -66,38 +66,49 @@ import org.apache.tomcat.util.security.P
>   *differences required to avoid race conditions
> during error
>   *handling.
>   * DISPATCHING  - The dispatch is being processed.
> + * MUST_ERROR   - ServletRequest.startAsync() has been called
> followed by an
> + *I/O error on a non-container thread. The main
> purpose of
> + *this state is to prevent additional async actions
> + *(complete(), dispatch() etc.) on the non-container
> thread.
> + *The container will perform the necessary error
> handling,
> + *including ensuring that the AsyncLister.onError()
> method
> + *is called.
>   * ERROR- Something went wrong.
>   *
> - * |-»--|
> - * |   \|/ /---«-
> --«--|
> - * |   |--«-E R R O R--«---«--
> -|  |
> - * |   |  complete() /|\/|\\ \-««---|
> |  |
> - * |   |  |  |  \
>|  |  |
> - * |   ||-»---|  |   \---»--|
>|  |  |
> - * |   |||  |dispatch()
>|  |  |
> - * |   ||| \|/
>   ^  |  |
> - * |   |||  |--|timeout()   |
>|  |  |
> - * |   || post() |  | \|/   | post()
>   |  |  |
> - * |   |||-- | --»DISPATCHED«-- |
> --COMPLETING«-|  |  |
> - * |   |||   |   /|\/|\ |   |
> | /|\ /|\  |  |  |
> - * |   ||||---»- | ---|  |  |startAsync()   |
>  timeout()|--|   |   |  |  |
> - * |   |^^|  |   |  |   |
>|   |  ^  |
> - * |   ||||   |-- \ -|  |   complete()  |

Re: Things that we can do to increase contributor involvement?

2017-06-12 Thread Coty Sutherland
On Mon, Jun 12, 2017 at 9:39 AM, Coty Sutherland  wrote:
> On Mon, Jun 12, 2017 at 3:20 AM, Violeta Georgieva  
> wrote:
>> 2017-06-11 23:57 GMT+03:00 Coty Sutherland :
>>>
>>> On Sun, Jun 11, 2017 at 4:20 PM, Violeta Georgieva 
>> wrote:
>>> > Hi Coty,
>>> >
>>> > 2017-06-02 17:50 GMT+03:00 Coty Sutherland :
>>> >>
>>> >> On Fri, Jun 2, 2017 at 10:27 AM, Violeta Georgieva <
>> violet...@apache.org>
>>> > wrote:
>>> >> > Hi,
>>> >> >
>>> >> > 2017-05-31 6:07 GMT+03:00 Coty Sutherland :
>>> >> >>
>>> >> >> Hi all,
>>> >> >>
>>> >> >> I've been thinking about things that we could do for Tomcat to help
>>> >> >> bring in new contributors and to be more appealing to new
>> developers.
>>> >> >> Right now we have http://tomcat.apache.org/getinvolved.html which
>> has
>>> >> >> a few bullet points and links to documentation, which is a bit
>> verbose,
>>> >> >> about how to contribute to an Apache project. We also have the wiki
>>> >> >> (https://wiki.apache.org/tomcat/FrontPage), which mentions nothing
>>> >> >> about contributing. Bugzilla is a bit daunting for newcomers
>> (thought
>>> >> >> we did create the "Beginners" tag to help identify some BZs for new
>>> >> >> folks to work on) too. I've been looking around for some ideas on
>> how
>>> > to
>>> >> >> make it easier for new people to contribute after having some
>>> >> >> conversations with friends about contributing to Tomcat and found
>> some
>>> >> >> interesting examples other projects are using to help bring new
>> people
>>> >> >> in, such as https://wiki.gnome.org/Newcomers (which is my favorite)
>>> >> >> and https://fedoraproject.org/wiki/Join. Obviously Tomcat isn't as
>>> >> >> large of a project as those, but it does have multiple places for
>>> >> >> people to contribute (Documentation, Patches, FAQ, wiki, etc) which
>>> >> >> could use different skill sets. This site
>>> >> >> http://whatcanidoforfedora.org/en would be really cool to implement,
>>> >> >> but at the ASF level I think (Tomcat isn't complex enough to warrant
>>> >> >> that, is it?).
>>> >> >>
>>> >> >> Anyway, the point of this email is really just to say that we should
>>> >> >> take some cues from other projects and try and develop a solid entry
>>> >> >> ramp to help entice new developers :) What does everyone else think?
>>> >> >
>>> >> > One thing that might help from my point of view is to provide
>> README.md
>>> > and
>>> >> > CONTRIBUTING.md for those who are working with GitHub replications of
>>> > the
>>> >> > repository. It is convenient to have the contribution's instruction
>>> >> > directly in the root of the repository.
>>> >> > e.g.
>>> >> > https://github.com/apache/jmeter/blob/trunk/README.md
>>> >> > https://github.com/apache/flink/blob/master/README.md
>>> >> >
>>> >> >
>>> >> > What do you think?
>>> >>
>>> >> Oh yeah. That's a great idea! I was just catching up on the thread and
>>> >> was trying to think of a way a way to let github users know what
>>> >> committers are doing with their PRs to get them committed (a README is
>>> >> obvious). I think that adding some transparency there may help them
>>> >> understand some issues that could cause latency.
>>> >>
>>> >
>>> > If you didn't start with README.md I can prepare some initial version.
>>>
>>> I hadn't started yet, but I intended to. It's on my TODO list :)
>>
>> Ok I'll leave it to you ;)

OK, published (what I think is) a solid first draft :) Now that
something is there, I'm sure everyone has ideas on what it needs to
be. Feel free to edit as needed. I tried to make the tone a bit jovial
so that it's a bit less daunting and more welcoming for new folks.

Cheers,

> Sounds good. I'm working on it now and should be committing the two
> files shortly in trunk for review. After I get some positive feedback
> then I'll port to 7, 8, and 8.5. I was going to try and make them
> generic documents, however since the branches are all in separate git
> repos anyway I'm adding some version specific information.
>
>> Violeta
>>
>>> If you want, you can put something up and I'll edit it as soon as I make
>>> time.
>>>
>>> > Regards,
>>> > Violeta
>>> >
>>> >> > Regards,
>>> >> > Violeta
>>> >> >>
>>> >> >>
>>> >> >> Thanks,
>>> >> >> Coty
>>> >>
>>> >> -
>>> >> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
>>> >> For additional commands, e-mail: dev-h...@tomcat.apache.org
>>> >>
>>>
>>> -
>>> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
>>> For additional commands, e-mail: dev-h...@tomcat.apache.org
>>>

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



[Bug 61180] New: Change log level of sessionIdGeneratorBase.createRandom to warn rather than info

2017-06-12 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=61180

Bug ID: 61180
   Summary: Change log level of
sessionIdGeneratorBase.createRandom to warn rather
than info
   Product: Tomcat 8
   Version: 8.0.x-trunk
  Hardware: All
OS: All
Status: NEW
  Severity: trivial
  Priority: P2
 Component: Catalina
  Assignee: dev@tomcat.apache.org
  Reporter: csuth...@redhat.com
  Target Milestone: 

I was just asked a question in #tomcat about why a user's tomcat installation
was taking ~36 minutes to start. After looking at their logging I immediately
noticed:

INFO [localhost-startStop-1]
org.apache.catalina.util.SessionIdGeneratorBase.createSecureRandom Creation of
SecureRandom instance for session ID generation using [SHA1PRNG] took
[2,157,784] milliseconds.

showing that the machine had low entropy. I knew that from experience, however
the user had no idea that was the issue because the logging looked normal to
them (no WARN or ERROR messages). Can we change the log level of the
sessionIdGeneratorBase.createRandom message to WARN rather than INFO so if it
takes longer than 100 ms to generate a random users are made aware? A WARN
message will at least make them look twice at the error line and google it as a
potential issue, which should yield the wiki page
(https://wiki.apache.org/tomcat/HowTo/FasterStartUp#Entropy_Source).

Also, the wording in the wiki page makes this sound like it should be a warning
anyway:

"You will see warning in the logs when this happens"

-- 
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 61180] Change log level of sessionIdGeneratorBase.createRandom to warn rather than info

2017-06-12 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=61180

Coty Sutherland  changed:

   What|Removed |Added

   Keywords||Beginner

--- Comment #1 from Coty Sutherland  ---
This is the definition of a "Beginner" issue IMO :)

-- 
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: r1798514 - /tomcat/trunk/conf/logging.properties

2017-06-12 Thread markt
Author: markt
Date: Mon Jun 12 19:21:46 2017
New Revision: 1798514

URL: http://svn.apache.org/viewvc?rev=1798514&view=rev
Log:
Revert debug config mistakenly included in r1798509

Modified:
tomcat/trunk/conf/logging.properties

Modified: tomcat/trunk/conf/logging.properties
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/conf/logging.properties?rev=1798514&r1=1798513&r2=1798514&view=diff
==
--- tomcat/trunk/conf/logging.properties (original)
+++ tomcat/trunk/conf/logging.properties Mon Jun 12 19:21:46 2017
@@ -65,8 +65,6 @@ org.apache.catalina.core.ContainerBase.[
 
 # To see debug messages for HTTP/2 handling, uncomment the following line:
 #org.apache.coyote.http2.level = FINE
-org.apache.coyote.level = FINEST
-org.apache.catalina.level = FINEST
 
 # To see debug messages for WebSocket handling, uncomment the following line:
 #org.apache.tomcat.websocket.level = FINE



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



svn commit: r1798533 - in /tomcat/trunk: java/org/apache/catalina/valves/CrawlerSessionManagerValve.java test/org/apache/catalina/valves/TestCrawlerSessionManagerValve.java webapps/docs/changelog.xml

2017-06-12 Thread violetagg
Author: violetagg
Date: Mon Jun 12 21:04:53 2017
New Revision: 1798533

URL: http://svn.apache.org/viewvc?rev=1798533&view=rev
Log:
A new configuration property 'crawlerIps' is added to the 
'o.a.catalina.valves.CrawlerSessionManagerValve'. Using this property one can 
specify a regular expression that will be used to identify crawlers based on 
their IP address. Based on a patch provided by Tetradeus via GitHub.

Added:

tomcat/trunk/test/org/apache/catalina/valves/TestCrawlerSessionManagerValve.java
   (with props)
Modified:
tomcat/trunk/java/org/apache/catalina/valves/CrawlerSessionManagerValve.java
tomcat/trunk/webapps/docs/changelog.xml
tomcat/trunk/webapps/docs/config/valve.xml

Modified: 
tomcat/trunk/java/org/apache/catalina/valves/CrawlerSessionManagerValve.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/valves/CrawlerSessionManagerValve.java?rev=1798533&r1=1798532&r2=1798533&view=diff
==
--- 
tomcat/trunk/java/org/apache/catalina/valves/CrawlerSessionManagerValve.java 
(original)
+++ 
tomcat/trunk/java/org/apache/catalina/valves/CrawlerSessionManagerValve.java 
Mon Jun 12 21:04:53 2017
@@ -49,6 +49,10 @@ public class CrawlerSessionManagerValve
 
 private String crawlerUserAgents = ".*[bB]ot.*|.*Yahoo! 
Slurp.*|.*Feedfetcher-Google.*";
 private Pattern uaPattern = null;
+
+private String crawlerIps = null;
+private Pattern ipPattern = null;
+
 private int sessionInactiveInterval = 60;
 
 
@@ -86,6 +90,31 @@ public class CrawlerSessionManagerValve
 
 
 /**
+ * Specify the regular expression (using {@link Pattern}) that will be used
+ * to identify crawlers based on their IP address. The default is no 
crawler
+ * IPs.
+ *
+ * @param crawlerIps The regular expression using {@link Pattern}
+ */
+public void setCrawlerIps(String crawlerIps) {
+this.crawlerIps = crawlerIps;
+if (crawlerIps == null || crawlerIps.length() == 0) {
+ipPattern = null;
+} else {
+ipPattern = Pattern.compile(crawlerIps);
+}
+}
+
+/**
+ * @see #setCrawlerIps(String)
+ * @return The current regular expression being used to match IP addresses.
+ */
+public String getCrawlerIps() {
+return crawlerIps;
+}
+
+
+/**
  * Specify the session timeout (in seconds) for a crawler's session. This 
is
  * typically lower than that for a user session. The default is 60 seconds.
  *
@@ -122,11 +151,11 @@ public class CrawlerSessionManagerValve
 
 boolean isBot = false;
 String sessionId = null;
-String clientIp = null;
+String clientIp = request.getRemoteAddr();
 
 if (log.isDebugEnabled()) {
-log.debug(request.hashCode() + ": ClientIp=" + 
request.getRemoteAddr()
-+ ", RequestedSessionId=" + 
request.getRequestedSessionId());
+log.debug(request.hashCode() + ": ClientIp=" + clientIp + ", 
RequestedSessionId="
++ request.getRequestedSessionId());
 }
 
 // If the incoming request has a valid session ID, no action is 
required
@@ -155,9 +184,16 @@ public class CrawlerSessionManagerValve
 }
 }
 
+if (ipPattern != null && ipPattern.matcher(clientIp).matches()) {
+isBot = true;
+
+if (log.isDebugEnabled()) {
+log.debug(request.hashCode() + ": Bot found. IP=" + 
clientIp);
+}
+}
+
 // If this is a bot, is the session ID known?
 if (isBot) {
-clientIp = request.getRemoteAddr();
 sessionId = clientIpSessionId.get(clientIp);
 if (sessionId != null) {
 request.setRequestedSessionId(sessionId);

Added: 
tomcat/trunk/test/org/apache/catalina/valves/TestCrawlerSessionManagerValve.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/valves/TestCrawlerSessionManagerValve.java?rev=1798533&view=auto
==
--- 
tomcat/trunk/test/org/apache/catalina/valves/TestCrawlerSessionManagerValve.java
 (added)
+++ 
tomcat/trunk/test/org/apache/catalina/valves/TestCrawlerSessionManagerValve.java
 Mon Jun 12 21:04:53 2017
@@ -0,0 +1,86 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * 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 writi

svn commit: r1798535 - in /tomcat/tc8.5.x/trunk: ./ java/org/apache/catalina/valves/CrawlerSessionManagerValve.java

2017-06-12 Thread violetagg
Author: violetagg
Date: Mon Jun 12 21:08:59 2017
New Revision: 1798535

URL: http://svn.apache.org/viewvc?rev=1798535&view=rev
Log:
Format the code. No functional changes.

Modified:
tomcat/tc8.5.x/trunk/   (props changed)

tomcat/tc8.5.x/trunk/java/org/apache/catalina/valves/CrawlerSessionManagerValve.java

Propchange: tomcat/tc8.5.x/trunk/
--
--- svn:mergeinfo (original)
+++ svn:mergeinfo Mon Jun 12 21:08:59 2017
@@ -1 +1 @@
-/tomcat/trunk:1734785,1734799,1734845,1734928,1735041,1735044,1735480,1735577,1735597,1735599-1735600,1735615,1736145,1736162,1736209,1736280,1736297,1736299,1736489,1736646,1736703,1736836,1736849,1737104-1737105,1737112,1737117,1737119-1737120,1737155,1737157,1737192,1737280,1737339,1737632,1737664,1737715,1737748,1737785,1737834,1737860,1737903,1737959,1738005,1738007,1738014-1738015,1738018,1738022,1738039,1738043,1738059-1738060,1738147,1738149,1738174-1738175,1738261,1738589,1738623-1738625,1738643,1738816,1738850,1738855,1738946-1738948,1738953-1738954,1738979,1738982,1739079-1739081,1739087,1739113,1739153,1739172,1739176,1739191,1739474,1739726,1739762,1739775,1739814,1739817-1739818,1739975,1740131,1740324,1740465,1740495,1740508-1740509,1740520,1740535,1740707,1740803,1740810,1740969,1740980,1740991,1740997,1741015,1741033,1741036,1741058,1741060,1741080,1741147,1741159,1741164,1741173,1741181,1741190,1741197,1741202,1741208,1741213,1741221,1741225,1741232,1741409,1741501
 
,1741677,1741892,1741896,1741984,1742023,1742042,1742071,1742090,1742093,1742101,1742105,1742111,1742139,1742146,1742148,1742166,1742181,1742184,1742187,1742246,1742248-1742251,1742263-1742264,1742268,1742276,1742369,1742387,1742448,1742509-1742512,1742917,1742919,1742933,1742975-1742976,1742984,1742986,1743019,1743115,1743117,1743124-1743125,1743134,1743425,1743554,1743679,1743696-1743698,1743700-1743701,1744058,1744064-1744065,1744125,1744194,1744229,1744270,1744323,1744432,1744684,1744697,1744705,1744713,1744760,1744786,1745083,1745142-1745143,1745145,1745177,1745179-1745180,1745227,1745248,1745254,1745337,1745467,1745473,1745535,1745576,1745735,1745744,1746304,1746306-1746307,1746319,1746327,1746338,1746340-1746341,1746344,1746427,1746441,1746473,1746490,1746492,1746495-1746496,1746499-1746501,1746503-1746507,1746509,1746549,1746551,1746554,1746556,1746558,1746584,1746620,1746649,1746724,1746939,1746989,1747014,1747028,1747035,1747210,1747225,1747234,1747253,1747404,1747506,1747
 
536,1747924,1747980,1747993,1748001,1748253,1748452,1748547,1748629,1748676,1748715,1749287,1749296,1749328,1749373,1749465,1749506,1749508,1749665-1749666,1749763,1749865-1749866,1749898,1749978,1749980,1750011,1750015,1750056,1750480,1750617,1750634,1750692,1750697,1750700,1750703,1750707,1750714,1750718,1750723,1750774,1750899,1750975,1750995,1751061,1751097,1751173,1751438,1751447,1751463,1751702,1752212,1752737,1752745,1753078,1753080,1753358,1753363,1754111,1754140-1754141,1754281,1754310,1754445,1754467,1754494,1754496,1754528,1754532-1754533,1754613,1754714,1754874,1754941,1754944,1754950-1754951,1755005,1755007,1755009,1755132,1755180-1755181,1755185,1755190,1755204-1755206,1755208,1755214,1755224,1755227,1755230,1755629,1755646-1755647,1755650,1755653,1755675,1755680,1755683,1755693,1755717,1755731-1755737,1755812,1755828,1755884,1755890,1755918-1755919,1755942,1755958,1755960,1755970,1755993,1756013,1756019,1756039,1756056,1756083-1756114,1756175,1756288-1756289,1756408-1
 
756410,1756778,1756798,1756878,1756898,1756939,1757123-1757124,1757126,1757128,1757132-1757133,1757136,1757145,1757167-1757168,1757175,1757180,1757182,1757195,1757271,1757278,1757347,1757353-1757354,1757363,1757374,1757399,1757406,1757408,1757485,1757495,1757499,1757527,1757578,1757684,1757722,1757727,1757790,1757799,1757813,1757853,1757883,1757903,1757976,1757997,1758000,1758058,1758072-1758075,1758078-1758079,1758223,1758257,1758261,1758276,1758292,1758369,1758378-1758383,1758421,1758423,1758425-1758427,1758430,1758443,1758448,1758459,1758483,1758486-1758487,1758499,1758525,1758556,1758580,1758582,1758584,1758588,1758842,1759019,1759212,1759224,1759227,1759252,1759274,1759513-1759516,1759611,1759757,1759785-1759790,1760005,1760022,1760109-1760110,1760135,1760200-1760201,1760227,1760300,1760397,1760446,1760454,1760640,1760648,1761057,1761422,1761491,1761498,1761500-1761501,1761550,1761553,1761572,1761574,1761625-1761626,1761628,1761682,1761740,1761752,1762051-1762053,1762123,176216
 
8,1762172,1762182,1762201-1762202,1762204,1762208,1762288,1762296,1762324,1762348,1762353,1762362,1762374,1762492,1762503,1762505,1762541,1762608,1762710,1762753,1762766,1762769,1762944,1762947,1762953,1763167,1763179,1763232,1763259,1763271-1763272,1763276-1763277,1763319-1763320,1763370,1763372,1763375,1763377,1763393,1763412,1763430,1763450,1763462,1763505,1763511-1763512,1763516,1763518,1763520,1763529,1763559,1763565,1763568,1763574,1763619,1763634-1763635,1763718,1763786,1

svn commit: r1798536 - in /tomcat/tc8.5.x/trunk: ./ java/org/apache/catalina/valves/CrawlerSessionManagerValve.java test/org/apache/catalina/valves/TestCrawlerSessionManagerValve.java webapps/docs/cha

2017-06-12 Thread violetagg
Author: violetagg
Date: Mon Jun 12 21:12:29 2017
New Revision: 1798536

URL: http://svn.apache.org/viewvc?rev=1798536&view=rev
Log:
A new configuration property 'crawlerIps' is added to the 
'o.a.catalina.valves.CrawlerSessionManagerValve'. Using this property one can 
specify a regular expression that will be used to identify crawlers based on 
their IP address. Based on a patch provided by Tetradeus via GitHub.

Added:

tomcat/tc8.5.x/trunk/test/org/apache/catalina/valves/TestCrawlerSessionManagerValve.java
  - copied unchanged from r1798533, 
tomcat/trunk/test/org/apache/catalina/valves/TestCrawlerSessionManagerValve.java
Modified:
tomcat/tc8.5.x/trunk/   (props changed)

tomcat/tc8.5.x/trunk/java/org/apache/catalina/valves/CrawlerSessionManagerValve.java
tomcat/tc8.5.x/trunk/webapps/docs/changelog.xml
tomcat/tc8.5.x/trunk/webapps/docs/config/valve.xml

Propchange: tomcat/tc8.5.x/trunk/
--
--- svn:mergeinfo (original)
+++ svn:mergeinfo Mon Jun 12 21:12:29 2017
@@ -1 +1 @@
-/tomcat/trunk:1734785,1734799,1734845,1734928,1735041,1735044,1735480,1735577,1735597,1735599-1735600,1735615,1736145,1736162,1736209,1736280,1736297,1736299,1736489,1736646,1736703,1736836,1736849,1737104-1737105,1737112,1737117,1737119-1737120,1737155,1737157,1737192,1737280,1737339,1737632,1737664,1737715,1737748,1737785,1737834,1737860,1737903,1737959,1738005,1738007,1738014-1738015,1738018,1738022,1738039,1738043,1738059-1738060,1738147,1738149,1738174-1738175,1738261,1738589,1738623-1738625,1738643,1738816,1738850,1738855,1738946-1738948,1738953-1738954,1738979,1738982,1739079-1739081,1739087,1739113,1739153,1739172,1739176,1739191,1739474,1739726,1739762,1739775,1739814,1739817-1739818,1739975,1740131,1740324,1740465,1740495,1740508-1740509,1740520,1740535,1740707,1740803,1740810,1740969,1740980,1740991,1740997,1741015,1741033,1741036,1741058,1741060,1741080,1741147,1741159,1741164,1741173,1741181,1741190,1741197,1741202,1741208,1741213,1741221,1741225,1741232,1741409,1741501
 
,1741677,1741892,1741896,1741984,1742023,1742042,1742071,1742090,1742093,1742101,1742105,1742111,1742139,1742146,1742148,1742166,1742181,1742184,1742187,1742246,1742248-1742251,1742263-1742264,1742268,1742276,1742369,1742387,1742448,1742509-1742512,1742917,1742919,1742933,1742975-1742976,1742984,1742986,1743019,1743115,1743117,1743124-1743125,1743134,1743425,1743554,1743679,1743696-1743698,1743700-1743701,1744058,1744064-1744065,1744125,1744194,1744229,1744270,1744323,1744432,1744684,1744697,1744705,1744713,1744760,1744786,1745083,1745142-1745143,1745145,1745177,1745179-1745180,1745227,1745248,1745254,1745337,1745467,1745473,1745535,1745576,1745735,1745744,1746304,1746306-1746307,1746319,1746327,1746338,1746340-1746341,1746344,1746427,1746441,1746473,1746490,1746492,1746495-1746496,1746499-1746501,1746503-1746507,1746509,1746549,1746551,1746554,1746556,1746558,1746584,1746620,1746649,1746724,1746939,1746989,1747014,1747028,1747035,1747210,1747225,1747234,1747253,1747404,1747506,1747
 
536,1747924,1747980,1747993,1748001,1748253,1748452,1748547,1748629,1748676,1748715,1749287,1749296,1749328,1749373,1749465,1749506,1749508,1749665-1749666,1749763,1749865-1749866,1749898,1749978,1749980,1750011,1750015,1750056,1750480,1750617,1750634,1750692,1750697,1750700,1750703,1750707,1750714,1750718,1750723,1750774,1750899,1750975,1750995,1751061,1751097,1751173,1751438,1751447,1751463,1751702,1752212,1752737,1752745,1753078,1753080,1753358,1753363,1754111,1754140-1754141,1754281,1754310,1754445,1754467,1754494,1754496,1754528,1754532-1754533,1754613,1754714,1754874,1754941,1754944,1754950-1754951,1755005,1755007,1755009,1755132,1755180-1755181,1755185,1755190,1755204-1755206,1755208,1755214,1755224,1755227,1755230,1755629,1755646-1755647,1755650,1755653,1755675,1755680,1755683,1755693,1755717,1755731-1755737,1755812,1755828,1755884,1755890,1755918-1755919,1755942,1755958,1755960,1755970,1755993,1756013,1756019,1756039,1756056,1756083-1756114,1756175,1756288-1756289,1756408-1
 
756410,1756778,1756798,1756878,1756898,1756939,1757123-1757124,1757126,1757128,1757132-1757133,1757136,1757145,1757167-1757168,1757175,1757180,1757182,1757195,1757271,1757278,1757347,1757353-1757354,1757363,1757374,1757399,1757406,1757408,1757485,1757495,1757499,1757527,1757578,1757684,1757722,1757727,1757790,1757799,1757813,1757853,1757883,1757903,1757976,1757997,1758000,1758058,1758072-1758075,1758078-1758079,1758223,1758257,1758261,1758276,1758292,1758369,1758378-1758383,1758421,1758423,1758425-1758427,1758430,1758443,1758448,1758459,1758483,1758486-1758487,1758499,1758525,1758556,1758580,1758582,1758584,1758588,1758842,1759019,1759212,1759224,1759227,1759252,1759274,1759513-1759516,1759611,1759757,1759785-1759790,1760005,1760022,1760109-1760110,1760135,1760200-1760201,1760227,1760300,1760397,1760446,1760454,1760640,1760648,1761057,1761422,1761491,1761498,1761500-1761501,1761550,1761553,17615

svn commit: r1798537 - in /tomcat/tc8.0.x/trunk: ./ java/org/apache/catalina/valves/CrawlerSessionManagerValve.java

2017-06-12 Thread violetagg
Author: violetagg
Date: Mon Jun 12 21:13:52 2017
New Revision: 1798537

URL: http://svn.apache.org/viewvc?rev=1798537&view=rev
Log:
Format the code. No functional changes.

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

tomcat/tc8.0.x/trunk/java/org/apache/catalina/valves/CrawlerSessionManagerValve.java

Propchange: tomcat/tc8.0.x/trunk/
--
--- svn:mergeinfo (original)
+++ svn:mergeinfo Mon Jun 12 21:13:52 2017
@@ -1,2 +1,2 @@
 
/tomcat/tc8.5.x/trunk:1735042,1737966,1743139-1743140,1744151,1747537,1747925,1748002,1754614,1754643,1762124,1762183,1762203,1763792,1772948,1777014,1779719,1782037,1782240,1782386-1782387,1785669,1786845,1788249,1788324,1788905,1789216,1789335,1791528,1791558,1796697-1796698,1797521
-/tomcat/trunk:1636524,1637156,1637176,1637188,1637331,1637684,1637695,1637890,1637892,1638720-1638725,1639653,1640010,1640083-1640084,1640088,1640275,1640322,1640347,1640361,1640365,1640403,1640410,1640652,1640655-1640658,1640688,1640700-1640883,1640903,1640976,1640978,1641000,1641026,1641038-1641039,1641051-1641052,1641058,1641064,1641300,1641369,1641374,1641380,1641486,1641634,1641656-1641692,1641704,1641707-1641718,1641720-1641722,1641735,1641981,1642233,1642280,1642554,1642564,1642595,1642606,1642668,1642679,1642697,1642699,1642766,1643002,1643045,1643054-1643055,1643066,1643121,1643128,1643206,1643209-1643210,1643216,1643249,1643270,1643283,1643309-1643310,1643323,1643365-1643366,1643370-1643371,1643465,1643474,1643536,1643570,1643634,1643649,1643651,1643654,1643675,1643731,1643733-1643734,1643761,1643766,1643814,1643937,1643963,1644017,1644169,1644201-1644203,1644321,1644323,1644516,1644523,1644529,1644535,1644730,1644768,1644784-1644785,1644790,1644793,1644815,1644884,1644886
 
,1644890,1644892,1644910,1644924,1644929-1644930,1644935,1644989,1645011,1645247,1645355,1645357-1645358,1645455,1645465,1645469,1645471,1645473,1645475,1645486-1645488,1645626,1645641,1645685,1645743,1645763,1645951-1645953,1645955,1645993,1646098-1646106,1646178,1646220,1646302,1646304,1646420,1646470-1646471,1646476,1646559,1646717-1646723,1646773,1647026,1647042,1647530,1647655,1648304,1648815,1648907,1649973,1650081,1650365,1651116,1651120,1651280,1651470,1652938,1652970,1653041,1653471,1653550,1653574,1653797,1653815-1653816,1653819,1653840,1653857,1653888,1653972,1654013,1654030,1654050,1654123,1654148,1654159,1654513,1654515,1654517,1654522,1654524,1654725,1654735,1654766,1654785,1654851-1654852,1654978,1655122-1655124,1655126-1655127,1655129-1655130,1655132-1655133,1655312,1655351,1655438,1655441,1655454,168,1656087,1656299,1656319,1656331,1656345,1656350,1656590,1656648-1656650,1656657,1657041,1657054,1657374,1657492,1657510,1657565,1657580,1657584,1657586,1657589,1657
 
592,1657607,1657609,1657682,1657907,1658207,1658734,1658781,1658790,1658799,1658802,1658804,1658833,1658840,1658966,1659043,1659053,1659059,1659174,1659184,1659188-1659189,1659216,1659263,1659293,1659304,1659306-1659307,1659382,1659384,1659428,1659471,1659486,1659505,1659516,1659521,1659524,1659559,1659562,1659803,1659806,1659814,1659833,1659862,1659905,1659919,1659948,1659967,1659983-1659984,1660060,1660074,1660077,1660133,1660168,1660331-1660332,1660353,1660358,1660924,1661386,1661770,1661867,1661972,1661990,1662200,1662308-1662309,1662548,1662614,1662696,1662736,1662985,1662988-1662989,1663264,1663277,1663298,1663534,1663562,1663676,1663715,1663754,1663768,1663772,1663781,1663893,1663995,1664143,1664163,1664174,1664301,1664317,1664347,1664657,1664659,1664710,1664863-1664864,1664866,1665085,1665292,1665559,1665653,1665661,1665672,1665694,1665697,1665736,1665779,1665976-1665977,1665980-1665981,1665985-1665986,1665989,1665998,1666004,1666008,1666013,1666017,1666024,1666116,1666386-1
 
666387,1666494,1666496,1666552,1666569,1666579,137,149,1666757,1666966,1666972,1666985,1666995,1666997,1667292,1667402,1667406,1667546,1667615,1667630,1667636,1667688,1667764,1667871,1668026,1668135,1668193,1668593,1668596,1668630,1668639,1668843,1669353,1669370,1669451,1669800,1669838,1669876,1669882,1670394,1670433,1670591,1670598-1670600,1670610,1670631,1670719,1670724,1670726,1670730,1670940,1671112,1672272,1672284,1673754,1674294,1675461,1675486,1675594,1675830,1676231,1676250-1676251,1676364,1676381,1676393,1676479,1676525,1676552,1676615,1676630,1676634,1676721,1676926,1676943,1677140,1677802,1678011,1678162,1678174,1678339,1678426-1678427,1678694,1678701,1679534,1679708,1679710,1679716,1680034,1680246,1681056,1681123,1681138,1681280,1681283,1681286,1681450,1681697,1681699,1681701,1681729,1681770,1681779,1681793,1681807,1681837-1681838,1681854,1681862,1681958,1682028,1682033,1682311,1682315,1682317,1682320,1682324,1682330,1682842,1684172,1684366,1684383,1684526-168452
 
7,1684549-1684550,1685556,1685591,1685739,1685744,1685772,1685816,1685826,1685891,1687242,1687261,1687268,1687340,1687544,1687551,1688563,1688841,1688878,165,1688896,1688901,1689345-16893

svn commit: r1798539 - in /tomcat/tc8.0.x/trunk: ./ java/org/apache/catalina/valves/CrawlerSessionManagerValve.java test/org/apache/catalina/valves/TestCrawlerSessionManagerValve.java webapps/docs/cha

2017-06-12 Thread violetagg
Author: violetagg
Date: Mon Jun 12 21:16:35 2017
New Revision: 1798539

URL: http://svn.apache.org/viewvc?rev=1798539&view=rev
Log:
A new configuration property 'crawlerIps' is added to the 
'o.a.catalina.valves.CrawlerSessionManagerValve'. Using this property one can 
specify a regular expression that will be used to identify crawlers based on 
their IP address. Based on a patch provided by Tetradeus via GitHub.

Added:

tomcat/tc8.0.x/trunk/test/org/apache/catalina/valves/TestCrawlerSessionManagerValve.java
  - copied unchanged from r1798533, 
tomcat/trunk/test/org/apache/catalina/valves/TestCrawlerSessionManagerValve.java
Modified:
tomcat/tc8.0.x/trunk/   (props changed)

tomcat/tc8.0.x/trunk/java/org/apache/catalina/valves/CrawlerSessionManagerValve.java
tomcat/tc8.0.x/trunk/webapps/docs/changelog.xml
tomcat/tc8.0.x/trunk/webapps/docs/config/valve.xml

Propchange: tomcat/tc8.0.x/trunk/
--
--- svn:mergeinfo (original)
+++ svn:mergeinfo Mon Jun 12 21:16:35 2017
@@ -1,2 +1,2 @@
 
/tomcat/tc8.5.x/trunk:1735042,1737966,1743139-1743140,1744151,1747537,1747925,1748002,1754614,1754643,1762124,1762183,1762203,1763792,1772948,1777014,1779719,1782037,1782240,1782386-1782387,1785669,1786845,1788249,1788324,1788905,1789216,1789335,1791528,1791558,1796697-1796698,1797521
-/tomcat/trunk:1636524,1637156,1637176,1637188,1637331,1637684,1637695,1637890,1637892,1638720-1638725,1639653,1640010,1640083-1640084,1640088,1640275,1640322,1640347,1640361,1640365,1640403,1640410,1640652,1640655-1640658,1640688,1640700-1640883,1640903,1640976,1640978,1641000,1641026,1641038-1641039,1641051-1641052,1641058,1641064,1641300,1641369,1641374,1641380,1641486,1641634,1641656-1641692,1641704,1641707-1641718,1641720-1641722,1641735,1641981,1642233,1642280,1642554,1642564,1642595,1642606,1642668,1642679,1642697,1642699,1642766,1643002,1643045,1643054-1643055,1643066,1643121,1643128,1643206,1643209-1643210,1643216,1643249,1643270,1643283,1643309-1643310,1643323,1643365-1643366,1643370-1643371,1643465,1643474,1643536,1643570,1643634,1643649,1643651,1643654,1643675,1643731,1643733-1643734,1643761,1643766,1643814,1643937,1643963,1644017,1644169,1644201-1644203,1644321,1644323,1644516,1644523,1644529,1644535,1644730,1644768,1644784-1644785,1644790,1644793,1644815,1644884,1644886
 
,1644890,1644892,1644910,1644924,1644929-1644930,1644935,1644989,1645011,1645247,1645355,1645357-1645358,1645455,1645465,1645469,1645471,1645473,1645475,1645486-1645488,1645626,1645641,1645685,1645743,1645763,1645951-1645953,1645955,1645993,1646098-1646106,1646178,1646220,1646302,1646304,1646420,1646470-1646471,1646476,1646559,1646717-1646723,1646773,1647026,1647042,1647530,1647655,1648304,1648815,1648907,1649973,1650081,1650365,1651116,1651120,1651280,1651470,1652938,1652970,1653041,1653471,1653550,1653574,1653797,1653815-1653816,1653819,1653840,1653857,1653888,1653972,1654013,1654030,1654050,1654123,1654148,1654159,1654513,1654515,1654517,1654522,1654524,1654725,1654735,1654766,1654785,1654851-1654852,1654978,1655122-1655124,1655126-1655127,1655129-1655130,1655132-1655133,1655312,1655351,1655438,1655441,1655454,168,1656087,1656299,1656319,1656331,1656345,1656350,1656590,1656648-1656650,1656657,1657041,1657054,1657374,1657492,1657510,1657565,1657580,1657584,1657586,1657589,1657
 
592,1657607,1657609,1657682,1657907,1658207,1658734,1658781,1658790,1658799,1658802,1658804,1658833,1658840,1658966,1659043,1659053,1659059,1659174,1659184,1659188-1659189,1659216,1659263,1659293,1659304,1659306-1659307,1659382,1659384,1659428,1659471,1659486,1659505,1659516,1659521,1659524,1659559,1659562,1659803,1659806,1659814,1659833,1659862,1659905,1659919,1659948,1659967,1659983-1659984,1660060,1660074,1660077,1660133,1660168,1660331-1660332,1660353,1660358,1660924,1661386,1661770,1661867,1661972,1661990,1662200,1662308-1662309,1662548,1662614,1662696,1662736,1662985,1662988-1662989,1663264,1663277,1663298,1663534,1663562,1663676,1663715,1663754,1663768,1663772,1663781,1663893,1663995,1664143,1664163,1664174,1664301,1664317,1664347,1664657,1664659,1664710,1664863-1664864,1664866,1665085,1665292,1665559,1665653,1665661,1665672,1665694,1665697,1665736,1665779,1665976-1665977,1665980-1665981,1665985-1665986,1665989,1665998,1666004,1666008,1666013,1666017,1666024,1666116,1666386-1
 
666387,1666494,1666496,1666552,1666569,1666579,137,149,1666757,1666966,1666972,1666985,1666995,1666997,1667292,1667402,1667406,1667546,1667615,1667630,1667636,1667688,1667764,1667871,1668026,1668135,1668193,1668593,1668596,1668630,1668639,1668843,1669353,1669370,1669451,1669800,1669838,1669876,1669882,1670394,1670433,1670591,1670598-1670600,1670610,1670631,1670719,1670724,1670726,1670730,1670940,1671112,1672272,1672284,1673754,1674294,1675461,1675486,1675594,1675830,1676231,1676250-1676251,1676364,1676381,1676393,1676479,1676525,1676552,1676615,1676630,1676634,1676721,1676926,1676943,1677140,1677802,1

svn commit: r1798540 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/catalina/valves/CrawlerSessionManagerValve.java

2017-06-12 Thread violetagg
Author: violetagg
Date: Mon Jun 12 21:22:41 2017
New Revision: 1798540

URL: http://svn.apache.org/viewvc?rev=1798540&view=rev
Log:
Format the code. No functional changes.

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

tomcat/tc7.0.x/trunk/java/org/apache/catalina/valves/CrawlerSessionManagerValve.java

Propchange: tomcat/tc7.0.x/trunk/
--
--- svn:mergeinfo (original)
+++ svn:mergeinfo Mon Jun 12 21:22:41 2017
@@ -1,3 +1,3 @@
 
/tomcat/tc8.0.x/trunk:1636525,1637336,1637685,1637709,1638726,1640089,1640276,1640349,1640363,1640366,1640642,1640672,1640674,1640689,1640884,1641001,1641065,1641067,1641375,1641638,1641723,1641726,1641729-1641730,1641736,1641988,1642669-1642670,1642698,1642701,1643205,1643215,1643217,1643230,1643232,1643273,1643285,1643329-1643330,1643511,1643513,1643521,1643539,1643571,1643581-1643582,1643635,1643655,1643738,1643964,1644018,1644333,1644954,1644992,1645014,1645360,1645456,1645627,1645642,1645686,1645903-1645904,1645908-1645909,1645913,1645920,1646458,1646460-1646462,1646735,1646738-1646741,1646744,1646746,1646748-1646755,1646757,1646759-1646760,1647043,1648816,1651420-1651422,1651844,1652926,1652939-1652940,1652973,1653798,1653817,1653841,1654042,1654161,1654736,1654767,1654787,1656592,1659907,1662986,1663265,1663278,1663325,1663535,1663567,1663679,1663997,1664175,1664321,1664872,1665061,1665086,1666027,1666395,1666503,1666506,1666560,1666570,1666581,1666759,1666967,1666988,1667553
 
-1667555,1667558,1667617,1667633,1667637,1667747,1667767,1667873,1668028,1668137,1668634,1669432,1669801,1669840,1669895-1669896,1670398,1670435,1670592,1670605-1670607,1670609,1670632,1670720,1670725,1670727,1670731,1671114,1672273,1672285,1673759,1674220,1674295,1675469,1675488,1675595,1675831,1676232,1676367-1676369,1676382,1676394,1676483,1676556,1676635,1678178,1679536,1679988,1680256,1681124,1681182,1681703,1681730,1681840,1681864,1681869,1682010,1682034,1682047,1682052-1682053,1682062,1682064,1682070,1682312,1682325,1682331,1682386,1684367,1684385,1685759,1685774,1685827,1685892,1687341,1688904,1689358,1689657,1689921,1692850,1693093,1693108,1693324,1694060,1694115,1694291,1694427,1694431,1694503,1694549,1694789,1694873,1694881,1695356,1695372,1695823-1695825,1696200,1696281,1696379,1696468,1700608,1700871,1700897,1700978,1701094,1701124,1701608,1701668,1701676,1701766,1701944,1702248,1702252,1702314,1702390,1702723,1702725,1702728,1702730,1702733,1702735,1702737,1702739,1702
 
742,1702744,1702748,1702751,1702754,1702758,1702760,1702763,1702766,1708779,1708782,1708806,1709314,1709670,1710347,1710442,1710448,1710490,1710574,1710578,1712226,1712229,1712235,1712255,1712618,1712649,1712655,1712860,1712899,1712903,1712906,1712913,1712926,1712975,1713185,1713262,1713287,1713613,1713621,1713872,1713976,1713994,1713998,1714004,1714013,1714059,1714538,1714580,1715189,1715207,1715544,1715549,1715637,1715639-1715645,1715667,1715683,1715866,1715978,1715981,1716216-1716217,1716355,1716414,1716421,1717208-1717209,1717257,1717283,1717288,1717291,1717421,1717517,1717529,1718797,1718840-1718843,1719348,1719357-1719358,1719400,1719491,1719737,1720235,1720396,1720442,1720446,1720450,1720463,1720658-1720660,1720756,1720816,1721813,1721818,1721831,1721861,1721867,1721882,1722523,1722527,1722800,1722926,1722941,1722997,1723130,1723440,1723488,1723890,1724434,1724674,1724792,1724803,1724902,1725128,1725131,1725154,1725167,1725911,1725921,1725929,1725963-1725965,1725970,1725974,1
 
726171-1726173,1726175,1726179-1726182,1726190-1726191,1726195-1726200,1726203,1726226,1726576,1726630,1726992,1727029,1727037,1727671,1727676,1727900,1728028,1728092,1728439,1728449,1729186,1729362,1731009,1731303,1731867,1731872,1731874,1731876,1731885,1731947,1731955,1731959,1731977,1731984,1732360,1732490,1732672,1732902,1733166,1733603,1733619,1733735,1733752,1733764,1733915,1733941,1733964,1734115,1734133,1734261,1734421,1734531,1736286,1737967,1738173,1738182,1738992,1739039,1739089-1739091,1739294,1739777,1739821,1739981,1740513,1740726,1741019,1741162,1741217,1743647,1743681,1744152,1744272,1746732,1746750,1752739,1754615,1755886,1756018,1759565,1761686,1762173,1762206,1766280,1767507-1767508,1767653,1767656,1769267,1772949,1773521,1773527,1774104,1777015,1777213,1779330,1783151,1784188,1784966,1785670,1786846,1788260,1788999,1789140,1789402,1791529,1791559,1795291,1796906,1797523
 
/tomcat/tc8.5.x/trunk:1735579,1736839,1737199,1737966,1738042,1738044,1738162,1738165,1738178,1739157,1739173,1739177,1739476,1740132,1740521,1740536,1740804,1740811,1740981,1741165,1741174,1741182,1741191,1741203,1741209,1741226,1741233,1741410,1742277,1743118,1743126,1743139-1743140,1743718,1743722,1743724,1744059,1744127,1744151,1744232,1744377,1744687,1744698,1744706,1745228,1746940,1748548,1748716,1749288,1749375,1749668-1749669,1750016,1750057,1750976,1751000,1751062,1751098,1754112,1754144,1754282,1754312,1754614,1754726,1754806,1754878,1754889,1754894,1754900

buildbot failure in on tomcat-8-trunk

2017-06-12 Thread buildbot
The Buildbot has detected a new failure on builder tomcat-8-trunk while 
building . Full details are available at:
https://ci.apache.org/builders/tomcat-8-trunk/builds/1034

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

Buildslave for this Build: silvanus_ubuntu

Build Reason: The AnyBranchScheduler scheduler named 'on-tomcat-8-commit' 
triggered this build
Build Source Stamp: [branch tomcat/tc8.0.x/trunk] 1798539
Blamelist: violetagg

BUILD FAILED: failed compile_1

Sincerely,
 -The Buildbot




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



svn commit: r1798541 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/catalina/valves/CrawlerSessionManagerValve.java test/org/apache/catalina/valves/TestCrawlerSessionManagerValve.java webapps/docs/cha

2017-06-12 Thread violetagg
Author: violetagg
Date: Mon Jun 12 21:25:26 2017
New Revision: 1798541

URL: http://svn.apache.org/viewvc?rev=1798541&view=rev
Log:
A new configuration property 'crawlerIps' is added to the 
'o.a.catalina.valves.CrawlerSessionManagerValve'. Using this property one can 
specify a regular expression that will be used to identify crawlers based on 
their IP address. Based on a patch provided by Tetradeus.

This closes #3

Added:

tomcat/tc7.0.x/trunk/test/org/apache/catalina/valves/TestCrawlerSessionManagerValve.java
  - copied unchanged from r1798533, 
tomcat/trunk/test/org/apache/catalina/valves/TestCrawlerSessionManagerValve.java
Modified:
tomcat/tc7.0.x/trunk/   (props changed)

tomcat/tc7.0.x/trunk/java/org/apache/catalina/valves/CrawlerSessionManagerValve.java
tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml
tomcat/tc7.0.x/trunk/webapps/docs/config/valve.xml

Propchange: tomcat/tc7.0.x/trunk/
--
--- svn:mergeinfo (original)
+++ svn:mergeinfo Mon Jun 12 21:25:26 2017
@@ -1,3 +1,3 @@
 
/tomcat/tc8.0.x/trunk:1636525,1637336,1637685,1637709,1638726,1640089,1640276,1640349,1640363,1640366,1640642,1640672,1640674,1640689,1640884,1641001,1641065,1641067,1641375,1641638,1641723,1641726,1641729-1641730,1641736,1641988,1642669-1642670,1642698,1642701,1643205,1643215,1643217,1643230,1643232,1643273,1643285,1643329-1643330,1643511,1643513,1643521,1643539,1643571,1643581-1643582,1643635,1643655,1643738,1643964,1644018,1644333,1644954,1644992,1645014,1645360,1645456,1645627,1645642,1645686,1645903-1645904,1645908-1645909,1645913,1645920,1646458,1646460-1646462,1646735,1646738-1646741,1646744,1646746,1646748-1646755,1646757,1646759-1646760,1647043,1648816,1651420-1651422,1651844,1652926,1652939-1652940,1652973,1653798,1653817,1653841,1654042,1654161,1654736,1654767,1654787,1656592,1659907,1662986,1663265,1663278,1663325,1663535,1663567,1663679,1663997,1664175,1664321,1664872,1665061,1665086,1666027,1666395,1666503,1666506,1666560,1666570,1666581,1666759,1666967,1666988,1667553
 
-1667555,1667558,1667617,1667633,1667637,1667747,1667767,1667873,1668028,1668137,1668634,1669432,1669801,1669840,1669895-1669896,1670398,1670435,1670592,1670605-1670607,1670609,1670632,1670720,1670725,1670727,1670731,1671114,1672273,1672285,1673759,1674220,1674295,1675469,1675488,1675595,1675831,1676232,1676367-1676369,1676382,1676394,1676483,1676556,1676635,1678178,1679536,1679988,1680256,1681124,1681182,1681703,1681730,1681840,1681864,1681869,1682010,1682034,1682047,1682052-1682053,1682062,1682064,1682070,1682312,1682325,1682331,1682386,1684367,1684385,1685759,1685774,1685827,1685892,1687341,1688904,1689358,1689657,1689921,1692850,1693093,1693108,1693324,1694060,1694115,1694291,1694427,1694431,1694503,1694549,1694789,1694873,1694881,1695356,1695372,1695823-1695825,1696200,1696281,1696379,1696468,1700608,1700871,1700897,1700978,1701094,1701124,1701608,1701668,1701676,1701766,1701944,1702248,1702252,1702314,1702390,1702723,1702725,1702728,1702730,1702733,1702735,1702737,1702739,1702
 
742,1702744,1702748,1702751,1702754,1702758,1702760,1702763,1702766,1708779,1708782,1708806,1709314,1709670,1710347,1710442,1710448,1710490,1710574,1710578,1712226,1712229,1712235,1712255,1712618,1712649,1712655,1712860,1712899,1712903,1712906,1712913,1712926,1712975,1713185,1713262,1713287,1713613,1713621,1713872,1713976,1713994,1713998,1714004,1714013,1714059,1714538,1714580,1715189,1715207,1715544,1715549,1715637,1715639-1715645,1715667,1715683,1715866,1715978,1715981,1716216-1716217,1716355,1716414,1716421,1717208-1717209,1717257,1717283,1717288,1717291,1717421,1717517,1717529,1718797,1718840-1718843,1719348,1719357-1719358,1719400,1719491,1719737,1720235,1720396,1720442,1720446,1720450,1720463,1720658-1720660,1720756,1720816,1721813,1721818,1721831,1721861,1721867,1721882,1722523,1722527,1722800,1722926,1722941,1722997,1723130,1723440,1723488,1723890,1724434,1724674,1724792,1724803,1724902,1725128,1725131,1725154,1725167,1725911,1725921,1725929,1725963-1725965,1725970,1725974,1
 
726171-1726173,1726175,1726179-1726182,1726190-1726191,1726195-1726200,1726203,1726226,1726576,1726630,1726992,1727029,1727037,1727671,1727676,1727900,1728028,1728092,1728439,1728449,1729186,1729362,1731009,1731303,1731867,1731872,1731874,1731876,1731885,1731947,1731955,1731959,1731977,1731984,1732360,1732490,1732672,1732902,1733166,1733603,1733619,1733735,1733752,1733764,1733915,1733941,1733964,1734115,1734133,1734261,1734421,1734531,1736286,1737967,1738173,1738182,1738992,1739039,1739089-1739091,1739294,1739777,1739821,1739981,1740513,1740726,1741019,1741162,1741217,1743647,1743681,1744152,1744272,1746732,1746750,1752739,1754615,1755886,1756018,1759565,1761686,1762173,1762206,1766280,1767507-1767508,1767653,1767656,1769267,1772949,1773521,1773527,1774104,1777015,1777213,1779330,1783151,1784188,1784966,1785670,1786846,1788260,1788999,1789140,1789402,1791529,1791559,1795291,1796906,179

svn commit: r1798543 - /tomcat/tc8.5.x/trunk/test/org/apache/catalina/valves/TestCrawlerSessionManagerValve.java

2017-06-12 Thread violetagg
Author: violetagg
Date: Mon Jun 12 21:33:40 2017
New Revision: 1798543

URL: http://svn.apache.org/viewvc?rev=1798543&view=rev
Log:
Fix the build

Modified:

tomcat/tc8.5.x/trunk/test/org/apache/catalina/valves/TestCrawlerSessionManagerValve.java

Modified: 
tomcat/tc8.5.x/trunk/test/org/apache/catalina/valves/TestCrawlerSessionManagerValve.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc8.5.x/trunk/test/org/apache/catalina/valves/TestCrawlerSessionManagerValve.java?rev=1798543&r1=1798542&r2=1798543&view=diff
==
--- 
tomcat/tc8.5.x/trunk/test/org/apache/catalina/valves/TestCrawlerSessionManagerValve.java
 (original)
+++ 
tomcat/tc8.5.x/trunk/test/org/apache/catalina/valves/TestCrawlerSessionManagerValve.java
 Mon Jun 12 21:33:40 2017
@@ -80,7 +80,7 @@ public class TestCrawlerSessionManagerVa
 if (isBot) {
 setter.andReturn(session);
 }
-
EasyMock.expect(request.getHeaders("user-agent")).andReturn(Collections.emptyEnumeration());
+
EasyMock.expect(request.getHeaders("user-agent")).andReturn(Collections.emptyEnumeration());
 return request;
 }
 }



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



svn commit: r1798544 - in /tomcat/tc8.0.x/trunk: ./ test/org/apache/catalina/valves/TestCrawlerSessionManagerValve.java

2017-06-12 Thread violetagg
Author: violetagg
Date: Mon Jun 12 21:34:34 2017
New Revision: 1798544

URL: http://svn.apache.org/viewvc?rev=1798544&view=rev
Log:
Fix the build

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

tomcat/tc8.0.x/trunk/test/org/apache/catalina/valves/TestCrawlerSessionManagerValve.java

Propchange: tomcat/tc8.0.x/trunk/
--
--- svn:mergeinfo (original)
+++ svn:mergeinfo Mon Jun 12 21:34:34 2017
@@ -1,2 +1,2 @@
-/tomcat/tc8.5.x/trunk:1735042,1737966,1743139-1743140,1744151,1747537,1747925,1748002,1754614,1754643,1762124,1762183,1762203,1763792,1772948,1777014,1779719,1782037,1782240,1782386-1782387,1785669,1786845,1788249,1788324,1788905,1789216,1789335,1791528,1791558,1796697-1796698,1797521
+/tomcat/tc8.5.x/trunk:1735042,1737966,1743139-1743140,1744151,1747537,1747925,1748002,1754614,1754643,1762124,1762183,1762203,1763792,1772948,1777014,1779719,1782037,1782240,1782386-1782387,1785669,1786845,1788249,1788324,1788905,1789216,1789335,1791528,1791558,1796697-1796698,1797521,1798543
 
/tomcat/trunk:1636524,1637156,1637176,1637188,1637331,1637684,1637695,1637890,1637892,1638720-1638725,1639653,1640010,1640083-1640084,1640088,1640275,1640322,1640347,1640361,1640365,1640403,1640410,1640652,1640655-1640658,1640688,1640700-1640883,1640903,1640976,1640978,1641000,1641026,1641038-1641039,1641051-1641052,1641058,1641064,1641300,1641369,1641374,1641380,1641486,1641634,1641656-1641692,1641704,1641707-1641718,1641720-1641722,1641735,1641981,1642233,1642280,1642554,1642564,1642595,1642606,1642668,1642679,1642697,1642699,1642766,1643002,1643045,1643054-1643055,1643066,1643121,1643128,1643206,1643209-1643210,1643216,1643249,1643270,1643283,1643309-1643310,1643323,1643365-1643366,1643370-1643371,1643465,1643474,1643536,1643570,1643634,1643649,1643651,1643654,1643675,1643731,1643733-1643734,1643761,1643766,1643814,1643937,1643963,1644017,1644169,1644201-1644203,1644321,1644323,1644516,1644523,1644529,1644535,1644730,1644768,1644784-1644785,1644790,1644793,1644815,1644884,1644886
 
,1644890,1644892,1644910,1644924,1644929-1644930,1644935,1644989,1645011,1645247,1645355,1645357-1645358,1645455,1645465,1645469,1645471,1645473,1645475,1645486-1645488,1645626,1645641,1645685,1645743,1645763,1645951-1645953,1645955,1645993,1646098-1646106,1646178,1646220,1646302,1646304,1646420,1646470-1646471,1646476,1646559,1646717-1646723,1646773,1647026,1647042,1647530,1647655,1648304,1648815,1648907,1649973,1650081,1650365,1651116,1651120,1651280,1651470,1652938,1652970,1653041,1653471,1653550,1653574,1653797,1653815-1653816,1653819,1653840,1653857,1653888,1653972,1654013,1654030,1654050,1654123,1654148,1654159,1654513,1654515,1654517,1654522,1654524,1654725,1654735,1654766,1654785,1654851-1654852,1654978,1655122-1655124,1655126-1655127,1655129-1655130,1655132-1655133,1655312,1655351,1655438,1655441,1655454,168,1656087,1656299,1656319,1656331,1656345,1656350,1656590,1656648-1656650,1656657,1657041,1657054,1657374,1657492,1657510,1657565,1657580,1657584,1657586,1657589,1657
 
592,1657607,1657609,1657682,1657907,1658207,1658734,1658781,1658790,1658799,1658802,1658804,1658833,1658840,1658966,1659043,1659053,1659059,1659174,1659184,1659188-1659189,1659216,1659263,1659293,1659304,1659306-1659307,1659382,1659384,1659428,1659471,1659486,1659505,1659516,1659521,1659524,1659559,1659562,1659803,1659806,1659814,1659833,1659862,1659905,1659919,1659948,1659967,1659983-1659984,1660060,1660074,1660077,1660133,1660168,1660331-1660332,1660353,1660358,1660924,1661386,1661770,1661867,1661972,1661990,1662200,1662308-1662309,1662548,1662614,1662696,1662736,1662985,1662988-1662989,1663264,1663277,1663298,1663534,1663562,1663676,1663715,1663754,1663768,1663772,1663781,1663893,1663995,1664143,1664163,1664174,1664301,1664317,1664347,1664657,1664659,1664710,1664863-1664864,1664866,1665085,1665292,1665559,1665653,1665661,1665672,1665694,1665697,1665736,1665779,1665976-1665977,1665980-1665981,1665985-1665986,1665989,1665998,1666004,1666008,1666013,1666017,1666024,1666116,1666386-1
 
666387,1666494,1666496,1666552,1666569,1666579,137,149,1666757,1666966,1666972,1666985,1666995,1666997,1667292,1667402,1667406,1667546,1667615,1667630,1667636,1667688,1667764,1667871,1668026,1668135,1668193,1668593,1668596,1668630,1668639,1668843,1669353,1669370,1669451,1669800,1669838,1669876,1669882,1670394,1670433,1670591,1670598-1670600,1670610,1670631,1670719,1670724,1670726,1670730,1670940,1671112,1672272,1672284,1673754,1674294,1675461,1675486,1675594,1675830,1676231,1676250-1676251,1676364,1676381,1676393,1676479,1676525,1676552,1676615,1676630,1676634,1676721,1676926,1676943,1677140,1677802,1678011,1678162,1678174,1678339,1678426-1678427,1678694,1678701,1679534,1679708,1679710,1679716,1680034,1680246,1681056,1681123,1681138,1681280,1681283,1681286,1681450,1681697,1681699,1681701,1681729,1681770,1681779,1681793,1681807,1681837-1681838,1681854,1681862,1681958,1682028,1682033,1682311,1682315

buildbot failure in on tomcat-7-trunk

2017-06-12 Thread buildbot
The Buildbot has detected a new failure on builder tomcat-7-trunk while 
building . Full details are available at:
https://ci.apache.org/builders/tomcat-7-trunk/builds/765

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

Buildslave for this Build: silvanus_ubuntu

Build Reason: The AnyBranchScheduler scheduler named 'on-tomcat-7-commit' 
triggered this build
Build Source Stamp: [branch tomcat/tc7.0.x/trunk] 1798541
Blamelist: violetagg

BUILD FAILED: failed compile_1

Sincerely,
 -The Buildbot




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



svn commit: r1798545 - in /tomcat/tc7.0.x/trunk: ./ test/org/apache/catalina/valves/TestCrawlerSessionManagerValve.java

2017-06-12 Thread violetagg
Author: violetagg
Date: Mon Jun 12 21:37:11 2017
New Revision: 1798545

URL: http://svn.apache.org/viewvc?rev=1798545&view=rev
Log:
Fix the build

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

tomcat/tc7.0.x/trunk/test/org/apache/catalina/valves/TestCrawlerSessionManagerValve.java

Propchange: tomcat/tc7.0.x/trunk/
--
--- svn:mergeinfo (original)
+++ svn:mergeinfo Mon Jun 12 21:37:11 2017
@@ -1,3 +1,3 @@
 
/tomcat/tc8.0.x/trunk:1636525,1637336,1637685,1637709,1638726,1640089,1640276,1640349,1640363,1640366,1640642,1640672,1640674,1640689,1640884,1641001,1641065,1641067,1641375,1641638,1641723,1641726,1641729-1641730,1641736,1641988,1642669-1642670,1642698,1642701,1643205,1643215,1643217,1643230,1643232,1643273,1643285,1643329-1643330,1643511,1643513,1643521,1643539,1643571,1643581-1643582,1643635,1643655,1643738,1643964,1644018,1644333,1644954,1644992,1645014,1645360,1645456,1645627,1645642,1645686,1645903-1645904,1645908-1645909,1645913,1645920,1646458,1646460-1646462,1646735,1646738-1646741,1646744,1646746,1646748-1646755,1646757,1646759-1646760,1647043,1648816,1651420-1651422,1651844,1652926,1652939-1652940,1652973,1653798,1653817,1653841,1654042,1654161,1654736,1654767,1654787,1656592,1659907,1662986,1663265,1663278,1663325,1663535,1663567,1663679,1663997,1664175,1664321,1664872,1665061,1665086,1666027,1666395,1666503,1666506,1666560,1666570,1666581,1666759,1666967,1666988,1667553
 
-1667555,1667558,1667617,1667633,1667637,1667747,1667767,1667873,1668028,1668137,1668634,1669432,1669801,1669840,1669895-1669896,1670398,1670435,1670592,1670605-1670607,1670609,1670632,1670720,1670725,1670727,1670731,1671114,1672273,1672285,1673759,1674220,1674295,1675469,1675488,1675595,1675831,1676232,1676367-1676369,1676382,1676394,1676483,1676556,1676635,1678178,1679536,1679988,1680256,1681124,1681182,1681703,1681730,1681840,1681864,1681869,1682010,1682034,1682047,1682052-1682053,1682062,1682064,1682070,1682312,1682325,1682331,1682386,1684367,1684385,1685759,1685774,1685827,1685892,1687341,1688904,1689358,1689657,1689921,1692850,1693093,1693108,1693324,1694060,1694115,1694291,1694427,1694431,1694503,1694549,1694789,1694873,1694881,1695356,1695372,1695823-1695825,1696200,1696281,1696379,1696468,1700608,1700871,1700897,1700978,1701094,1701124,1701608,1701668,1701676,1701766,1701944,1702248,1702252,1702314,1702390,1702723,1702725,1702728,1702730,1702733,1702735,1702737,1702739,1702
 
742,1702744,1702748,1702751,1702754,1702758,1702760,1702763,1702766,1708779,1708782,1708806,1709314,1709670,1710347,1710442,1710448,1710490,1710574,1710578,1712226,1712229,1712235,1712255,1712618,1712649,1712655,1712860,1712899,1712903,1712906,1712913,1712926,1712975,1713185,1713262,1713287,1713613,1713621,1713872,1713976,1713994,1713998,1714004,1714013,1714059,1714538,1714580,1715189,1715207,1715544,1715549,1715637,1715639-1715645,1715667,1715683,1715866,1715978,1715981,1716216-1716217,1716355,1716414,1716421,1717208-1717209,1717257,1717283,1717288,1717291,1717421,1717517,1717529,1718797,1718840-1718843,1719348,1719357-1719358,1719400,1719491,1719737,1720235,1720396,1720442,1720446,1720450,1720463,1720658-1720660,1720756,1720816,1721813,1721818,1721831,1721861,1721867,1721882,1722523,1722527,1722800,1722926,1722941,1722997,1723130,1723440,1723488,1723890,1724434,1724674,1724792,1724803,1724902,1725128,1725131,1725154,1725167,1725911,1725921,1725929,1725963-1725965,1725970,1725974,1
 
726171-1726173,1726175,1726179-1726182,1726190-1726191,1726195-1726200,1726203,1726226,1726576,1726630,1726992,1727029,1727037,1727671,1727676,1727900,1728028,1728092,1728439,1728449,1729186,1729362,1731009,1731303,1731867,1731872,1731874,1731876,1731885,1731947,1731955,1731959,1731977,1731984,1732360,1732490,1732672,1732902,1733166,1733603,1733619,1733735,1733752,1733764,1733915,1733941,1733964,1734115,1734133,1734261,1734421,1734531,1736286,1737967,1738173,1738182,1738992,1739039,1739089-1739091,1739294,1739777,1739821,1739981,1740513,1740726,1741019,1741162,1741217,1743647,1743681,1744152,1744272,1746732,1746750,1752739,1754615,1755886,1756018,1759565,1761686,1762173,1762206,1766280,1767507-1767508,1767653,1767656,1769267,1772949,1773521,1773527,1774104,1777015,1777213,1779330,1783151,1784188,1784966,1785670,1786846,1788260,1788999,1789140,1789402,1791529,1791559,1795291,1796906,1797523
-/tomcat/tc8.5.x/trunk:1735579,1736839,1737199,1737966,1738042,1738044,1738162,1738165,1738178,1739157,1739173,1739177,1739476,1740132,1740521,1740536,1740804,1740811,1740981,1741165,1741174,1741182,1741191,1741203,1741209,1741226,1741233,1741410,1742277,1743118,1743126,1743139-1743140,1743718,1743722,1743724,1744059,1744127,1744151,1744232,1744377,1744687,1744698,1744706,1745228,1746940,1748548,1748716,1749288,1749375,1749668-1749669,1750016,1750057,1750976,1751000,1751062,1751098,1754112,1754144,1754282,1754312,1754614,1754726,1754806,1754878,1754889,1754894,1754900,1754945,1754954,175495

svn commit: r1798546 - /tomcat/trunk/res/ide-support/eclipse/eclipse.classpath

2017-06-12 Thread violetagg
Author: violetagg
Date: Mon Jun 12 21:44:12 2017
New Revision: 1798546

URL: http://svn.apache.org/viewvc?rev=1798546&view=rev
Log:
When using EasyMock for mocking classes, one needs cglib and objenesis in the 
class path.

Modified:
tomcat/trunk/res/ide-support/eclipse/eclipse.classpath

Modified: tomcat/trunk/res/ide-support/eclipse/eclipse.classpath
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/res/ide-support/eclipse/eclipse.classpath?rev=1798546&r1=1798545&r2=1798546&view=diff
==
--- tomcat/trunk/res/ide-support/eclipse/eclipse.classpath (original)
+++ tomcat/trunk/res/ide-support/eclipse/eclipse.classpath Mon Jun 12 21:44:12 
2017
@@ -27,5 +27,7 @@
 
 
 
+
+
 
 



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



svn commit: r1798547 - in /tomcat/tc8.5.x/trunk: ./ res/ide-support/eclipse/eclipse.classpath

2017-06-12 Thread violetagg
Author: violetagg
Date: Mon Jun 12 21:46:27 2017
New Revision: 1798547

URL: http://svn.apache.org/viewvc?rev=1798547&view=rev
Log:
When using EasyMock for mocking classes, one needs cglib and objenesis in the 
class path.

Modified:
tomcat/tc8.5.x/trunk/   (props changed)
tomcat/tc8.5.x/trunk/res/ide-support/eclipse/eclipse.classpath

Propchange: tomcat/tc8.5.x/trunk/
--
--- svn:mergeinfo (original)
+++ svn:mergeinfo Mon Jun 12 21:46:27 2017
@@ -1 +1 @@
-/tomcat/trunk:1734785,1734799,1734845,1734928,1735041,1735044,1735480,1735577,1735597,1735599-1735600,1735615,1736145,1736162,1736209,1736280,1736297,1736299,1736489,1736646,1736703,1736836,1736849,1737104-1737105,1737112,1737117,1737119-1737120,1737155,1737157,1737192,1737280,1737339,1737632,1737664,1737715,1737748,1737785,1737834,1737860,1737903,1737959,1738005,1738007,1738014-1738015,1738018,1738022,1738039,1738043,1738059-1738060,1738147,1738149,1738174-1738175,1738261,1738589,1738623-1738625,1738643,1738816,1738850,1738855,1738946-1738948,1738953-1738954,1738979,1738982,1739079-1739081,1739087,1739113,1739153,1739172,1739176,1739191,1739474,1739726,1739762,1739775,1739814,1739817-1739818,1739975,1740131,1740324,1740465,1740495,1740508-1740509,1740520,1740535,1740707,1740803,1740810,1740969,1740980,1740991,1740997,1741015,1741033,1741036,1741058,1741060,1741080,1741147,1741159,1741164,1741173,1741181,1741190,1741197,1741202,1741208,1741213,1741221,1741225,1741232,1741409,1741501
 
,1741677,1741892,1741896,1741984,1742023,1742042,1742071,1742090,1742093,1742101,1742105,1742111,1742139,1742146,1742148,1742166,1742181,1742184,1742187,1742246,1742248-1742251,1742263-1742264,1742268,1742276,1742369,1742387,1742448,1742509-1742512,1742917,1742919,1742933,1742975-1742976,1742984,1742986,1743019,1743115,1743117,1743124-1743125,1743134,1743425,1743554,1743679,1743696-1743698,1743700-1743701,1744058,1744064-1744065,1744125,1744194,1744229,1744270,1744323,1744432,1744684,1744697,1744705,1744713,1744760,1744786,1745083,1745142-1745143,1745145,1745177,1745179-1745180,1745227,1745248,1745254,1745337,1745467,1745473,1745535,1745576,1745735,1745744,1746304,1746306-1746307,1746319,1746327,1746338,1746340-1746341,1746344,1746427,1746441,1746473,1746490,1746492,1746495-1746496,1746499-1746501,1746503-1746507,1746509,1746549,1746551,1746554,1746556,1746558,1746584,1746620,1746649,1746724,1746939,1746989,1747014,1747028,1747035,1747210,1747225,1747234,1747253,1747404,1747506,1747
 
536,1747924,1747980,1747993,1748001,1748253,1748452,1748547,1748629,1748676,1748715,1749287,1749296,1749328,1749373,1749465,1749506,1749508,1749665-1749666,1749763,1749865-1749866,1749898,1749978,1749980,1750011,1750015,1750056,1750480,1750617,1750634,1750692,1750697,1750700,1750703,1750707,1750714,1750718,1750723,1750774,1750899,1750975,1750995,1751061,1751097,1751173,1751438,1751447,1751463,1751702,1752212,1752737,1752745,1753078,1753080,1753358,1753363,1754111,1754140-1754141,1754281,1754310,1754445,1754467,1754494,1754496,1754528,1754532-1754533,1754613,1754714,1754874,1754941,1754944,1754950-1754951,1755005,1755007,1755009,1755132,1755180-1755181,1755185,1755190,1755204-1755206,1755208,1755214,1755224,1755227,1755230,1755629,1755646-1755647,1755650,1755653,1755675,1755680,1755683,1755693,1755717,1755731-1755737,1755812,1755828,1755884,1755890,1755918-1755919,1755942,1755958,1755960,1755970,1755993,1756013,1756019,1756039,1756056,1756083-1756114,1756175,1756288-1756289,1756408-1
 
756410,1756778,1756798,1756878,1756898,1756939,1757123-1757124,1757126,1757128,1757132-1757133,1757136,1757145,1757167-1757168,1757175,1757180,1757182,1757195,1757271,1757278,1757347,1757353-1757354,1757363,1757374,1757399,1757406,1757408,1757485,1757495,1757499,1757527,1757578,1757684,1757722,1757727,1757790,1757799,1757813,1757853,1757883,1757903,1757976,1757997,1758000,1758058,1758072-1758075,1758078-1758079,1758223,1758257,1758261,1758276,1758292,1758369,1758378-1758383,1758421,1758423,1758425-1758427,1758430,1758443,1758448,1758459,1758483,1758486-1758487,1758499,1758525,1758556,1758580,1758582,1758584,1758588,1758842,1759019,1759212,1759224,1759227,1759252,1759274,1759513-1759516,1759611,1759757,1759785-1759790,1760005,1760022,1760109-1760110,1760135,1760200-1760201,1760227,1760300,1760397,1760446,1760454,1760640,1760648,1761057,1761422,1761491,1761498,1761500-1761501,1761550,1761553,1761572,1761574,1761625-1761626,1761628,1761682,1761740,1761752,1762051-1762053,1762123,176216
 
8,1762172,1762182,1762201-1762202,1762204,1762208,1762288,1762296,1762324,1762348,1762353,1762362,1762374,1762492,1762503,1762505,1762541,1762608,1762710,1762753,1762766,1762769,1762944,1762947,1762953,1763167,1763179,1763232,1763259,1763271-1763272,1763276-1763277,1763319-1763320,1763370,1763372,1763375,1763377,1763393,1763412,1763430,1763450,1763462,1763505,1763511-1763512,1763516,1763518,1763520,1763529,1763559,1763565,1763568,1763574,1763619,17636

svn commit: r1798548 - in /tomcat/tc8.0.x/trunk: ./ res/ide-support/eclipse/eclipse.classpath

2017-06-12 Thread violetagg
Author: violetagg
Date: Mon Jun 12 21:47:55 2017
New Revision: 1798548

URL: http://svn.apache.org/viewvc?rev=1798548&view=rev
Log:
When using EasyMock for mocking classes, one needs cglib and objenesis in the 
class path.

Modified:
tomcat/tc8.0.x/trunk/   (props changed)
tomcat/tc8.0.x/trunk/res/ide-support/eclipse/eclipse.classpath

Propchange: tomcat/tc8.0.x/trunk/
--
--- svn:mergeinfo (original)
+++ svn:mergeinfo Mon Jun 12 21:47:55 2017
@@ -1,2 +1,2 @@
 
/tomcat/tc8.5.x/trunk:1735042,1737966,1743139-1743140,1744151,1747537,1747925,1748002,1754614,1754643,1762124,1762183,1762203,1763792,1772948,1777014,1779719,1782037,1782240,1782386-1782387,1785669,1786845,1788249,1788324,1788905,1789216,1789335,1791528,1791558,1796697-1796698,1797521,1798543
-/tomcat/trunk:1636524,1637156,1637176,1637188,1637331,1637684,1637695,1637890,1637892,1638720-1638725,1639653,1640010,1640083-1640084,1640088,1640275,1640322,1640347,1640361,1640365,1640403,1640410,1640652,1640655-1640658,1640688,1640700-1640883,1640903,1640976,1640978,1641000,1641026,1641038-1641039,1641051-1641052,1641058,1641064,1641300,1641369,1641374,1641380,1641486,1641634,1641656-1641692,1641704,1641707-1641718,1641720-1641722,1641735,1641981,1642233,1642280,1642554,1642564,1642595,1642606,1642668,1642679,1642697,1642699,1642766,1643002,1643045,1643054-1643055,1643066,1643121,1643128,1643206,1643209-1643210,1643216,1643249,1643270,1643283,1643309-1643310,1643323,1643365-1643366,1643370-1643371,1643465,1643474,1643536,1643570,1643634,1643649,1643651,1643654,1643675,1643731,1643733-1643734,1643761,1643766,1643814,1643937,1643963,1644017,1644169,1644201-1644203,1644321,1644323,1644516,1644523,1644529,1644535,1644730,1644768,1644784-1644785,1644790,1644793,1644815,1644884,1644886
 
,1644890,1644892,1644910,1644924,1644929-1644930,1644935,1644989,1645011,1645247,1645355,1645357-1645358,1645455,1645465,1645469,1645471,1645473,1645475,1645486-1645488,1645626,1645641,1645685,1645743,1645763,1645951-1645953,1645955,1645993,1646098-1646106,1646178,1646220,1646302,1646304,1646420,1646470-1646471,1646476,1646559,1646717-1646723,1646773,1647026,1647042,1647530,1647655,1648304,1648815,1648907,1649973,1650081,1650365,1651116,1651120,1651280,1651470,1652938,1652970,1653041,1653471,1653550,1653574,1653797,1653815-1653816,1653819,1653840,1653857,1653888,1653972,1654013,1654030,1654050,1654123,1654148,1654159,1654513,1654515,1654517,1654522,1654524,1654725,1654735,1654766,1654785,1654851-1654852,1654978,1655122-1655124,1655126-1655127,1655129-1655130,1655132-1655133,1655312,1655351,1655438,1655441,1655454,168,1656087,1656299,1656319,1656331,1656345,1656350,1656590,1656648-1656650,1656657,1657041,1657054,1657374,1657492,1657510,1657565,1657580,1657584,1657586,1657589,1657
 
592,1657607,1657609,1657682,1657907,1658207,1658734,1658781,1658790,1658799,1658802,1658804,1658833,1658840,1658966,1659043,1659053,1659059,1659174,1659184,1659188-1659189,1659216,1659263,1659293,1659304,1659306-1659307,1659382,1659384,1659428,1659471,1659486,1659505,1659516,1659521,1659524,1659559,1659562,1659803,1659806,1659814,1659833,1659862,1659905,1659919,1659948,1659967,1659983-1659984,1660060,1660074,1660077,1660133,1660168,1660331-1660332,1660353,1660358,1660924,1661386,1661770,1661867,1661972,1661990,1662200,1662308-1662309,1662548,1662614,1662696,1662736,1662985,1662988-1662989,1663264,1663277,1663298,1663534,1663562,1663676,1663715,1663754,1663768,1663772,1663781,1663893,1663995,1664143,1664163,1664174,1664301,1664317,1664347,1664657,1664659,1664710,1664863-1664864,1664866,1665085,1665292,1665559,1665653,1665661,1665672,1665694,1665697,1665736,1665779,1665976-1665977,1665980-1665981,1665985-1665986,1665989,1665998,1666004,1666008,1666013,1666017,1666024,1666116,1666386-1
 
666387,1666494,1666496,1666552,1666569,1666579,137,149,1666757,1666966,1666972,1666985,1666995,1666997,1667292,1667402,1667406,1667546,1667615,1667630,1667636,1667688,1667764,1667871,1668026,1668135,1668193,1668593,1668596,1668630,1668639,1668843,1669353,1669370,1669451,1669800,1669838,1669876,1669882,1670394,1670433,1670591,1670598-1670600,1670610,1670631,1670719,1670724,1670726,1670730,1670940,1671112,1672272,1672284,1673754,1674294,1675461,1675486,1675594,1675830,1676231,1676250-1676251,1676364,1676381,1676393,1676479,1676525,1676552,1676615,1676630,1676634,1676721,1676926,1676943,1677140,1677802,1678011,1678162,1678174,1678339,1678426-1678427,1678694,1678701,1679534,1679708,1679710,1679716,1680034,1680246,1681056,1681123,1681138,1681280,1681283,1681286,1681450,1681697,1681699,1681701,1681729,1681770,1681779,1681793,1681807,1681837-1681838,1681854,1681862,1681958,1682028,1682033,1682311,1682315,1682317,1682320,1682324,1682330,1682842,1684172,1684366,1684383,1684526-168452
 
7,1684549-1684550,1685556,1685591,1685739,1685744,1685772,1685816,1685826,1685891,1687242,1687261,1687268,1687340,1687544,1687551,1688563,1688841,1688878,1

svn commit: r1798549 - in /tomcat/tc7.0.x/trunk: ./ res/ide-support/eclipse/eclipse.classpath

2017-06-12 Thread violetagg
Author: violetagg
Date: Mon Jun 12 21:49:13 2017
New Revision: 1798549

URL: http://svn.apache.org/viewvc?rev=1798549&view=rev
Log:
When using EasyMock for mocking classes, one needs cglib and objenesis in the 
class path.

Modified:
tomcat/tc7.0.x/trunk/   (props changed)
tomcat/tc7.0.x/trunk/res/ide-support/eclipse/eclipse.classpath

Propchange: tomcat/tc7.0.x/trunk/
--
--- svn:mergeinfo (original)
+++ svn:mergeinfo Mon Jun 12 21:49:13 2017
@@ -1,3 +1,3 @@
 
/tomcat/tc8.0.x/trunk:1636525,1637336,1637685,1637709,1638726,1640089,1640276,1640349,1640363,1640366,1640642,1640672,1640674,1640689,1640884,1641001,1641065,1641067,1641375,1641638,1641723,1641726,1641729-1641730,1641736,1641988,1642669-1642670,1642698,1642701,1643205,1643215,1643217,1643230,1643232,1643273,1643285,1643329-1643330,1643511,1643513,1643521,1643539,1643571,1643581-1643582,1643635,1643655,1643738,1643964,1644018,1644333,1644954,1644992,1645014,1645360,1645456,1645627,1645642,1645686,1645903-1645904,1645908-1645909,1645913,1645920,1646458,1646460-1646462,1646735,1646738-1646741,1646744,1646746,1646748-1646755,1646757,1646759-1646760,1647043,1648816,1651420-1651422,1651844,1652926,1652939-1652940,1652973,1653798,1653817,1653841,1654042,1654161,1654736,1654767,1654787,1656592,1659907,1662986,1663265,1663278,1663325,1663535,1663567,1663679,1663997,1664175,1664321,1664872,1665061,1665086,1666027,1666395,1666503,1666506,1666560,1666570,1666581,1666759,1666967,1666988,1667553
 
-1667555,1667558,1667617,1667633,1667637,1667747,1667767,1667873,1668028,1668137,1668634,1669432,1669801,1669840,1669895-1669896,1670398,1670435,1670592,1670605-1670607,1670609,1670632,1670720,1670725,1670727,1670731,1671114,1672273,1672285,1673759,1674220,1674295,1675469,1675488,1675595,1675831,1676232,1676367-1676369,1676382,1676394,1676483,1676556,1676635,1678178,1679536,1679988,1680256,1681124,1681182,1681703,1681730,1681840,1681864,1681869,1682010,1682034,1682047,1682052-1682053,1682062,1682064,1682070,1682312,1682325,1682331,1682386,1684367,1684385,1685759,1685774,1685827,1685892,1687341,1688904,1689358,1689657,1689921,1692850,1693093,1693108,1693324,1694060,1694115,1694291,1694427,1694431,1694503,1694549,1694789,1694873,1694881,1695356,1695372,1695823-1695825,1696200,1696281,1696379,1696468,1700608,1700871,1700897,1700978,1701094,1701124,1701608,1701668,1701676,1701766,1701944,1702248,1702252,1702314,1702390,1702723,1702725,1702728,1702730,1702733,1702735,1702737,1702739,1702
 
742,1702744,1702748,1702751,1702754,1702758,1702760,1702763,1702766,1708779,1708782,1708806,1709314,1709670,1710347,1710442,1710448,1710490,1710574,1710578,1712226,1712229,1712235,1712255,1712618,1712649,1712655,1712860,1712899,1712903,1712906,1712913,1712926,1712975,1713185,1713262,1713287,1713613,1713621,1713872,1713976,1713994,1713998,1714004,1714013,1714059,1714538,1714580,1715189,1715207,1715544,1715549,1715637,1715639-1715645,1715667,1715683,1715866,1715978,1715981,1716216-1716217,1716355,1716414,1716421,1717208-1717209,1717257,1717283,1717288,1717291,1717421,1717517,1717529,1718797,1718840-1718843,1719348,1719357-1719358,1719400,1719491,1719737,1720235,1720396,1720442,1720446,1720450,1720463,1720658-1720660,1720756,1720816,1721813,1721818,1721831,1721861,1721867,1721882,1722523,1722527,1722800,1722926,1722941,1722997,1723130,1723440,1723488,1723890,1724434,1724674,1724792,1724803,1724902,1725128,1725131,1725154,1725167,1725911,1725921,1725929,1725963-1725965,1725970,1725974,1
 
726171-1726173,1726175,1726179-1726182,1726190-1726191,1726195-1726200,1726203,1726226,1726576,1726630,1726992,1727029,1727037,1727671,1727676,1727900,1728028,1728092,1728439,1728449,1729186,1729362,1731009,1731303,1731867,1731872,1731874,1731876,1731885,1731947,1731955,1731959,1731977,1731984,1732360,1732490,1732672,1732902,1733166,1733603,1733619,1733735,1733752,1733764,1733915,1733941,1733964,1734115,1734133,1734261,1734421,1734531,1736286,1737967,1738173,1738182,1738992,1739039,1739089-1739091,1739294,1739777,1739821,1739981,1740513,1740726,1741019,1741162,1741217,1743647,1743681,1744152,1744272,1746732,1746750,1752739,1754615,1755886,1756018,1759565,1761686,1762173,1762206,1766280,1767507-1767508,1767653,1767656,1769267,1772949,1773521,1773527,1774104,1777015,1777213,1779330,1783151,1784188,1784966,1785670,1786846,1788260,1788999,1789140,1789402,1791529,1791559,1795291,1796906,1797523
 
/tomcat/tc8.5.x/trunk:1735579,1736839,1737199,1737966,1738042,1738044,1738162,1738165,1738178,1739157,1739173,1739177,1739476,1740132,1740521,1740536,1740804,1740811,1740981,1741165,1741174,1741182,1741191,1741203,1741209,1741226,1741233,1741410,1742277,1743118,1743126,1743139-1743140,1743718,1743722,1743724,1744059,1744127,1744151,1744232,1744377,1744687,1744698,1744706,1745228,1746940,1748548,1748716,1749288,1749375,1749668-1749669,1750016,1750057,1750976,1751000,1751062,1751098,1754112,1754144,1754282,1754312,1754614,1754726,1754806,175

Re: Tomcat6 bug report

2017-06-12 Thread Mark Thomas
On 12/06/17 14:15, Orlando José Luque Moraira wrote:
> Hi, first time here.

This question belongs on the users list, not on the dev list.

Mark


> 
> Found in tomcat6, not sure if it have been solved / changed in a newer
> version.
> 
> Resumee: in a URLed jSessionId controlled session, if you do, from the
> client, one or more html requests to get files but without jSessionId in
> the URL, tomcat create a new "general site" cookie with a new jSessionId
> and this new cookie is stored in the client. After a jSessionId is set in
> that "general site" cookie, the jSessionId of the cookie is used instead of
> the URLed jSessionId value, which is completely ignored from now, breaking
> the URLed jSessionId functionality completely.
> 
> Example of requests in a row:
> furball.com<-- server: set-cookie: jSessionId = 1234
> furball.com;jsessionid=1234 --> we see the cookie in the client
> furball.com/other  <-- server: set-cookie: jSessionId = 5678
> furball.com;jsessionid=1234 --> it ignores the 1234 and starts using the
> cookie jSessionId, so the jSessionId dissapear from the URLs shown in the
> page... really it is a new session :-(
> 
> 
> Any subsequent html request is flawed in the same way until you manage to
> clean that cookie.
> 
> 
> Seems like those new cokies are stored... in like a new local
> not-session-related-cookie. As said, any subsequent html request will have
> its jSessionId ignored and tomcat will use the one stored in that
> "not-session-related-cookie", so you lose your original session and are in
> danger of corruption between tabs with different sessions.
> 
> In a embedded webbrowser, this breaks all sessions working until you
> restart the embedding application.
> 
> In webbrowser applications (as firefox) suffer the same way and as far as I
> saw, deleting al cookies did not solve the issue. Weird. (I have the cache
> madness so much... :-S).
> 
> 
> 
> Please, have this been resolved in newer versions? If possible, in which
> one?
> 
> Thanks!
> 
> Regards,
>Orlando
> 


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



svn commit: r1798560 - /tomcat/tc7.0.x/trunk/test/org/apache/catalina/valves/TestCrawlerSessionManagerValve.java

2017-06-12 Thread violetagg
Author: violetagg
Date: Tue Jun 13 06:24:57 2017
New Revision: 1798560

URL: http://svn.apache.org/viewvc?rev=1798560&view=rev
Log:
Collections.emptyEnumeration() does not exist in java 6, use alternative.

Modified:

tomcat/tc7.0.x/trunk/test/org/apache/catalina/valves/TestCrawlerSessionManagerValve.java

Modified: 
tomcat/tc7.0.x/trunk/test/org/apache/catalina/valves/TestCrawlerSessionManagerValve.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/test/org/apache/catalina/valves/TestCrawlerSessionManagerValve.java?rev=1798560&r1=1798559&r2=1798560&view=diff
==
--- 
tomcat/tc7.0.x/trunk/test/org/apache/catalina/valves/TestCrawlerSessionManagerValve.java
 (original)
+++ 
tomcat/tc7.0.x/trunk/test/org/apache/catalina/valves/TestCrawlerSessionManagerValve.java
 Tue Jun 13 06:24:57 2017
@@ -80,7 +80,8 @@ public class TestCrawlerSessionManagerVa
 if (isBot) {
 setter.andReturn(session);
 }
-
EasyMock.expect(request.getHeaders("user-agent")).andReturn(Collections.emptyEnumeration());
+EasyMock.expect(request.getHeaders("user-agent"))
+
.andReturn(Collections.enumeration(Collections.emptyList()));
 return request;
 }
 }



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