[Bug 55078] Configuring a DataSource Resource with dataSourceJNDI does not work as expected

2013-06-07 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=55078 --- Comment #2 from Timothy Stewart --- This problem isn't quite what I thought. I have some other issue than the one I described. The org.apache.tomcat.jdbc.pool.DataSourceFactory has code to resolve the DataSource from JNDI, and set it

[Bug 55078] Configuring a DataSource Resource with dataSourceJNDI does not work as expected

2013-06-07 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=55078 --- Comment #1 from Timothy Stewart --- Created attachment 30407 --> https://issues.apache.org/bugzilla/attachment.cgi?id=30407&action=edit Possible fix Replaced the TODO with a JNDI lookup and set the datasource on the poolProperties be

[Bug 51813] Tomcat randomly crashes with [libtcnative-1.so.1+0x152ca] Java_org_apache_tomcat_jni_Socket_sendbb+0x5a

2013-06-07 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=51813 --- Comment #9 from Christopher Schultz --- (In reply to Rainer Jung from comment #8) > (In reply to Christopher Schultz from comment #7) > > Created attachment 30402 [details] > > Patch against tcnative-1.1.x > > > > I'm not sure if I sho

[Bug 55078] New: Configuring a DataSource Resource with dataSourceJNDI does not work as expected

2013-06-07 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=55078 Bug ID: 55078 Summary: Configuring a DataSource Resource with dataSourceJNDI does not work as expected Product: Tomcat 7 Version: trunk Hardware: All

[Bug 55071] try finally catch structure masks original exception in JDBCRealm.getPassword()

2013-06-07 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=55071 --- Comment #4 from bruce weertman --- Thanks I've made the switch. -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: d

[Bug 55071] try finally catch structure masks original exception in JDBCRealm.getPassword()

2013-06-07 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=55071 --- Comment #3 from Christopher Schultz --- While this is likely to be fixed, you should understand that JDBCRealm is not appropriate for any real-world situation. Instead, you should use DataSourceRealm, which is much more robust. -- You

[Bug 51813] Tomcat randomly crashes with [libtcnative-1.so.1+0x152ca] Java_org_apache_tomcat_jni_Socket_sendbb+0x5a

2013-06-07 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=51813 --- Comment #8 from Rainer Jung --- (In reply to Christopher Schultz from comment #7) > Created attachment 30402 [details] > Patch against tcnative-1.1.x > > I'm not sure if I should be checking s->net or s->sock. They both seem to be > se

[Bug 55071] try finally catch structure masks original exception in JDBCRealm.getPassword()

2013-06-07 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=55071 --- Comment #2 from bruce weertman --- I was wondering about that. I never do that on SELECT statements myself. After I read that piece of code, I was thinking that perhaps I've been doing SQL wrong for all these years! Looking at http://

Re: Where to put proposals for tcnative?

2013-06-07 Thread Christopher Schultz
Rainer, On 6/6/13 4:54 PM, Rainer Jung wrote: > On 06.06.2013 19:56, Christopher Schultz wrote: >> Rainer, >> >> On 6/6/13 1:32 PM, Rainer Jung wrote: >>> On 06.06.2013 18:22, Christopher Schultz wrote: > >>> Proposals mostly for the C files probably make sense in a STATUS file >>> here if they a

Re: Where to put proposals for tcnative?

2013-06-07 Thread Rainer Jung
On 07.06.2013 16:05, Christopher Schultz wrote: > Rainer, > In this particular case, it seems like my changes should be relatively > compatible between the two. Should I patch trunk and back-port, patch > 1.1 and forward-port, or patch 1.1 and ignore trunk (thus making the > problem worse). I susp

Re: [VOTE] Release Apache Tomcat 7.0.41

2013-06-07 Thread Rainer Jung
On 06.06.2013 14:06, Mark Thomas wrote: > The proposed Apache Tomcat 7.0.41 release is now available for voting. > > It can be obtained from: > https://dist.apache.org/repos/dist/dev/tomcat/tomcat-7/v7.0.41/ > The Maven staging repo is: > https://repository.apache.org/content/repositories/orgapach

[Bug 55073] wasted work in ValidateVisitor.isExpression()

2013-06-07 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=55073 Adrian Nistor changed: What|Removed |Added Keywords||PatchAvailable CC|

[Bug 55073] New: wasted work in ValidateVisitor.isExpression()

2013-06-07 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=55073 Bug ID: 55073 Summary: wasted work in ValidateVisitor.isExpression() Product: Tomcat 7 Version: 7.0.40 Hardware: PC OS: Linux Status: NEW Severity:

[Bug 55071] try finally catch structure masks original exception in JDBCRealm.getPassword()

2013-06-07 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=55071 --- Comment #1 from Christopher Schultz --- That commit() looks weird to me. COMMIT after SELECT? -- You are receiving this mail because: You are the assignee for the bug. -

Re: Where to put proposals for tcnative?

2013-06-07 Thread Christopher Schultz
Rainer, On 6/6/13 5:24 PM, Rainer Jung wrote: > On 06.06.2013 23:00, Caldarale, Charles R wrote: >>> From: Christopher Schultz [mailto:ch...@christopherschultz.net] >>> Subject: Re: Where to put proposals for tcnative? >> >>> PS Here's what I get when trying to build tcnative-trunk: >> >>> $ sh b

Re: Where to put proposals for tcnative?

2013-06-07 Thread Christopher Schultz
Rainer, On 6/6/13 5:06 PM, Rainer Jung wrote: > On 06.06.2013 22:54, Rainer Jung wrote: >> On 06.06.2013 19:56, Christopher Schultz wrote: >>> Rainer, >>> >>> On 6/6/13 1:32 PM, Rainer Jung wrote: On 06.06.2013 18:22, Christopher Schultz wrote: >> Proposals mostly for the C files probabl

Re: [VOTE] Release Apache Tomcat 7.0.41

2013-06-07 Thread Ognjen Blagojevic
Mark, On 6.6.2013 14:06, Mark Thomas wrote: The proposed Apache Tomcat 7.0.41 release is now available for voting. It can be obtained from: https://dist.apache.org/repos/dist/dev/tomcat/tomcat-7/v7.0.41/ The Maven staging repo is: https://repository.apache.org/content/repositories/orgapachetomc