Skip to content
This repository has been archived by the owner on Apr 21, 2023. It is now read-only.

PreserveUrls doc is misleading with its mention of 'forbid' #1488

Open
jmarantz opened this issue Jan 26, 2017 · 1 comment
Open

PreserveUrls doc is misleading with its mention of 'forbid' #1488

jmarantz opened this issue Jan 26, 2017 · 1 comment
Assignees

Comments

@jmarantz
Copy link
Contributor

The doc cllaims that setting *PreserveUrls options forbids the use of certain filters. This was true when the *PreserveUrls options were initially created, but became false once OptimizeForBandwidth was created, allowing for configuration overrides at different levels (root/vhost/directory/options) to take precedence.

See discussion on mod-pagespeed-discuss: https://groups.google.com/d/msgid/mod-pagespeed-discuss/a6708ff5-1ce6-47b3-8313-bff425cafe5f%40googlegroups.com?utm_medium=email&utm_source=footer

The doc should be updated.

@jmarantz jmarantz self-assigned this Jan 26, 2017
@michael-niemand
Copy link

michael-niemand commented Jan 26, 2017

+1
You guys are great. Thanks for the quick response and taking action.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants