vertica / vertica-nodejs

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

temporary removal of mTLS + ssl related test fixes #4

Closed DMickens closed 2 years ago

DMickens commented 2 years ago

This PR is for the desired state of TLS for this release. I have been unable to verify mTLS success so the connection parameters associated with mTLS have been commented out. You will also see that we have moved from a pfx (key & certificate) file for client verification to individual client key and certificate files as this is a more common approach. The property names have been changed to reflect that and be more clear. This also has fixed 20 or so of the failing tests stemming from using the old ssl function. The removal of some tests is due to the fact that we are supporting 4 explicit modes of tls, and not modes such as 'prefer'. This mimics our JDBC implementation