I like the second option.the two conditional branches simply have to be swapped. Or better we could remove the else-branch and put the disabled-check before the 'if'. Or...?
if engine is disabled
decline
if symlinks-not-allowed
error with forbidden
Sounds right to me. Feel like coding it up and committing it? =)
I'd commit it to httpd-2.0 (as that code is there too), then backport to 1.3 when it gets approved for the 2.0 branch. *insert mumble about too many backports* -- justin
