bug in mod_jaxcent

[kneubi]

I have compiled mod_jaxcent.cpp for Apache, and have noticed that the
"Reloadable Classpath" does not work.
The samples did only when i did put the the directory to the "main"
classpath setting.

I have reviewed the source code from mod_jaxcent.cpp and found some
copy and paste typos.
On line 152 and 153 the function set_reloadable_classes() is called
instead of the real functions corresponding to the config options.

Here is the patch to fix this issue:
--------------------------------------------------------------------------------------------------------------------
*** mod_jaxcent.cpp.orig Fri Sep 11 14:48:36 2009
--- mod_jaxcent.cpp Fri Sep 11 14:42:21 2009
***************
*** 149,156 ****
AP_INIT_RAW_ARGS("JaxcentConfigXML", (CmdFn) set_config_file,
NULL, RSRC_CONF|ACCESS_CONF, "Jaxcent configuration XML file"),
AP_INIT_RAW_ARGS("ReloadableClasses", (CmdFn)
set_reloadable_classes, NULL, RSRC_CONF|ACCESS_CONF, "Jaxcent
Reloadable classpath"),
AP_INIT_RAW_ARGS("DisplayServletMessage", (CmdFn)
set_display_servlet_message, NULL, RSRC_CONF|ACCESS_CONF, "Whether to
display standard message or error 404"),
! AP_INIT_RAW_ARGS("RefreshURLMap", (CmdFn)
set_reloadable_classes, NULL, RSRC_CONF|ACCESS_CONF, "Whether to re-
read the config XML file when it changes"),
! AP_INIT_RAW_ARGS("KeepAliveTimeout", (CmdFn)
set_reloadable_classes, NULL, RSRC_CONF|ACCESS_CONF, "Keep alive
timeout in seconds"),
AP_INIT_RAW_ARGS("Parameter", (CmdFn) set_parameter, NULL,
RSRC_CONF|ACCESS_CONF, "Parameter(s) to be passed to the Jaxcent
framework"),

{ NULL }
--- 149,156 ----
AP_INIT_RAW_ARGS("JaxcentConfigXML", (CmdFn) set_config_file,
NULL, RSRC_CONF|ACCESS_CONF, "Jaxcent configuration XML file"),
AP_INIT_RAW_ARGS("ReloadableClasses", (CmdFn)
set_reloadable_classes, NULL, RSRC_CONF|ACCESS_CONF, "Jaxcent
Reloadable classpath"),
AP_INIT_RAW_ARGS("DisplayServletMessage", (CmdFn)
set_display_servlet_message, NULL, RSRC_CONF|ACCESS_CONF, "Whether to
display standard message or error 404"),
! AP_INIT_RAW_ARGS("RefreshURLMap", (CmdFn) set_refresh_url_map,
NULL, RSRC_CONF|ACCESS_CONF, "Whether to re-read the config XML file
when it changes"),
! AP_INIT_RAW_ARGS("KeepAliveTimeout", (CmdFn)
set_keep_alive_timeout, NULL, RSRC_CONF|ACCESS_CONF, "Keep alive
timeout in seconds"),
AP_INIT_RAW_ARGS("Parameter", (CmdFn) set_parameter, NULL,
RSRC_CONF|ACCESS_CONF, "Parameter(s) to be passed to the Jaxcent
framework"),

{ NULL }
--------------------------------------------------------------------------------------------------------------------

Afterwards the samples work as expected and the classes are reloaded
when they change.

Furthermore the download link at http://www.jaxcent.com/download.html
does still point to version 0.20 instead of version 0.21

Regards,

Kneubi

[Jaxcent]

Thanks. You are right on both the issues. They have been fixed.

On Sep 11, 6:26 am, kneubi <beat.kneubu...@rega-sense.ch> wrote:
> I have compiled mod_jaxcent.cpp for Apache, and have noticed that the
> "Reloadable Classpath" does not work.
> The samples did only when i did put the the directory to the "main"
> classpath setting.
>
> I have reviewed the source code from mod_jaxcent.cpp and found some
> copy and paste typos.
> On line 152 and 153 the function set_reloadable_classes() is called
> instead of the real functions corresponding to the config options.
>
> Here is the patch to fix this issue:

> ---------------------------------------------------------------------------­-----------------------------------------

> ---------------------------------------------------------------------------­-----------------------------------------

>
> Afterwards the samples work as expected and the classes are reloaded
> when they change.
>

> Furthermore the download link athttp://www.jaxcent.com/download.html