fix(agent): fix bug in observe option check

coapjs / node-coap

CoAP - Node.js style

MIT License

528 stars 154 forks source link

fix(agent): fix bug in observe option check #337

Closed JKRhb closed 2 years ago

JKRhb commented 2 years ago

While #330 mostly fixed the problems with the observe request parameter, it also introduced a new bug that caused the Agent to always set the observe option to 0 in requests. This PR fixes the issue, making the Agent behave normally again.

coveralls commented 2 years ago

Pull Request Test Coverage Report for Build 2191099491

1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.01%) to 87.321%

Totals
Change from base Build 2177950353:	-0.01%
Covered Lines:	1143
Relevant Lines:	1260

💛 - Coveralls

Apollon77 commented 2 years ago

Do you want to dorectly include versdion increase for release?

JKRhb commented 2 years ago

Do you want to dorectly include versdion increase for release?

Oh, sure, I'll update the PR :)

JKRhb commented 2 years ago

Done :)