Hooks context about fastify

[mcollina]

Let's leave it as it is and this issue open, and maybe we'll do that if there is a need.

[jsumners]

I claim there is definitely a need for this. As a concrete example, I am attempting to use it in fastify-caching to access this.cache so I can store an indicator that the resource can be served from the user's cache:

instance.addHook('onResponse', function (res, next) {
  if (!res.hasHeader('etag')) return next()
  this.cache.set(res.getHeader('etag'), true, 1000, next)
})

[mcollina]

definitely 👍 . I think we should work on a PR after #330 lands.

[jsumners]

Is there anyone familiar with how this works that is willing to start the PR now that #330 is merged? cc: @fastify/fastify

[StarpTech]

Should be easy, here a start:

• onSend:

  fastify/lib/reply.js

  Line 212 in 6d596ab

  fn(this.request, this.reply, this.payload, cb)

• fastify/fastify.js

  Line 252 in 6d596ab

  fn(this.req, this.res, cb)

[StarpTech]

I created a PR #408

[StarpTech]

I created a new one #414

[jsumners]

It'd be really great if you simply pulled in upstream changes to an existing PR to resolve conflicts in the future.

[StarpTech]

For me it was easier to create a new one instead to merge the changes. I can take care about this in future.

[StarpTech]

Fixed in #414