search

byronwasti / movine

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

TLS support #8

Closed ahanoff closed 4 years ago

ahanoff commented 4 years ago

As I can see postgres::NoTls hard-coded in postgres adaptor, it seems like TLS is not yet supported. Any plans to add?

byronwasti commented 4 years ago

Hey, thanks for reaching out. I set up a PR to fix this issue. I haven't gone through and tested things with a generated cert, but given how straightforward the implementation is I'm happy to merge as-is if it looks good to you.

ahanoff commented 4 years ago

@byronwasti thanks, that's pretty fast. Let me test it on my db tonight, and I will let you know the outcome.

byronwasti commented 4 years ago

TLS support has been merged and should be present in version 0.7.0. Let me know if you have any issues with that, and thanks for reaching out!