couchbaselabs / node-ottoman

Node.js ODM for Couchbase
https://ottomanjs.com/
Apache License 2.0
287 stars 98 forks source link

Ottoman Testing with SDK 4.0 #648

Closed AV25242 closed 1 year ago

AV25242 commented 2 years ago

The plan is to get Ottoman over Node.js SDK 4.1 which will release this month, as a step towards getting Ottoman updated it will be good to start testing Ottoman with SDK 4.0, a release with SDK 4.0 is not recommended at this time, this is just a proactive measure to get us to SDK 4.1 in May.

ejscribner commented 2 years ago

Tested it out with 4.0; everything worked well with a few minor changes:

/src/ottoman/ottoman.ts:

/src/couchbase.ts:

Additionally, the connection string parameter to ignore TLS verification was renamed from ssl=no_verify to tls_verify=none in v4.0 (I believe this is due to the change to couchbase++). Connections to TLS-enabled clusters (like Capella) will either need the root certificate passed in with the connection or ?tls_verify=none at the end of the connection string.

I wrote a quick tutorial with a bit more info if needed: https://developer.couchbase.com/tutorial-nodejs-tls-connection

ejscribner commented 2 years ago

An update after running the test suite against Ottoman built with Node.js SDK 4.0:

Local/On Prem cluster (Docker)

Capella