Jakarta ISAPI Filter Patch
Hi! I did some modifications to the Jakarta ISAPI filter in connectors/trunk/jk/native. It would be nice to see most of them included the distribution. 1. a new worker parameter: socket_nodelay, default 1 2. parameter socket_timeout: the documentation stated that if the parameter was set to zero it would wait infinitely, however, the code says: if it is set larger than zero, change it, i.e. if it were set to zero the system default would be used. The new implementation is: If it is set to -1 the socket timeout is set to infinitely. If it is set to larger or equal to zero, the socket timeout is set to the given value in seconds. Any other value will use the system default. 3. The status worker has a new button, that reloads the configuration. This allows one to change the filter settigns without having to restart IIS. This feature is not tested thoroughly, e.g. we have to make sure that all incoming requests are stalled and all active requests are served until the configuration is reloaded. 4. Errors in WriteClient() are ignored, see http://west-wind.com/weblog/posts/1266.aspx 5. The last parameter in the ServerSupportFunction should be the length of the headers buffer (not the length of the status buffer) Martin - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
RE: Jakarta ISAPI Filter Patch
I forgot that the mailing list doesn't accept attachments. I created a bugzilla issue and attached the patch to it. The bug number is #39552 (http://issues.apache.org/bugzilla/show_bug.cgi?id=39552), The attachment i #18254 (http://issues.apache.org/bugzilla/attachment.cgi?id=18254). Martin > -Original Message- > From: Martin Goldhahn [mailto:[EMAIL PROTECTED] > Sent: Thursday, May 11, 2006 9:50 AM > To: dev@tomcat.apache.org > Subject: Jakarta ISAPI Filter Patch > > Hi! > I did some modifications to the Jakarta ISAPI filter in > connectors/trunk/jk/native. It would be nice to see most of > them included the distribution. > 1. a new worker parameter: socket_nodelay, default 1 2. > parameter socket_timeout: the documentation stated that if > the parameter was set to zero it would wait infinitely, > however, the code > says: if it is set larger than zero, change it, i.e. if it > were set to zero the system default would be used. The new > implementation is: If it is set to -1 the socket timeout is > set to infinitely. If it is set to larger or equal to zero, > the socket timeout is set to the given value in seconds. Any > other value will use the system default. > 3. The status worker has a new button, that reloads the configuration. > This allows one to change the filter settigns without having > to restart IIS. This feature is not tested thoroughly, e.g. > we have to make sure that all incoming requests are stalled > and all active requests are served until the configuration is > reloaded. > 4. Errors in WriteClient() are ignored, see > http://west-wind.com/weblog/posts/1266.aspx > 5. The last parameter in the ServerSupportFunction should be > the length of the headers buffer (not the length of the status buffer) > > Martin > > - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
RE: Jakarta ISAPI Filter Patch
I reopened the case. The patch is for the iis filter. Maybe you got confused by the mail subject. ISAPI is just the API that the redirector uses. And the name of the DLL is isapi_redirect.dll. Hence the subject. Sorry for the confusion. Martin > -Original Message- > From: Mladen Turk [mailto:[EMAIL PROTECTED] > Sent: Thursday, May 11, 2006 11:35 AM > To: Tomcat Developers List > Subject: Re: Jakarta ISAPI Filter Patch > > Martin Goldhahn wrote: > > Hi! > > I did some modifications to the Jakarta ISAPI filter in > > connectors/trunk/jk/native. It would be nice to see most of them > > included the distribution. > > Like explained in the bz.. > This connector is unsupported! > Use native/iis instead for using iside Microsoft IIS. > Other one unsupported is domino connector. > > I doubt anyone will drop a tear because of that. > I'm only sorry you took a lots of time for that patch. > > Regards, > Mladen. > > - > To unsubscribe, e-mail: [EMAIL PROTECTED] For > additional commands, e-mail: [EMAIL PROTECTED] > > - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
