search

movio / bramble

A federated GraphQL API gateway
https://movio.github.io/bramble/
MIT License
497 stars 55 forks source link

Ensure the destination map for merging is never nil #160

Closed lucianjon closed 2 years ago

lucianjon commented 2 years ago

Bramble currently panic when the data in the first entry of executionResults was marshalled from a json null. This is due to us ending up with a typed nil as our destination map that wasn't being caught.

The PR ensures that the first destination map passed into the merge is never nil.