vertica / vertica-nodejs

Official native node.js client for the Vertica Analytics Database.
https://www.vertica.com/
Apache License 2.0
12 stars 14 forks source link

Fix for #89 #103

Closed pietermees closed 1 year ago

pietermees commented 1 year ago

Fix for "Invalid startup packet layout: expected terminator as last byte"

CLAassistant commented 1 year ago

CLA assistant check
All committers have signed the CLA.

sitingren commented 1 year ago

@DMickens CI test regression is introduced from this PR.

DMickens commented 1 year ago

@sitingren Reverting and investigating.

Not sure why CI tests weren't ran automatically and caught in the PR, will look into that as well

pietermees commented 1 year ago

This change worked for me in my use case and was recommended on the original issue #89 I'm not sure why the test suite would not have run though.