mod_imap update?

["Aram W. Mirzadeh" <aw...@qosina.com>]

This makes sense....  should I make a patch and put it in for a vote on the 
next release?

>From: djk@chbi.co.uk
>Subject: WWW Form Bug Report: "mod_imap.c doesn't tell WHY it returns
SERVER_ERROR" on Linux
>
>Symptoms:
>--
>as per one line summary, you just return server_error with
>no clue as to why, this makes one suspect the server! (erronously)
>
>Please would you make SURE that all server_errors return useful error
>messages in error_log so we determine the _html_ errors
>that we are making.
>
>The patch supplied below is against mod_imap_new.c in contrib
>(or patches, can't remember which)
>--
>
>Backtrace:
>--
>--- /home/djk/mod_imap_new.c    Mon Jan 15 09:17:23 1996
>+++ mod_imap.c  Mon Jan 15 10:28:15 1996
>@@ -367,8 +367,10 @@
>
>   FILE *imap;
>
>-  if (! (imap = fopen(r->filename, "r")))
>+  if (! (imap = fopen(r->filename, "r"))) {
>+   log_reason(sys_errlist[errno], r->filename, r);
>     return SERVER_ERROR;
>+  }
>
>   if (r->server->port == 80 )
>     sprintf(base_uri, "http://%s", r->server->server_hostname);
>@@ -494,6 +496,7 @@
>     return REDIRECT;
>   }
>
>+  log_reason("Nothing matches and no default", r->filename, r);
>   return SERVER_ERROR;  /* they lose -- but we get here if and only if there
>               was no default and nothing matched */

This is the 2nd message from the same poster. 

>The slash between the address part of the URL isn't 
>inserted between it and the relative url presented. But
>this DOES work correctly on the issued mod_imap.c (so the
>patch I sent needs to be put onto the stock mod_imap.c
>

<Aram>
--
Aram W. Mirzadeh, MIS Manager, Qosina Corporation
http://www.qosina.com/~awm/, awm@qosina.com
Apache httpd server team http://www.apache.org