bugfix(microservices): let use unicode characters in TCP Microservice…

[ivibe]

… messages

add own json-socket implementation

#1734

PR Checklist

Please check if your PR fulfills the following requirements:

• The commit message follows our guidelines: https://github.com/nestjs/nest/blob/master/CONTRIBUTING.md
• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

[x] Bugfix

What is the current behavior?

Issue Number: #1734

What is the new behavior?

Unicode characters in TCP Microservice messages don't cause errors

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

The core algorithm was borrowed from original json-socket package (https://github.com/sebastianseilund/node-json-socket). Basically all unnecessary code was removed, only the core functionality was kept.

[coveralls]

Pull Request Test Coverage Report for Build 2034

• 84 of 92 (91.3%) changed or added relevant lines in 6 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage decreased (-0.08%) to 93.626%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
packages/microservices/json-socket.ts	50	53	94.34%
packages/microservices/test/json-socket/helpers.ts	28	33	84.85%

Totals
Change from base Build 2033:	-0.08%
Covered Lines:	3078
Relevant Lines:	3231

---

💛 - Coveralls

[coveralls]

Pull Request Test Coverage Report for Build 2034

• 84 of 92 (91.3%) changed or added relevant lines in 6 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage decreased (-0.08%) to 93.626%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
packages/microservices/json-socket.ts	50	53	94.34%
packages/microservices/test/json-socket/helpers.ts	28	33	84.85%

Totals
Change from base Build 2033:	-0.08%
Covered Lines:	3078
Relevant Lines:	3231

---

💛 - Coveralls

[kamilmysliwiec]

Thank you :) LGTM

[ivibe]

Glad to help!

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.