fix: reset object payload to null

dafortune / hapi-qs

Bring back qs support for hapi 12

10 stars 7 forks source link

fix: reset object payload to null #13

Closed ro-ka closed 8 years ago

ro-ka commented 8 years ago

Fixes #12.

dafortune commented 8 years ago

Hi @ro-ka thanks for your PR. Unfortunatelly part of this plugin goal is to bring back the same behavior qs feature used to have on Hapi<=11, I've just tested it and did see that current behavior is the one expected on hapi@11, so I don't think that changing the default behavior is good in this case.

Let me investigate the issue regarding what you pointed out (probably tomorrow) and if there is a good use case for it we might want to add this as an opt-in feature instead of a default one.

ro-ka commented 8 years ago

As noted in #12, I found a workaround for my problem. Will close this PR as there might be a better solution than the one I provided as a quick fix without deeper knowledge of the Hapi origins.