search

opentok / opentok-node

OpenTok Server SDK for node.js
MIT License
164 stars 167 forks source link

fix: post encoding for client #318

Closed manchuck closed 11 months ago

manchuck commented 1 year ago

What is this PR doing?

How should this be manually tested?

What are the relevant tickets?

codecov[bot] commented 1 year ago

Codecov Report

Merging #318 (bd57982) into main (111be34) will increase coverage by 18.0%. Report is 5 commits behind head on main. The diff coverage is 79.5%.

Additional details and impacted files | [Files](https://app.codecov.io/gh/opentok/opentok-node/pull/318?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opentok) | Coverage Δ | | |---|---|---| | [spec/opentok\_spec.js](https://app.codecov.io/gh/opentok/opentok-node/pull/318?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opentok#diff-c3BlYy9vcGVudG9rX3NwZWMuanM=) | `100.0% <100.0%> (ø)` | | | [lib/moderation.js](https://app.codecov.io/gh/opentok/opentok-node/pull/318?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opentok#diff-bGliL21vZGVyYXRpb24uanM=) | `93.1% <88.8%> (-2.6%)` | :arrow_down: | | [lib/signaling.js](https://app.codecov.io/gh/opentok/opentok-node/pull/318?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opentok#diff-bGliL3NpZ25hbGluZy5qcw==) | `88.8% <90.9%> (ø)` | | | [lib/opentok.js](https://app.codecov.io/gh/opentok/opentok-node/pull/318?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opentok#diff-bGliL29wZW50b2suanM=) | `76.4% <28.5%> (-0.4%)` | :arrow_down: | | [lib/client.js](https://app.codecov.io/gh/opentok/opentok-node/pull/318?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opentok#diff-bGliL2NsaWVudC5qcw==) | `93.6% <90.7%> (+14.7%)` | :arrow_up: | | [lib/render.js](https://app.codecov.io/gh/opentok/opentok-node/pull/318?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opentok#diff-bGliL3JlbmRlci5qcw==) | `19.6% <12.5%> (-2.0%)` | :arrow_down: | | [lib/callbacks.js](https://app.codecov.io/gh/opentok/opentok-node/pull/318?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opentok#diff-bGliL2NhbGxiYWNrcy5qcw==) | `74.6% <77.1%> (+1.6%)` | :arrow_up: | | [lib/archiving.js](https://app.codecov.io/gh/opentok/opentok-node/pull/318?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opentok#diff-bGliL2FyY2hpdmluZy5qcw==) | `75.4% <77.2%> (+66.8%)` | :arrow_up: | ... and [1 file with indirect coverage changes](https://app.codecov.io/gh/opentok/opentok-node/pull/318/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opentok)