JialuZhang commented on issue #1291:
URL:
https://github.com/apache/incubator-pagespeed-mod/issues/1291#issuecomment-785151687
@GuillaumeRossolini
In your posted configuration, the line: "RewriteEngine Off" is a
misconfiguration (all three of "RewriteEngine Off" are misconfiguration), and
adding it to your system will not change any system behavior.
Why "RewriteEngine Off" is allowed by Apache is that, if you include
multiple "RewriteRule" parameters in your configuration, then instead of
commenting them all, you can explicitly using “RewriteEngine Off” to disable
all "RewriteRule".
More importantly, the default value of “RewriteEngine" is already an "off",
so adding “RewriteEngine Off" is quite unnecessary and it may cause confusion
to users.
Since herein there is no "RewriteRule", deleting “RewriteEngine Off” would
be ideal.
Related Apache source code snippet:
```
run_rewritemap_programs(server_rec *s , apr_pool_t *p){
if (conf->state == ENGINE_DISABLED) { // usage of "RewriteEngine"
return APR_SUCCESS; // early return
rewritemap_program(...); // usage of "RewriteRule"
}
```
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
