Skip to content

Added glob matching feature for setting cache headers. - #183

Merged
brettz9 merged 1 commit into
cloudhead:masterfrom
lightswitch05:feature/allow-greater-cache-control
May 21, 2021
Merged

Added glob matching feature for setting cache headers.#183
brettz9 merged 1 commit into
cloudhead:masterfrom
lightswitch05:feature/allow-greater-cache-control

Conversation

@lightswitch05

Copy link
Copy Markdown
Contributor
  • Keep backwards compatibility with previous cache options
  • Add tests to ensure glob matching with defined cache values
  • Updated README with instructions for new cache feature

Example Usage:

fileServer = new static.Server('/public', {
cache: {
'**/*.txt': 100,
'**/': 300
}
});

@lightswitch05
lightswitch05force-pushed the feature/allow-greater-cache-control branch from 277f632 to b341fc0CompareSeptember 27, 2016 15:54
* Keep backwards compatibility with previous cache options
* Add tests to ensure glob matching with defined cache values
* Updated README with instructions for new cache feature
@lightswitch05
lightswitch05force-pushed the feature/allow-greater-cache-control branch from b341fc0 to 6ccc79aCompareSeptember 27, 2016 15:56
@lightswitch05

Copy link
Copy Markdown
ContributorAuthor

Any comments or thoughts on this pull request?

@v29neil

Copy link
Copy Markdown

any progress on this feature ?

@brettz9
brettz9 merged commit e53e174 into cloudhead:masterMay 21, 2021
Zarel pushed a commit to Zarel/node-static that referenced this pull request Aug 10, 2021
* Keep backwards compatibility with previous cache options
* Add tests to ensure glob matching with defined cache values
* Updated README with instructions for new cache feature
Zarel pushed a commit to Zarel/node-static that referenced this pull request Aug 10, 2021
* Keep backwards compatibility with previous cache options
* Add tests to ensure glob matching with defined cache values
* Updated README with instructions for new cache feature
Sign up for freeto 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.

3 participants

@lightswitch05@v29neil@brettz9