Hey this is cool :).

[EMAIL PROTECTED] wrote:
Log:
add JkStausUpdate task for mod:jk status worker (greater mod_jk 1.2.9 only)

I thought it was meant to be an XML, to pick up the config from apache. Also, can you keep the cmd=update and add something like: "&mime=html|xml|txt" ?

Further more 'lr' and 'lb' params are using integer values (worker id's)
so you will need to use integers or change the code to accept strings.

The reason why I choose the integers or id's was because in that case
there is no need to url encode worker names. Of course we can what
documentation says accept only alnums for worker names, but that will
break backward compatibility in some cases.
I would like for this version to add ERROR message in the log if
worker name is non-url safe, but still allow it for now.

IMO ant task should pick up default values from apache directly by
parsing xml output.

Regards,
Mladen

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Reply via email to