medic / couchdb-migration

0 stars 0 forks source link

Get couchUrl dinamically to support multiple ports. #9

Closed dianabarsan closed 1 year ago

dianabarsan commented 1 year ago

Small updates to readme. Re-adds ands respects COUCH_CLUSTER_PORT.

dianabarsan commented 1 year ago

I pushed an additional change here.

Unfortunately the clustering port is not exposed when running against a couchdb running inside the medic-os container. So initializing URLs failed. I changed it so it will only build the clustering API url when specifically needed (it's not needed when running against 3.x couchdb). Additionally, I added NODE_TLS_REJECT_UNAUTHORIZED env variable, in case medic-os uses a self-signed cert.

Please let me know if you're still in agreement. If yes, I'll merge this, bump the version, publish a new image, and update the docker-compose file as well.

Thanks!