jkyberneees / fastify-gateway

A Node.js API gateway that just works!
https://www.npmjs.com/package/k-fastify-gateway
MIT License
108 stars 14 forks source link

Update fastify-reply-from to the latest version 🚀 #10

Closed greenkeeper[bot] closed 5 years ago

greenkeeper[bot] commented 5 years ago

The devDependency fastify-reply-from was updated from 0.5.4 to 0.6.0.

This version is not covered by your current version range.

If you don’t accept this pull request, your project will work just like it did before. However, you might be missing out on a bunch of new features, fixes and/or performance improvements from the dependency update.


Release Notes for v0.6.0
  • Change onresponse contract to allow request, reply and res #43
Commits

The new version differs by 4 commits.

  • 9718e8a Bumped v0.6.0.
  • cf85a79 updated deps
  • 171abe4 Change onresponse contract to allow request, reply, and res. (#43)
  • 4e27722 fix(package): update tiny-lru to version 6.0.1 (#42)

See the full diff

FAQ and help There is a collection of [frequently asked questions](https://greenkeeper.io/faq.html). If those don’t help, you can always [ask the humans behind Greenkeeper](https://github.com/greenkeeperio/greenkeeper/issues/new).

Your Greenkeeper bot :palm_tree:

justintime4tea commented 5 years ago

I was having zero success with the gateway until I noticed this PR and tested downgrading my fastify-reply-from to 0.5.4. For whatever reason fastify-reply-from 0.6.0 has breaking changes so I think this PR needs additional files changed to be compatible with 0.6.0. I tested the demo and experienced the same issue as in my own project.

jkyberneees commented 5 years ago

I was having zero success with the gateway until I noticed this PR and tested downgrading my fastify-reply-from to 0.5.4. For whatever reason fastify-reply-from 0.6.0 has breaking changes so I think this PR needs additional files changed to be compatible with 0.6.0. I tested the demo and experienced the same issue as in my own project.

Hi @justintime4tea, you are right, there is a breaking change in fastify-reply-from v0.6.x. I will release an update accordingly in v2.x. Thanks

jkyberneees commented 5 years ago

JFYI @justintime4tea: https://www.npmjs.com/package/k-fastify-gateway v2 is out!