Author: markt Date: Sun Apr 11 14:25:21 2010 New Revision: 932912 URL: http://svn.apache.org/viewvc?rev=932912&view=rev Log: Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=49018 Fix processing of time argument in the "Expire sessions" action in the Manager web application. The patch also provides separate templates for different variants of a message, printed by that action. That is to allow proper translation of those messages, though I cannot provide such translation by myself. (kkolinko)
Modified: tomcat/tc6.0.x/trunk/STATUS.txt tomcat/tc6.0.x/trunk/java/org/apache/catalina/manager/LocalStrings.properties tomcat/tc6.0.x/trunk/java/org/apache/catalina/manager/LocalStrings_de.properties tomcat/tc6.0.x/trunk/java/org/apache/catalina/manager/LocalStrings_es.properties tomcat/tc6.0.x/trunk/java/org/apache/catalina/manager/LocalStrings_fr.properties tomcat/tc6.0.x/trunk/java/org/apache/catalina/manager/LocalStrings_ja.properties tomcat/tc6.0.x/trunk/java/org/apache/catalina/manager/ManagerServlet.java tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml Modified: tomcat/tc6.0.x/trunk/STATUS.txt URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=932912&r1=932911&r2=932912&view=diff ============================================================================== --- tomcat/tc6.0.x/trunk/STATUS.txt (original) +++ tomcat/tc6.0.x/trunk/STATUS.txt Sun Apr 11 14:25:21 2010 @@ -243,16 +243,6 @@ PATCHES PROPOSED TO BACKPORT: +1: markt, kkolinko -1: -* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=49018 - Fix processing of time argument in the "Expire sessions" action in the - Manager web application. - The patch also provides separate templates for different variants of a - message, printed by that action. That is to allow proper translation of - those messages, though I cannot provide such translation by myself. - http://svn.apache.org/viewvc?rev=931415&view=rev - +1: kkolinko, markt, kfujino - -1: - * Use chunked encoding for http 1.1 requests with no content-length (regardless of keep-alive) so client can differentiate between complete and partial responses. Modified: tomcat/tc6.0.x/trunk/java/org/apache/catalina/manager/LocalStrings.properties URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/catalina/manager/LocalStrings.properties?rev=932912&r1=932911&r2=932912&view=diff ============================================================================== --- tomcat/tc6.0.x/trunk/java/org/apache/catalina/manager/LocalStrings.properties (original) +++ tomcat/tc6.0.x/trunk/java/org/apache/catalina/manager/LocalStrings.properties Sun Apr 11 14:25:21 2010 @@ -98,7 +98,9 @@ managerServlet.saveFail=FAIL - Configura managerServlet.saved=OK - Server configuration saved managerServlet.savedContext=OK - Context {0} configuration saved managerServlet.sessiondefaultmax=Default maximum session inactive interval {0} minutes -managerServlet.sessiontimeout={0} minutes:{1} sessions +managerServlet.sessiontimeout={0} minutes: {1} sessions +managerServlet.sessiontimeout.unlimited=unlimited time: {0} sessions +managerServlet.sessiontimeout.expired={0} minutes: {1} sessions were expired managerServlet.sessions=OK - Session information for application at context path {0} managerServlet.started=OK - Started application at context path {0} managerServlet.startFailed=FAIL - Application at context path {0} could not be started Modified: tomcat/tc6.0.x/trunk/java/org/apache/catalina/manager/LocalStrings_de.properties URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/catalina/manager/LocalStrings_de.properties?rev=932912&r1=932911&r2=932912&view=diff ============================================================================== --- tomcat/tc6.0.x/trunk/java/org/apache/catalina/manager/LocalStrings_de.properties (original) +++ tomcat/tc6.0.x/trunk/java/org/apache/catalina/manager/LocalStrings_de.properties Sun Apr 11 14:25:21 2010 @@ -80,7 +80,10 @@ managerServlet.resourcesType=OK - Auflis managerServlet.rolesList=OK - Auflistung der Sicherheits-Rollen managerServlet.saveFail=FEHLER - Speichern der Konfiguration fehlgeschlagen: {0} managerServlet.sessiondefaultmax=Voreingestellter Sitzungsablauf nach maximal {0} Minuten Inaktivit\u00e4t +#TODO: Please review the following three messages. These are displayed when "Expire sessions" button is pressed in the Manager webapp: managerServlet.sessiontimeout={0} Minuten: {1} Sitzungen +managerServlet.sessiontimeout.unlimited=unlimited Minuten: {0} Sitzungen +managerServlet.sessiontimeout.expired={0} Minuten: expired {1} Sitzungen managerServlet.sessions=OK - Sitzungs-Informationen f\u00fcr Anwendung mit Kontext Pfad {0} managerServlet.started=OK - Anwendung mit Kontext Pfad {0} gestartet managerServlet.startFailed=FEHLER - Anwendung mit Kontext Pfad {0} konnte nicht gestartet werden Modified: tomcat/tc6.0.x/trunk/java/org/apache/catalina/manager/LocalStrings_es.properties URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/catalina/manager/LocalStrings_es.properties?rev=932912&r1=932911&r2=932912&view=diff ============================================================================== --- tomcat/tc6.0.x/trunk/java/org/apache/catalina/manager/LocalStrings_es.properties (original) +++ tomcat/tc6.0.x/trunk/java/org/apache/catalina/manager/LocalStrings_es.properties Sun Apr 11 14:25:21 2010 @@ -86,7 +86,10 @@ managerServlet.saveFail = FAIL - Fallo a managerServlet.saved = OK - Configuraci\u00F3n de Servidor guardada managerServlet.savedContext = OK - Configuraci\u00F3n de Contexto {0} guardada managerServlet.sessiondefaultmax = Intervalo m\u00E1ximo por defecto de sesi\u00F3n inactiva {0} minutos -managerServlet.sessiontimeout = {0} minutos\: {1} sesiones +#TODO: Please review the following three messages. These are displayed when "Expire sessions" button is pressed in the Manager webapp: +managerServlet.sessiontimeout={0} minutos: {1} sesiones +managerServlet.sessiontimeout.unlimited=unlimited minutos: {0} sesiones +managerServlet.sessiontimeout.expired={0} minutos: expired {1} sesiones managerServlet.sessions = OK - Informaci\u00F3n de sesi\u00F3n para aplicaci\u00F3n en trayectoria de contexto {0} managerServlet.started = OK - Arrancada aplicaci\u00F3n en trayectoria de contexto {0} managerServlet.startFailed = FALLO - No se pudo arrancar la aplicaci\u00F3n en trayectoria de contexto {0} Modified: tomcat/tc6.0.x/trunk/java/org/apache/catalina/manager/LocalStrings_fr.properties URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/catalina/manager/LocalStrings_fr.properties?rev=932912&r1=932911&r2=932912&view=diff ============================================================================== --- tomcat/tc6.0.x/trunk/java/org/apache/catalina/manager/LocalStrings_fr.properties (original) +++ tomcat/tc6.0.x/trunk/java/org/apache/catalina/manager/LocalStrings_fr.properties Sun Apr 11 14:25:21 2010 @@ -88,7 +88,10 @@ managerServlet.saveFail=ECHEC - La sauve managerServlet.saved=OK - Configuration serveur sauvegard\u00e9e managerServlet.savedContext=OK - Configuration du contexte {0} sauvegard\u00e9e managerServlet.sessiondefaultmax=Interval par d\u00e9faut de maximum de session inactive {0} minutes -managerServlet.sessiontimeout={0} minutes:{1} sessions +#TODO: Please review the following three messages. These are displayed when "Expire sessions" button is pressed in the Manager webapp: +managerServlet.sessiontimeout={0} minutes: {1} sessions +managerServlet.sessiontimeout.unlimited=unlimited time: {0} sessions +managerServlet.sessiontimeout.expired={0} minutes: {1} sessions were expired managerServlet.sessions=OK - Information de session pour l''application au chemin de contexte {0} managerServlet.started=OK - Application d\u00e9marr\u00e9e pour le chemin de contexte {0} managerServlet.startFailed=ECHEC - L''application pour le chemin de contexte {0} n''a pas pu \u00eatre d\u00e9marr\u00e9e Modified: tomcat/tc6.0.x/trunk/java/org/apache/catalina/manager/LocalStrings_ja.properties URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/catalina/manager/LocalStrings_ja.properties?rev=932912&r1=932911&r2=932912&view=diff ============================================================================== --- tomcat/tc6.0.x/trunk/java/org/apache/catalina/manager/LocalStrings_ja.properties (original) +++ tomcat/tc6.0.x/trunk/java/org/apache/catalina/manager/LocalStrings_ja.properties Sun Apr 11 14:25:21 2010 @@ -79,7 +79,10 @@ managerServlet.saveFail=FAIL - \u8a2d\u5 managerServlet.saved=OK - \u30b5\u30fc\u30d0\u306e\u8a2d\u5b9a\u3092\u4fdd\u5b58\u3057\u307e\u3057\u305f managerServlet.savedContext=OK - \u30b3\u30f3\u30c6\u30ad\u30b9\u30c8 {0} \u306e\u8a2d\u5b9a\u3092\u4fdd\u5b58\u3057\u307e\u3057\u305f managerServlet.sessiondefaultmax=\u30c7\u30d5\u30a9\u30eb\u30c8\u306e\u6700\u5927\u30bb\u30c3\u30b7\u30e7\u30f3\u505c\u6b62\u9593\u9694\u306f{0}\u5206\u3067\u3059 +#TODO: Please review the following three messages. These are displayed when "Expire sessions" button is pressed in the Manager webapp: managerServlet.sessiontimeout={0}\u5206: {1}\u30bb\u30c3\u30b7\u30e7\u30f3 +managerServlet.sessiontimeout.unlimited=unlimited \u5206: {0}\u30bb\u30c3\u30b7\u30e7\u30f3 +managerServlet.sessiontimeout.expired={0}\u5206: expired {1}\u30bb\u30c3\u30b7\u30e7\u30f3 managerServlet.sessions=OK - \u30b3\u30f3\u30c6\u30ad\u30b9\u30c8\u30d1\u30b9 {0} \u306e\u30a2\u30d7\u30ea\u30b1\u30fc\u30b7\u30e7\u30f3\u306e\u30bb\u30c3\u30b7\u30e7\u30f3\u60c5\u5831\u3067\u3059 managerServlet.started=OK - \u30b3\u30f3\u30c6\u30ad\u30b9\u30c8\u30d1\u30b9 {0} \u3067\u30a2\u30d7\u30ea\u30b1\u30fc\u30b7\u30e7\u30f3\u3092\u8d77\u52d5\u3057\u307e\u3057\u305f managerServlet.startFailed=FAIL - \u30b3\u30f3\u30c6\u30ad\u30b9\u30c8\u30d1\u30b9 {0} \u306e\u30a2\u30d7\u30ea\u30b1\u30fc\u30b7\u30e7\u30f3\u304c\u8d77\u52d5\u3067\u304d\u307e\u305b\u3093 Modified: tomcat/tc6.0.x/trunk/java/org/apache/catalina/manager/ManagerServlet.java URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/catalina/manager/ManagerServlet.java?rev=932912&r1=932911&r2=932912&view=diff ============================================================================== --- tomcat/tc6.0.x/trunk/java/org/apache/catalina/manager/ManagerServlet.java (original) +++ tomcat/tc6.0.x/trunk/java/org/apache/catalina/manager/ManagerServlet.java Sun Apr 11 14:25:21 2010 @@ -1175,7 +1175,7 @@ public class ManagerServlet int time = (int)((now-sessions[i].getLastAccessedTimeInternal())/1000); if (idle >= 0 && time >= idle*60) { sessions[i].expire(); - idle++; + expired++; } time=time/60/histoInterval; if (time < 0) @@ -1199,11 +1199,11 @@ public class ManagerServlet ">=" + maxCount*histoInterval, "" + timeout[maxCount-1])); if (notimeout > 0) - writer.println(sm.getString("managerServlet.sessiontimeout", - "unlimited","" + notimeout)); + writer.println(sm.getString("managerServlet.sessiontimeout.unlimited", + "" + notimeout)); if (idle >= 0) - writer.println(sm.getString("managerServlet.sessiontimeout", - "" + idle,"expired " + expired)); + writer.println(sm.getString("managerServlet.sessiontimeout.expired", + "" + idle,"" + expired)); } catch (Throwable t) { log("ManagerServlet.sessions[" + displayPath + "]", t); writer.println(sm.getString("managerServlet.exception", Modified: tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml?rev=932912&r1=932911&r2=932912&view=diff ============================================================================== --- tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml (original) +++ tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml Sun Apr 11 14:25:21 2010 @@ -68,6 +68,10 @@ <bug>48790</bug>: Fix thread safety issue in the count of the maximum number of active session. (markt/kkolinko) </fix> + <fix> + <bug>49018</bug>: Fix processing of time argument in the Expire sessions + action in the Manager web application. (kkolinko) + </fix> </changelog> </subsection> <subsection name="Coyote"> --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org