FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Fix handshake with Node.js not completing when external systems interfere with Node.js's stdout stream by aKzenT · Pull Request #146 · JeringTech/Javascript.NodeJS · GitHub

Fix handshake with Node.js not completing when external systems interfere with Node.js's stdout stream - #146

Merged
JeremyTCD merged 5 commits into
JeringTech:masterfrom
aKzenT:fix145
Aug 25, 2022
Merged

Fix handshake with Node.js not completing when external systems interfere with Node.js's stdout stream#146
JeremyTCD merged 5 commits into
JeringTech:masterfrom
aKzenT:fix145

Conversation

aKzenT commented Aug 24, 2022

Copy link
Copy Markdown
Collaborator

Did not have time to test it yet, but let me know if I'm on the right track

aKzenT and others added 2 commits August 24, 2022 18:48
- Fixed regex pattern.
- Updated tests.
- Minor re-arranging of class members.

Copy link
Copy Markdown
Member

It looks great! I've cleaned up the tests and added you to the ReadMe contributor list.

I need to update the Azure pipeline. Will publish a new package once that's done, probably in a couple of hours.

JeremyTCD linked an issue Aug 25, 2022 that may be closed by this pull request
JeremyTCD changed the title Fixes #145 Fix handshake with Node.js not completing when external systems interfere with Node.js's stdout stream Aug 25, 2022
JeremyTCD merged commit f4e90c3 into JeringTech:master Aug 25, 2022

Copy link
Copy Markdown
Member

New package is live: 7.0.0-beta.0.

Bumped to 7.0.0 because the public API changed. Thanks for the contribution, really appreciate it!

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Node startup fails on Azure App Service

2 participants


Back | FazBrowse Home | New Git URL