Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add How-to on caching compressed content #100

Closed
wants to merge 4 commits into from

Conversation

nisbet-hubbard
Copy link

This is an oft-asked question on various support forums, and the usual answer was: no, it can’t.

This PR supplies the missing tutorial.

@y82 y82 self-assigned this Nov 6, 2024
@y82
Copy link
Collaborator

y82 commented Nov 6, 2024

Hi @nisbet-hubbard,

Thank you for your contribution - we appreciate your work on the patch. However, as the brotli module is a third-party module outside of nginx codebase and maintenance, this change cannot be incorporated.

Please also consider the following guidelines:

  • All changes should be combined into a single commit before opening a pull request
  • Refer to the README for the guidelines on commit message formatting
  • The initial doc revision is rev="1"

Thank you once again for your time and effort.

@nisbet-hubbard
Copy link
Author

Closing in favour of #105.

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

Successfully merging this pull request may close these issues.

2 participants