byronwasti / movine

A migration manager written in Rust that attempts to be smart yet minimal
MIT License
109 stars 9 forks source link

TLS certificate options #23

Open couchand opened 1 year ago

couchand commented 1 year ago

I've only just discovered this tool but it looks really neat!

One system I'm working with uses client certs for PG authentication, so I was excited to see your README point out that the software supports the PGSSLCERT environment variable. Diving into the implementation, it looks like it uses that file for the root CA cert, rather than the client cert. So, I'd suggest two changes:

The first one should be a pretty quick change but the latter one is perhaps a bit more effort. I'd be happy to work up a PR if you'd be open to adding this support?

byronwasti commented 1 year ago

Thanks for the interest! I haven't really had time to work on this, and likely won't dedicate too much time to it. The library/binary is a bit rough around the edges as a fair warning. I will take a look at your PRs, thanks for making them

byronwasti commented 1 year ago

Hey sorry for the super late response, I'm likely not going to have time to dedicate to this project for the foreseeable future. I'll merge the PRs and clean up the CI stuff at this point though