Skip to content

Conversation

@eugenioclrc
Copy link

Sometimes yo need to avoid cache on certains errors;

app.get('/dont-cache-onthefly', cacher.cache('day'), function(req, res) {
  // some complex stuff
  // if have an error response dont cache it:
    req.noCaching = true;
  // endif

  res.send(complexStuff)
})

@addisonj
Copy link
Owner

addisonj commented Nov 8, 2015

This looks good! I am away from my computer but can get this merged and published later today

@eugenioclrc
Copy link
Author

Hi @addisonj could you approve the pull request and publish the module? 🙏

@addisonj
Copy link
Owner

Dang! Sorry to have spaced this for so long,I blame the holidays....

I noticed one latent semicolon in the test fixture and forgot to circle
back on it. If you want to fix it, feel free otherwise I will fix when I
merge (hopefully later today)

Thanks for the reminder!

On Wed, Jan 6, 2016, 3:07 PM Eugenio [email protected] wrote:

Hi @addisonj https://github.com/addisonj could you approve the pull
request and publish the module? [image: 🙏]


Reply to this email directly or view it on GitHub
#28 (comment).

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