fix bad use of http.Response - Githubissues

openfaas / nats-queue-worker

Queue-worker for OpenFaaS with NATS Streaming

https://docs.openfaas.com/reference/async/

MIT License

129 stars 59 forks source link

fix bad use of http.Response #68

Closed chennqqi closed 5 years ago

chennqqi commented 5 years ago

Signed-off-by: chennqqi chennqqi@qq.com

fix bad use of http.Response

Description

fix bad use of http.Response http.Response arguments of client.Do(request) only can be call if err == nil

Motivation and Context

How Has This Been Tested?

tested add panic function into faas function then do a sync call to check if nats-worker crash

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I've read the CONTRIBUTION guide
[x] I have signed-off my commits with git commit -s
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.