Bug report for Tomcat Native [2017/08/27]

2017-08-27 Thread bugzilla
+---+
| Bugzilla Bug ID   |
| +-+
| | Status: UNC=Unconfirmed NEW=New ASS=Assigned|
| | OPN=ReopenedVER=Verified(Skipped Closed/Resolved)   |
| |   +-+
| |   | Severity: BLK=Blocker CRI=Critical  REG=Regression  MAJ=Major   |
| |   |   MIN=Minor   NOR=NormalENH=Enhancement TRV=Trivial |
| |   |   +-+
| |   |   | Date Posted |
| |   |   |  +--+
| |   |   |  | Description  |
| |   |   |  |  |
|53940|New|Enh|2012-09-27|Added support for new CRL loading after expiration|
|55087|New|Cri|2013-06-10|tomcat crashes in tcnative-1.dll with OCSP when OC|
|56378|New|Nor|2014-04-09|Cert load fails if cert is located in path with no|
|57815|New|Enh|2015-04-15|Improve error message when OpenSSL does not suppor|
|58194|Inf|Maj|2015-07-30|Tomcat crash EXCEPTION_ACCESS_VIOLATION in tcnativ|
|59286|New|Nor|2016-04-07|Socket binding failures when using APR|
+-+---+---+--+--+
| Total6 bugs   |
+---+

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



Bug report for Tomcat 8 [2017/08/27]

2017-08-27 Thread bugzilla
+---+
| Bugzilla Bug ID   |
| +-+
| | Status: UNC=Unconfirmed NEW=New ASS=Assigned|
| | OPN=ReopenedVER=Verified(Skipped Closed/Resolved)   |
| |   +-+
| |   | Severity: BLK=Blocker CRI=Critical  REG=Regression  MAJ=Major   |
| |   |   MIN=Minor   NOR=NormalENH=Enhancement TRV=Trivial |
| |   |   +-+
| |   |   | Date Posted |
| |   |   |  +--+
| |   |   |  | Description  |
| |   |   |  |  |
|43925|Opn|Enh|2007-11-21|org.apache.jasper.runtime.BodyContentImpl causing |
|51497|New|Enh|2011-07-11|Use canonical IPv6 text representation in logs|
|53737|Opn|Enh|2012-08-18|Use ServletContext.getJspConfigDescriptor() in Jas|
|53930|New|Enh|2012-09-24|allow capture of catalina stdout/stderr to a comma|
|54700|New|Enh|2013-03-15|Improvement: Add support for system property to sp|
|54741|New|Enh|2013-03-22|Add org.apache.catalina.startup.Tomcat#addWebapp(S|
|55243|New|Enh|2013-07-11|Add special search string for nested roles|
|55252|New|Enh|2013-07-12|Separate Ant and command-line wrappers for JspC   |
|55383|New|Enh|2013-08-07|Improve markup and design of Tomcat's HTML pages  |
|9|New|Enh|2013-09-14|UserDatabaseRealm enhacement: may use local JNDI  |
|55675|New|Enh|2013-10-18|Checking and handling invalid configuration option|
|55770|New|Enh|2013-11-12|Allow the crlFile to be reloaded  |
|55788|New|Enh|2013-11-16|TagPlugins should key on tag QName rather than imp|
|55969|New|Enh|2014-01-07|Security-related enhancements to the Windows Insta|
|56166|New|Enh|2014-02-20|Suggestions for exception handling (avoid potentia|
|56361|New|Enh|2014-04-08|org.apache.tomcat.websocket.WsWebSocketContainer#b|
|56398|New|Enh|2014-04-11|Support Arquillian-based unit testing |
|56399|New|Enh|2014-04-11|Re-factor request/response recycling so Coyote and|
|56402|New|Enh|2014-04-11|Add support for HTTP Upgrade to AJP components|
|56448|New|Enh|2014-04-23|Implement a robust solution for client initiated S|
|56522|Opn|Enh|2014-05-14|jasper-el 8 does not comply to EL Spec 3.0 regardi|
|56546|New|Enh|2014-05-19|Improve thread trace logging in WebappClassLoader.|
|56676|New|Enh|2014-06-26|Normalize access to native library|
|56713|New|Enh|2014-07-12|Limit time that incoming request waits while webap|
|56724|New|Enh|2014-07-15|Restart Container background thread if it died une|
|56890|Inf|Maj|2014-08-26|getRealPath returns null  |
|56966|New|Enh|2014-09-11|AccessLogValve's elapsed time has 15ms precision o|
|57130|New|Enh|2014-10-22|Allow digest.sh to accept password from a file or |
|57287|New|Enh|2014-11-29|Sort files listed by DefaultServlet   |
|57345|New|Enh|2014-12-12|APR/Native HTTPS Connector Should Support All Open|
|57421|New|Enh|2015-01-07|Farming default directories   |
|57486|New|Enh|2015-01-23|Improve reuse of ProtectedFunctionMapper instances|
|57665|New|Enh|2015-03-05|support x-forwarded-host  |
|57701|New|Enh|2015-03-13|Implement "[Redeploy]" button for a web applicatio|
|57830|New|Enh|2015-04-18|Add support for ProxyProtocol |
|58052|Opn|Enh|2015-06-19|RewriteValve: Implement additional RewriteRule dir|
|58072|New|Enh|2015-06-23|ECDH curve selection  |
|58433|New|Enh|2015-09-21|RemoteIpValve not activated on redirect from mappi|
|58577|New|Enh|2015-11-03|JMX Proxy Servlet can't handle overloaded methods |
|58837|New|Enh|2016-01-12|support "X-Content-Security-Policy" a.k.a as "CSP"|
|58935|Opn|Enh|2016-01-29|Re-deploy from war without deleting context   |
|59232|New|Enh|2016-03-24|Make the context name of an app available via JNDI|
|59423|New|Enh|2016-05-03|amend "No LoginModules configured for ..." with hi|
|59758|New|Enh|2016-06-27|Add http proxy username-password credentials suppo|
|60276|New|Enh|2016-10-19|upgrade HTTP/2 can't use gzip compress.   |
|60281|Ver|Nor|2016-10-20|Pathname of uploaded WAR file should not be contai|
|60511|Inf|Maj|2016-12-22|org.apache.coyote.ajp.AjpNio2Protocol sends wrong |
|60560|New|Enh|2017-01-07|Support systemd/inetd style socket activation |
|60721|Ver|Nor|2017-02-10|Unable to find key spec if more applications use b|
|60762|New|Enh|2017-02-21|Enhancement: Add support for runtime SNI changes i|
|60781|New|Nor|2017-02-27|Access Log Valve does not escape the same as mod_l|
|60849|

Bug report for Tomcat 9 [2017/08/27]

2017-08-27 Thread bugzilla
+---+
| Bugzilla Bug ID   |
| +-+
| | Status: UNC=Unconfirmed NEW=New ASS=Assigned|
| | OPN=ReopenedVER=Verified(Skipped Closed/Resolved)   |
| |   +-+
| |   | Severity: BLK=Blocker CRI=Critical  REG=Regression  MAJ=Major   |
| |   |   MIN=Minor   NOR=NormalENH=Enhancement TRV=Trivial |
| |   |   +-+
| |   |   | Date Posted |
| |   |   |  +--+
| |   |   |  | Description  |
| |   |   |  |  |
|47467|New|Enh|2009-07-02|Deployment of the war file by URL when contextpath|
|48672|New|Enh|2010-02-03|Tomcat Virtual Host Manager (/host-manager) needs |
|57505|New|Enh|2015-01-27|Add integration tests for JspC|
|57661|New|Enh|2015-03-04|Delay sending of 100 continue response until appli|
|57767|Opn|Enh|2015-03-27|Websocket client proprietary configuration|
|58242|New|Enh|2015-08-13|Scanning jars in classpath to get annotations in p|
|58530|New|Enh|2015-10-23|Proposal for new Manager HTML GUI |
|58548|New|Enh|2015-10-26|support certifcate transparency   |
|58590|New|Enh|2015-11-05|org.apache.catalina.realm.MemoryRealm can use back|
|58859|New|Enh|2016-01-14|Allow to limit charsets / encodings supported by T|
|59179|New|Enh|2016-03-14|HTTP Public Key Pinning (HPKP) for Tomcat |
|59203|New|Enh|2016-03-21|Try to call Thread.interrupt before calling Thread|
|59344|Ver|Enh|2016-04-18|PEM file support for JSSE |
|59750|New|Enh|2016-06-24|Amend "authenticate" method with context by means |
|59901|New|Enh|2016-07-26|Reduce I/O associated with JSP compilation|
|60523|Opn|Enh|2016-12-27|Reduce number of network packets that server sends|
|60997|New|Enh|2017-04-17|Enhance SemaphoreValve to support denied status an|
|61171|New|Enh|2017-06-09|Add port offset attribute (portOffset?) to Server |
|61189|New|Enh|2017-06-15|CGIServlet should be able to set specific environm|
|61223|New|Enh|2017-06-26|Enhance the documentation for mbeans-descriptors.x|
|61280|New|Enh|2017-07-11|Support characters sets other than ISO 8859-1 in H|
|61393|New|Min|2017-08-08|org.apache.tomcat.jni.TestSocketServer timeout fai|
|61394|New|Min|2017-08-08|NIO/NIO2 + OpenSSL renegotiation doesn't send list|
+-+---+---+--+--+
| Total   23 bugs   |
+---+

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



Bug report for Taglibs [2017/08/27]

2017-08-27 Thread bugzilla
+---+
| Bugzilla Bug ID   |
| +-+
| | Status: UNC=Unconfirmed NEW=New ASS=Assigned|
| | OPN=ReopenedVER=Verified(Skipped Closed/Resolved)   |
| |   +-+
| |   | Severity: BLK=Blocker CRI=Critical  REG=Regression  MAJ=Major   |
| |   |   MIN=Minor   NOR=NormalENH=Enhancement TRV=Trivial |
| |   |   +-+
| |   |   | Date Posted |
| |   |   |  +--+
| |   |   |  | Description  |
| |   |   |  |  |
|38193|Ass|Enh|2006-01-09|[RDC] BuiltIn Grammar support for Field   |
|38600|Ass|Enh|2006-02-10|[RDC] Enable RDCs to be used in X+V markup (X+RDC)|
|42413|New|Enh|2007-05-14|[PATCH] Log Taglib enhancements   |
|46052|New|Nor|2008-10-21|SetLocaleSupport is slow to initialize when many l|
|48333|New|Enh|2009-12-02|TLD generator |
|57434|New|Nor|2015-01-11|Race condition in EL1.0 validation|
|57548|New|Min|2015-02-08|Auto-generate the value for org.apache.taglibs.sta|
|57684|New|Min|2015-03-10|Version info should be taken from project version |
|59359|New|Enh|2016-04-20|(Task) Extend validity period for signing KEY - be|
|59668|New|Nor|2016-06-06|x:forEach retains the incorrect scope when used in|
+-+---+---+--+--+
| Total   10 bugs   |
+---+

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



Bug report for Tomcat 7 [2017/08/27]

2017-08-27 Thread bugzilla
+---+
| Bugzilla Bug ID   |
| +-+
| | Status: UNC=Unconfirmed NEW=New ASS=Assigned|
| | OPN=ReopenedVER=Verified(Skipped Closed/Resolved)   |
| |   +-+
| |   | Severity: BLK=Blocker CRI=Critical  REG=Regression  MAJ=Major   |
| |   |   MIN=Minor   NOR=NormalENH=Enhancement TRV=Trivial |
| |   |   +-+
| |   |   | Date Posted |
| |   |   |  +--+
| |   |   |  | Description  |
| |   |   |  |  |
|41007|Opn|Enh|2006-11-20|Can't define customized 503 error page|
|43866|New|Enh|2007-11-14|add support for session attribute propagation with|
|47242|New|Enh|2009-05-22|request for AJP command line client   |
|49395|New|Enh|2010-06-06|manager.findLeaks : display the date when the leak|
|49821|New|Enh|2010-08-25|Tomcat CLI [PATCH/Contribution]   |
|50019|New|Enh|2010-09-28|Adding JNDI "lookup-name" support In XML and Resou|
|50175|New|Enh|2010-10-28|Enhance memory leak detection by selectively apply|
|50234|New|Enh|2010-11-08|JspC use servlet 3.0 features |
|50670|New|Enh|2011-01-27|Tribes | RpcChannel | Add option to specify extern|
|50944|Ver|Blk|2011-03-18|JSF: java.lang.NullPointerException at com.sun.fac|
|51195|New|Enh|2011-05-13|"Find leaks" reports a false positive memory/class|
|51423|Inf|Enh|2011-06-23|[Patch] to add a path and a version parameters to |
|51496|New|Enh|2011-07-11|NSIS - Warn that duplicate service name will resul|
|51587|New|Enh|2011-07-29|Implement status and uptime commands  |
|51953|New|Enh|2011-10-04|Proposal: netmask filtering valve and filter [PATC|
|52381|New|Enh|2011-12-22|Please add OSGi metadata  |
|52448|New|Enh|2012-01-11|Cache jar indexes in WebappClassLoader to speed up|
|52489|New|Enh|2012-01-19|Enhancement request for code signing of war files |
|52688|New|Enh|2012-02-16|Add ability to remove old access log files [PATCHE|
|52952|New|Enh|2012-03-20|Improve ExtensionValidator handling for embedded s|
|53085|New|Enh|2012-04-16|[perf] [concurrency] DefaultInstanceManager.annota|
|53387|New|Enh|2012-06-08|SSI: Allow to use $1 to get result of regular expr|
|53411|Opn|Enh|2012-06-13|NullPointerException in org.apache.tomcat.util.buf|
|53492|New|Enh|2012-07-01|Make JspC shell multithreaded |
|53553|New|Enh|2012-07-16|[PATCH] Deploy uploaded WAR with context.xml from |
|53620|New|Enh|2012-07-30|[juli] delay opening a file until something gets l|
|54499|New|Enh|2013-01-29|Implementation of Extensible EL Interpreter   |
|54802|New|Enh|2013-04-04|Provide location information for exceptions thrown|
|55104|New|Enh|2013-06-16|Allow passing arguments with spaces to Commons Dae|
|55470|New|Enh|2013-08-23|Help users for ClassNotFoundExceptions during star|
|55477|New|Enh|2013-08-23|Add a solution to map an realm name to a security |
|56148|New|Enh|2014-02-17|support (multiple) ocsp stapling  |
|56181|New|Enh|2014-02-23|RemoteIpValve & RemoteIpFilter: HttpServletRequest|
|56300|New|Enh|2014-03-22|[Tribes] No useful examples, lack of documentation|
|56438|New|Enh|2014-04-21|If jar scan does not find context config or TLD co|
|56614|New|Enh|2014-06-12|Add a switch to ignore annotations detection on ta|
|56787|New|Enh|2014-07-29|Simplified jndi name parsing  |
|57367|New|Enh|2014-12-18|If JAR scan experiences a stack overflow, give the|
|57827|New|Enh|2015-04-17|Enable adding/removing of members via jmx in a sta|
|57870|New|Enh|2015-04-29|backport GzipOutputFilter #doWrite to Tomcat 7 to |
|57872|New|Enh|2015-04-29|Do not auto-switch session cookie to version=1 due|
|57892|New|Enh|2015-05-05|Log once a warning if a symbolic link is ignored (|
|58338|New|Nor|2015-09-07|BasicDataSourceFactory uses wrong attribute name  |
|59716|New|Enh|2016-06-17|Allow JNDI configuration of CorsFilter|
|60597|New|Enh|2017-01-17|Add ability to set cipher suites for websocket cli|
|60944|Inf|Nor|2017-03-30|Tomcat Production Issue connections in CLOSE_WAIT |
|61367|Inf|Nor|2017-08-01|NPE exception in org.apache.catalina.connector.Coy|
+-+---+---+--+--+
| Total   47 bugs   |
+---+

-
To unsubscribe

Bug report for Tomcat Connectors [2017/08/27]

2017-08-27 Thread bugzilla
+---+
| Bugzilla Bug ID   |
| +-+
| | Status: UNC=Unconfirmed NEW=New ASS=Assigned|
| | OPN=ReopenedVER=Verified(Skipped Closed/Resolved)   |
| |   +-+
| |   | Severity: BLK=Blocker CRI=Critical  REG=Regression  MAJ=Major   |
| |   |   MIN=Minor   NOR=NormalENH=Enhancement TRV=Trivial |
| |   |   +-+
| |   |   | Date Posted |
| |   |   |  +--+
| |   |   |  | Description  |
| |   |   |  |  |
|35959|Opn|Enh|2005-08-01|mod_jk not independant of UseCanonicalName|
|43303|New|Enh|2007-09-04|Versioning under Windows not reported by many conn|
|45313|New|Nor|2008-06-30|mod_jk 1.2.26 & apache 2.2.9 static compiled on so|
|46767|New|Enh|2009-02-25|mod_jk to send DECLINED in case no fail-over tomca|
|47327|New|Enh|2009-06-07|Return tomcat authenticated user back to mod_jk (A|
|47750|New|Maj|2009-08-27|ISAPI: Loss of worker settings when changing via j|
|47795|New|Maj|2009-09-07|service sticky_session not being set correctly wit|
|48513|New|Enh|2010-01-09|IIS Quick setup instructions  |
|48564|New|Enh|2010-01-18|Allow to turn off retries for LB worker   |
|48830|New|Nor|2010-03-01|IIS shutdown blocked in endpoint service when serv|
|49063|New|Enh|2010-04-07|Please add JkStripSession status in jk-status work|
|49822|New|Enh|2010-08-25|Add hash lb worker method |
|49903|New|Enh|2010-09-09|Make workers file reloadable  |
|52483|New|Enh|2012-01-18|Print JkOptions's options in log file and jkstatus|
|53883|New|Maj|2012-09-17|isapi_redirect v 1.2.37 crashes w3wp.exe  on the p|
|53977|New|Maj|2012-10-07|32bits isapi connector cannot work in wow64 mode  |
|54027|New|Cri|2012-10-18|isapi send request to outside address instead of i|
|54117|New|Maj|2012-11-08|access violation exception in isapi_redirect.dll  |
|54621|New|Enh|2013-02-28|[PATCH] custom mod_jk availability checks |
|56489|New|Enh|2014-05-05|Include a directory for configuration files   |
|56576|New|Enh|2014-05-29|Websocket support |
|57402|New|Enh|2014-12-30|Provide correlation ID between mod_jk log and acce|
|57403|New|Enh|2014-12-30|Persist configuration changes made via status work|
|57407|New|Enh|2014-12-31|Make session_cookie, session_path and session_cook|
|57790|New|Enh|2015-04-03|Check worker names for typos  |
|57946|New|Nor|2015-05-23|Configuration example for mod_jk should be updated|
|58287|New|Nor|2015-08-26|Questionable use of "Global" objects on Windows   |
|59897|New|Nor|2016-07-25|Buffer Overflow in FD_SET in nb_connect (jk_connec|
|60240|New|Min|2016-10-11|Duplicate initialization log entry in mod_jk.log  |
|60745|New|Nor|2017-02-18|False positive: Somebody try to hack into the site|
+-+---+---+--+--+
| Total   30 bugs   |
+---+

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



Bug report for Tomcat Modules [2017/08/27]

2017-08-27 Thread bugzilla
+---+
| Bugzilla Bug ID   |
| +-+
| | Status: UNC=Unconfirmed NEW=New ASS=Assigned|
| | OPN=ReopenedVER=Verified(Skipped Closed/Resolved)   |
| |   +-+
| |   | Severity: BLK=Blocker CRI=Critical  REG=Regression  MAJ=Major   |
| |   |   MIN=Minor   NOR=NormalENH=Enhancement TRV=Trivial |
| |   |   +-+
| |   |   | Date Posted |
| |   |   |  +--+
| |   |   |  | Description  |
| |   |   |  |  |
|50571|Inf|Nor|2011-01-11|Tomcat 7 JDBC connection pool exception enhancemen|
|51595|Inf|Nor|2011-08-01|org.apache.tomcat.jdbc.pool.jmx.ConnectionPool sho|
|51879|Inf|Enh|2011-09-22|Improve access to Native Connection Methods   |
|52024|Inf|Enh|2011-10-13|Custom interceptor to support automatic failover o|
|53199|Inf|Enh|2012-05-07|Refactor ConnectionPool to use ScheduledExecutorSe|
|54437|New|Enh|2013-01-16|Update PoolProperties javadoc for ConnectState int|
|54929|Inf|Nor|2013-05-05|jdbc-pool cannot be used with Java 1.5, "java.lang|
|55078|New|Nor|2013-06-07|Configuring a DataSource Resource with dataSourceJ|
|55662|New|Enh|2013-10-17|Add a way to set an instance of java.sql.Driver di|
|56046|New|Enh|2014-01-21|org.apache.tomcat.jdbc.pool.XADataSource InitSQL p|
|56088|New|Maj|2014-01-29|AbstractQueryReport$StatementProxy throws exceptio|
|56310|Inf|Maj|2014-03-25|PooledConnection and XAConnection not handled corr|
|56586|New|Nor|2014-06-02|initSQL should be committed if defaultAutoCommit =|
|56775|New|Nor|2014-07-28|PoolCleanerTime schedule issue|
|56779|New|Nor|2014-07-28|Allow multiple connection initialization statement|
|56790|New|Nor|2014-07-29|Resizing pool.maxActive to a higher value at runti|
|56798|New|Nor|2014-07-31|Idle eviction strategy could perform better (and i|
|56804|New|Nor|2014-08-02|Use a default validationQueryTimeout other than "f|
|56805|New|Nor|2014-08-02|datasource.getConnection() may be unnecessarily bl|
|56837|New|Nor|2014-08-11|if validationQuery have error with timeBetweenEvic|
|56970|New|Nor|2014-09-11|MaxActive vs. MaxTotal for commons-dbcp and tomcat|
|56974|New|Nor|2014-09-12|jdbc-pool validation query defaultAutoCommit statu|
|57460|New|Nor|2015-01-19|[DB2]Connection broken after few hours but not rem|
|57729|New|Enh|2015-03-20|Add QueryExecutionReportInterceptor to log query e|
|58489|Opn|Maj|2015-10-08|QueryStatsComparator throws IllegalArgumentExcepti|
|59077|New|Nor|2016-02-26|DataSourceFactory creates a neutered data source  |
|59569|New|Nor|2016-05-18|isWrapperFor/unwrap implementations incorrect |
|59879|New|Nor|2016-07-18|StatementCache interceptor returns ResultSet objec|
|60195|New|Nor|2016-10-02|No javadoc in Maven Central   |
|60522|New|Nor|2016-12-27|An option for setting if the transaction should be|
|60524|Inf|Nor|2016-12-28|NPE in SlowQueryReport in tomcat-jdbc-7.0.68  |
|60645|New|Nor|2017-01-25|StatementFinalizer is not thread-safe |
|61032|New|Nor|2017-04-24|min pool size is not being respected  |
|61103|New|Nor|2017-05-18|StatementCache potentially returning incorrect sta|
|61302|New|Enh|2017-07-15|Refactoring of DataSourceProxy|
|61303|New|Enh|2017-07-15|Refactoring of ConnectionPool |
|61312|New|Nor|2017-07-17|NullPointerException in StatementCache.isCached   |
+-+---+---+--+--+
| Total   37 bugs   |
+---+

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



svn commit: r1806356 - /tomcat/trunk/java/org/apache/tomcat/util/net/openssl/OpenSSLEngine.java

2017-08-27 Thread rjung
Author: rjung
Date: Sun Aug 27 12:21:41 2017
New Revision: 1806356

URL: http://svn.apache.org/viewvc?rev=1806356&view=rev
Log:
Revert part of r1805525:

Some attributes of the SSL connection actually
can change dynamically after initialization,
especially the auth mode during renegotiation.

Not allowing this broke multiple junit tests
in TestCustomSsl and TestClientCert.

Modified:
tomcat/trunk/java/org/apache/tomcat/util/net/openssl/OpenSSLEngine.java

Modified: 
tomcat/trunk/java/org/apache/tomcat/util/net/openssl/OpenSSLEngine.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/net/openssl/OpenSSLEngine.java?rev=1806356&r1=1806355&r2=1806356&view=diff
==
--- tomcat/trunk/java/org/apache/tomcat/util/net/openssl/OpenSSLEngine.java 
(original)
+++ tomcat/trunk/java/org/apache/tomcat/util/net/openssl/OpenSSLEngine.java Sun 
Aug 27 12:21:41 2017
@@ -1001,9 +1001,6 @@ public final class OpenSSLEngine extends
 
 @Override
 public void setUseClientMode(boolean clientMode) {
-if (initialized) {
-return;
-}
 if (clientMode != this.clientMode) {
 throw new UnsupportedOperationException();
 }
@@ -1035,9 +1032,6 @@ public final class OpenSSLEngine extends
 }
 
 private void setClientAuth(ClientAuthMode mode) {
-if (initialized) {
-return;
-}
 if (clientMode) {
 return;
 }



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



Re: [VOTE] Release Apache Tomcat Native 1.2.13

2017-08-27 Thread Rainer Jung

Am 26.08.2017 um 19:11 schrieb Rainer Jung:

Am 25.08.2017 um 23:25 schrieb Rainer Jung:

Am 25.08.2017 um 18:24 schrieb Mark Thomas:

Version 1.2.13 includes the following changes compared to 1.2.12:

- Update minimum recommended OpenSSL version to 1.0.2l
- Update minimum recommended APR version to 1.6.2
- Windows binaries built with OpenSSL 1.0.2l and APR 1.6.2
- Support for the SSL_CONF_cmd API

Various other fixes and improvements. See the changelog for details.

The proposed release artefacts can be found at [1],
and the build was done using tag [2].

The Apache Tomcat Native 1.2.13 is
 [ ] Stable, go ahead and release
 [ ] Broken because of ...


Another thing: the client auth with cert tests (TestClientCert and
TestCustomSsl) fail for me currently when using TC trunk with a Java
connector (NIO/NIO2) and OpenSSL impl (plus tcnative 1.2.13, APR 1.6.2,
OpenSSL 1.0.2l).

I am pretty sure the tests worked when I had added the additional client
CA checks, but they do no longer. But there might have been some other
setup, e.g. OpenSSL 1.1.0 instead of 1.0.2 or something else. i need to
investigate further.

The test does not only fail because the key manager does not get called,
but the access to /unprotected also fails with a 401.

These tests work with APR and also with the JSSE impl.


This was broken by myself in r1805525 and now fixed again in r1806356 by 
partially reverting that change.


Sorry for the noise,

Rainer


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



Re: [VOTE] Release Apache Tomcat Native 1.2.13

2017-08-27 Thread Rainer Jung

Am 25.08.2017 um 23:25 schrieb Rainer Jung:

Am 25.08.2017 um 18:24 schrieb Mark Thomas:

Version 1.2.13 includes the following changes compared to 1.2.12:

- Update minimum recommended OpenSSL version to 1.0.2l
- Update minimum recommended APR version to 1.6.2
- Windows binaries built with OpenSSL 1.0.2l and APR 1.6.2
- Support for the SSL_CONF_cmd API

Various other fixes and improvements. See the changelog for details.

The proposed release artefacts can be found at [1],
and the build was done using tag [2].

The Apache Tomcat Native 1.2.13 is
 [ ] Stable, go ahead and release
 [ ] Broken because of ...


Partial result: although tests in general look good until now, I found
one unit test specific problem on Linux using the APR connector and
OpenSSL 1.0.2.

The new test TestDefaultServletEncoding in TC 9 head executes many test
cases. All of them use the same Tomcat child process, but each test case
runs org.apache.catalina.core.AprLifecycleListener.initializeSSL() which
goes down to the native initialize() in native/src/ssl.c. There we
create a thread local using apr_threadkey_private_create() which in turn
calls pthread_key_create(). That pthread API is limited to creating not
more than PTHREAD_KEYS_MAX keys. On typical linux systems this limit is
1024, but the test runs initializeSSL() for about 2600 times. the first
1024 succeed, the remaining ones throw an exception in initializeSSL()
with errno EAGAIN (which is expected when getting above the limit).

On Solaris the limit is 128 but the problem does not occur. When using
OpenSSL 1.1.0 and above, that part of the native code doesn't run and
the problem also does not show up, the same for Windows. And for JSSE
connectors with OpenSSL impl the AprLifecycleListener only calls
initializeSSL() in the SSL related tests, so not for
TestDefaultServletEncoding in contrast to the APR connector.

I'd expect the initializeSSL() call for real TC and other tcnative using
apps is only called once or at least not extremely often, so that should
not be a problem outside of our unit tests. Still it would be nice if we
could add a cleanup using apr_threadkey_private_delete() somewhere.
Unfortunately in order to be able to call a cleanup from TC code, e.g.
the AprLifecycleListener, we would need an API extension in tcnative first.


Concerning the problem of the thread key cleanup: test 
TestDefaultServletEncoding already calls 
AprLifecycleListener.terminateAPR() during each encoding test, which in 
turn calls Library.terminate(), which does the cleanup of the global 
pool and executes the registered ssl clean up function 
ssl_init_cleanup(). We could add the clean up of the thread_exit_key 
there. For example the following patch works:


Index: native/src/ssl.c
===
--- native/src/ssl.c (revision 1806205)
+++ native/src/ssl.c (working copy)
@@ -52,6 +52,7 @@

 #if ! (defined(WIN32) || defined(WIN64))
 apr_threadkey_t *thread_exit_key;
+static int threadkey_initialized = 0;
 #endif
 #endif

@@ -331,6 +332,12 @@
 return APR_SUCCESS;
 ssl_initialized = 0;

+#if OPENSSL_VERSION_NUMBER < 0x1010L && ! (defined(WIN32) || 
defined(WIN64))

+if (threadkey_initialized) {
+threadkey_initialized = 0;
+apr_threadkey_private_delete(thread_exit_key);
+}
+#endif
 if (tcn_password_callback.cb.obj) {
 JNIEnv *env;
 tcn_get_java_env(&env);
@@ -766,6 +773,7 @@
 tcn_ThrowAPRException(e, err);
 return (jint)err;
 }
+threadkey_initialized = 1;
 #endif
 /* Initialize thread support */
 ssl_thread_setup(tcn_global_pool);


Unfortunately that would be a change to tcnative, so we would have to do 
another tag. I would prefer to include this fix in the release.


Regards,

Rainer

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



svn commit: r1806376 - /tomcat/native/trunk/native/src/ssl.c

2017-08-27 Thread rjung
Author: rjung
Date: Sun Aug 27 16:24:08 2017
New Revision: 1806376

URL: http://svn.apache.org/viewvc?rev=1806376&view=rev
Log:
Fix thread local key leak introduced in r1781943.

The leak breaks TC trunk test TestDefaultServletEncoding
which executes more than 2500 APR init/terminate.

Modified:
tomcat/native/trunk/native/src/ssl.c

Modified: tomcat/native/trunk/native/src/ssl.c
URL: 
http://svn.apache.org/viewvc/tomcat/native/trunk/native/src/ssl.c?rev=1806376&r1=1806375&r2=1806376&view=diff
==
--- tomcat/native/trunk/native/src/ssl.c (original)
+++ tomcat/native/trunk/native/src/ssl.c Sun Aug 27 16:24:08 2017
@@ -52,6 +52,7 @@ struct CRYPTO_dynlock_value {
 
 #if ! (defined(WIN32) || defined(WIN64))
 apr_threadkey_t *thread_exit_key;
+static int threadkey_initialized = 0;
 #endif
 #endif
 
@@ -331,6 +332,12 @@ static apr_status_t ssl_init_cleanup(voi
 return APR_SUCCESS;
 ssl_initialized = 0;
 
+#if OPENSSL_VERSION_NUMBER < 0x1010L && ! (defined(WIN32) || 
defined(WIN64))
+if (threadkey_initialized) {
+threadkey_initialized = 0;
+apr_threadkey_private_delete(thread_exit_key);
+}
+#endif
 if (tcn_password_callback.cb.obj) {
 JNIEnv *env;
 tcn_get_java_env(&env);
@@ -766,6 +773,7 @@ TCN_IMPLEMENT_CALL(jint, SSL, initialize
 tcn_ThrowAPRException(e, err);
 return (jint)err;
 }
+threadkey_initialized = 1;
 #endif
 /* Initialize thread support */
 ssl_thread_setup(tcn_global_pool);



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




Re: [VOTE] Release Apache Tomcat Native 1.2.13

2017-08-27 Thread Mark Thomas
On 27/08/17 15:01, Rainer Jung wrote:



> Unfortunately that would be a change to tcnative, so we would have to do
> another tag. I would prefer to include this fix in the release.

Thanks for tracking these various issues down.

I'm happy to tag 1.2.14. I'll do that early next week.

Before I do, I do want to look into why the TestDefaultServletEncoding
unit tests didn't fail as I would have expected in this case. Another
job for next week.

Kind regards,

Mark

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



svn commit: r1806380 - /tomcat/native/trunk/xdocs/miscellaneous/changelog.xml

2017-08-27 Thread rjung
Author: rjung
Date: Sun Aug 27 17:08:45 2017
New Revision: 1806380

URL: http://svn.apache.org/viewvc?rev=1806380&view=rev
Log:
Update changelog, add entry for r1806376.

Modified:
tomcat/native/trunk/xdocs/miscellaneous/changelog.xml

Modified: tomcat/native/trunk/xdocs/miscellaneous/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/native/trunk/xdocs/miscellaneous/changelog.xml?rev=1806380&r1=1806379&r2=1806380&view=diff
==
--- tomcat/native/trunk/xdocs/miscellaneous/changelog.xml (original)
+++ tomcat/native/trunk/xdocs/miscellaneous/changelog.xml Sun Aug 27 17:08:45 
2017
@@ -34,6 +34,14 @@
   This is the Changelog for Tomcat Native 1.2.
   
 
+
+  
+
+  Fix a thread local key leak. Only relevant when doing
+  SSL.initialize() and Library.terminate() a lot of times. (rjung)
+
+  
+
 
   
 



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



Re: [VOTE] Release Apache Tomcat Native 1.2.13

2017-08-27 Thread Rainer Jung

Am 27.08.2017 um 18:57 schrieb Mark Thomas:

On 27/08/17 15:01, Rainer Jung wrote:




Unfortunately that would be a change to tcnative, so we would have to do
another tag. I would prefer to include this fix in the release.


Thanks for tracking these various issues down.

I'm happy to tag 1.2.14. I'll do that early next week.


Great. In the meantime I can run the tests with OpenSSL 1.1.0 and master.


Before I do, I do want to look into why the TestDefaultServletEncoding
unit tests didn't fail as I would have expected in this case. Another
job for next week.


At least in the logs I find:

...
25-Aug-2017 21:31:17.326 INFO [main] 
org.apache.catalina.startup.LoggingBaseTest.setUp Starting test case 
[testEncoding[1023: contextEnc[ibm850], fileEnc[cp1252], useBom[false], 
target[cp1252], useInclude[true], outputEnc[utf-8], 
callSetCharacterEnc[true], useWriter[false], expectedPass[true]]]
25-Aug-2017 21:31:17.361 INFO [main] 
org.apache.catalina.core.AprLifecycleListener.lifecycleEvent Loaded APR 
based Apache Tomcat Native library [1.2.13] using APR version [1.6.2].
25-Aug-2017 21:31:17.361 INFO [main] 
org.apache.catalina.core.AprLifecycleListener.lifecycleEvent APR 
capabilities: IPv6 [true], sendfile [true], accept filters [false], 
random [true].
25-Aug-2017 21:31:17.361 INFO [main] 
org.apache.catalina.core.AprLifecycleListener.lifecycleEvent APR/OpenSSL 
configuration: useAprConnector [false], useOpenSSL [true]
25-Aug-2017 21:31:16.144 SEVERE [main] 
org.apache.catalina.core.AprLifecycleListener.lifecycleEvent Failed to 
initialize the SSLEngine.

 org.apache.tomcat.jni.Error: 11: Resource temporarily unavailable
at org.apache.tomcat.jni.SSL.initialize(Native Method)
at sun.reflect.GeneratedMethodAccessor26.invoke(Unknown Source)
at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)

at java.lang.reflect.Method.invoke(Method.java:498)
at 
org.apache.catalina.core.AprLifecycleListener.initializeSSL(AprLifecycleListener.java:289)
at 
org.apache.catalina.core.AprLifecycleListener.lifecycleEvent(AprLifecycleListener.java:136)
at 
org.apache.catalina.util.LifecycleBase.fireLifecycleEvent(LifecycleBase.java:123)
at 
org.apache.catalina.util.LifecycleBase.setStateInternal(LifecycleBase.java:424)
at 
org.apache.catalina.util.LifecycleBase.init(LifecycleBase.java:135)
at 
org.apache.catalina.util.LifecycleBase.start(LifecycleBase.java:173)

at org.apache.catalina.startup.Tomcat.start(Tomcat.java:372)
at 
org.apache.catalina.startup.TomcatBaseTest$TomcatWithFastSessionIDs.start(TomcatBaseTest.java:828)
at 
org.apache.catalina.servlets.TestDefaultServletEncoding.testEncoding(TestDefaultServletEncoding.java:191)

...
25-Aug-2017 21:31:17.364 INFO [main] 
org.apache.coyote.AbstractProtocol.init Initializing ProtocolHandler 
["http-apr-127.0.0.1-auto-1024"]
25-Aug-2017 21:31:17.368 INFO [main] 
org.apache.catalina.core.StandardService.startInternal Starting service 
[Tomcat]
25-Aug-2017 21:31:17.368 INFO [main] 
org.apache.catalina.core.StandardEngine.startInternal Starting Servlet 
Engine: Apache Tomcat/9.0.0.M27-dev
25-Aug-2017 21:31:17.450 INFO [main] 
org.apache.coyote.AbstractProtocol.start Starting ProtocolHandler 
["http-apr-127.0.0.1-auto-1024-36488"]
25-Aug-2017 21:31:17.479 INFO [main] 
org.apache.coyote.AbstractProtocol.pause Pausing ProtocolHandler 
["http-apr-127.0.0.1-auto-1024-36488"]
25-Aug-2017 21:31:17.530 INFO [main] 
org.apache.catalina.core.StandardService.stopInternal Stopping service 
[Tomcat]
25-Aug-2017 21:31:17.572 INFO [main] 
org.apache.coyote.AbstractProtocol.stop Stopping ProtocolHandler 
["http-apr-127.0.0.1-auto-1024-36488"]
25-Aug-2017 21:31:17.626 INFO [main] 
org.apache.coyote.AbstractProtocol.destroy Destroying ProtocolHandler 
["http-apr-127.0.0.1-auto-1024-36488"]


So the expection happens in the SSL init part only and the tomcat 
instance is still usable for the test via the http port.


Only on some small virtual machines I had JVM crashes in addtion to the 
above exception. And the exception only shows once we have reached the 
maximum number of about 1024 keys.


Regards,

Rainer

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



svn commit: r1806399 - in /tomcat/native/trunk: native/src/sslinfo.c xdocs/miscellaneous/changelog.xml

2017-08-27 Thread rjung
Author: rjung
Date: Mon Aug 28 03:19:15 2017
New Revision: 1806399

URL: http://svn.apache.org/viewvc?rev=1806399&view=rev
Log:
Replace use of deprecated ASN1_STRING_data with
ASN1_STRING_get0_data when building against
OpenSSL 1.1.0 and newer.

Modified:
tomcat/native/trunk/native/src/sslinfo.c
tomcat/native/trunk/xdocs/miscellaneous/changelog.xml

Modified: tomcat/native/trunk/native/src/sslinfo.c
URL: 
http://svn.apache.org/viewvc/tomcat/native/trunk/native/src/sslinfo.c?rev=1806399&r1=1806398&r2=1806399&view=diff
==
--- tomcat/native/trunk/native/src/sslinfo.c (original)
+++ tomcat/native/trunk/native/src/sslinfo.c Mon Aug 28 03:19:15 2017
@@ -182,7 +182,11 @@ static char *lookup_ssl_cert_dn(X509_NAM
 ASN1_STRING *adata = X509_NAME_ENTRY_get_data(xsne);
 int len = ASN1_STRING_length(adata);
 result = malloc(len + 1);
+#if OPENSSL_VERSION_NUMBER < 0x1010L || defined(LIBRESSL_VERSION_NUMBER)
 memcpy(result, ASN1_STRING_data(adata), len);
+#else
+memcpy(result, ASN1_STRING_get0_data(adata), len);
+#endif
 result[len] = '\0';
 
 #if APR_CHARSET_EBCDIC

Modified: tomcat/native/trunk/xdocs/miscellaneous/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/native/trunk/xdocs/miscellaneous/changelog.xml?rev=1806399&r1=1806398&r2=1806399&view=diff
==
--- tomcat/native/trunk/xdocs/miscellaneous/changelog.xml (original)
+++ tomcat/native/trunk/xdocs/miscellaneous/changelog.xml Mon Aug 28 03:19:15 
2017
@@ -37,6 +37,11 @@
 
   
 
+  Replace use of deprecated ASN1_STRING_data with
+  ASN1_STRING_get0_data when building against
+  OpenSSL 1.1.0 and newer. (rjung)
+
+
   Fix a thread local key leak. Only relevant when doing
   SSL.initialize() and Library.terminate() a lot of times. (rjung)
 



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



[Bug 61450] issue when certificateKeyAlias is not set

2017-08-27 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=61450

Svetlin Zarev  changed:

   What|Removed |Added

 CC||svetlin.za...@abv.bg

-- 
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: r1806404 - in /tomcat/native/trunk: native/src/sslcontext.c xdocs/miscellaneous/changelog.xml

2017-08-27 Thread rjung
Author: rjung
Date: Mon Aug 28 06:49:29 2017
New Revision: 1806404

URL: http://svn.apache.org/viewvc?rev=1806404&view=rev
Log:
Fix a small memory leak during certificate initialization.

Also silence a compiler warning: SSL_CTX_set_ecdh_auto() isn't
needed for OpenSSL 1.1.0 and above and using it there results
in a compiler warning because the compat macro is a noop.

Backport of r1735770 from mod_ssl and partial backport of
r1787728 also from mod_ssl.

Modified:
tomcat/native/trunk/native/src/sslcontext.c
tomcat/native/trunk/xdocs/miscellaneous/changelog.xml

Modified: tomcat/native/trunk/native/src/sslcontext.c
URL: 
http://svn.apache.org/viewvc/tomcat/native/trunk/native/src/sslcontext.c?rev=1806404&r1=1806403&r2=1806404&view=diff
==
--- tomcat/native/trunk/native/src/sslcontext.c (original)
+++ tomcat/native/trunk/native/src/sslcontext.c Mon Aug 28 06:49:29 2017
@@ -953,7 +953,7 @@ TCN_IMPLEMENT_CALL(jboolean, SSLContext,
 const char *p;
 char err[256];
 #ifdef HAVE_ECC
-EC_GROUP *ecparams;
+EC_GROUP *ecparams = NULL;
 int nid;
 EC_KEY *eckey = NULL;
 #endif
@@ -1034,6 +1034,7 @@ TCN_IMPLEMENT_CALL(jboolean, SSLContext,
  * If only for PEM files move above to the PEM handling */
 if ((idx == 0) && (dhparams = SSL_dh_GetParamFromFile(cert_file))) {
 SSL_CTX_set_tmp_dh(c->ctx, dhparams);
+DH_free(dhparams);
 }
 
 #ifdef HAVE_ECC
@@ -1048,8 +1049,11 @@ TCN_IMPLEMENT_CALL(jboolean, SSLContext,
 SSL_CTX_set_tmp_ecdh(c->ctx, eckey);
 }
 /*
- * ...otherwise, configure NIST P-256 (required to enable ECDHE)
+ * ...otherwise, enable auto curve selection (OpenSSL 1.0.2)
+ * or configure NIST P-256 (required to enable ECDHE for earlier versions)
+ * ECDH is always enabled in 1.1.0 unless excluded from SSLCipherList
  */
+#if (OPENSSL_VERSION_NUMBER < 0x1010L)
 else {
 #if defined(SSL_CTX_set_ecdh_auto)
 SSL_CTX_set_ecdh_auto(c->ctx, 1);
@@ -1058,7 +1062,10 @@ TCN_IMPLEMENT_CALL(jboolean, SSLContext,
 SSL_CTX_set_tmp_ecdh(c->ctx, eckey);
 #endif
 }
+#endif
+/* OpenSSL assures us that _free() is NULL-safe */
 EC_KEY_free(eckey);
+EC_GROUP_free(ecparams);
 #endif
 SSL_CTX_set_tmp_dh_callback(c->ctx, SSL_callback_tmp_DH);
 
@@ -1168,6 +1175,7 @@ TCN_IMPLEMENT_CALL(jboolean, SSLContext,
 /*
  * TODO try to read the ECDH curve name from somewhere...
  */
+#if (OPENSSL_VERSION_NUMBER < 0x1010L)
 #if defined(SSL_CTX_set_ecdh_auto)
 SSL_CTX_set_ecdh_auto(c->ctx, 1);
 #else
@@ -1176,6 +1184,7 @@ TCN_IMPLEMENT_CALL(jboolean, SSLContext,
 EC_KEY_free(eckey);
 #endif
 #endif
+#endif
 SSL_CTX_set_tmp_dh_callback(c->ctx, SSL_callback_tmp_DH);
 cleanup:
 free(key);

Modified: tomcat/native/trunk/xdocs/miscellaneous/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/native/trunk/xdocs/miscellaneous/changelog.xml?rev=1806404&r1=1806403&r2=1806404&view=diff
==
--- tomcat/native/trunk/xdocs/miscellaneous/changelog.xml (original)
+++ tomcat/native/trunk/xdocs/miscellaneous/changelog.xml Mon Aug 28 06:49:29 
2017
@@ -37,6 +37,9 @@
 
   
 
+  Fix a small memory leak during certificate initialization. (rjung)
+
+
   Replace use of deprecated ASN1_STRING_data with
   ASN1_STRING_get0_data when building against
   OpenSSL 1.1.0 and newer. (rjung)



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