NagiosEnterprises / ndoutils

NDOUtils - Database Output for Nagios Core
GNU General Public License v2.0
48 stars 19 forks source link

add feature to optionally use encrypted connection to db #53

Closed savec80 closed 4 years ago

savec80 commented 5 years ago

Hello team, Can option for using encrypted connection to database be added to the module? Please have a look at the patch in attach, where I implemented it. Also there are added some changes from this commit: 34bf52b2a290a03221f53a30a4c721acedddc710 in order option --enable-ssl for configure works. Thanks. db_ssl.txt

jomann09 commented 4 years ago

Closing this because the branch in question being merged is no longer relevant and will be removed shortly. I added this as an issue #59 and we will take a look at the changes you made along with whatever is required to make sure --enable-ssl works (and is available) in ndo 3.

sawolf commented 4 years ago

@savec80, what distribution were you using to create that patch? I wasn't able to compile it using CentOS 7/mariadb-5.5.64.

savec80 commented 4 years ago

Hello Sebastian, I used RHEL7. However according to https://dev.mysql.com/doc/refman/5.6/en/mysql-options.html mysql version 5.6 at least is required for db data encryption, so I used: mysql-community-devel-8.0.13-1 in order to compile it. Not sure if it's a blocker or not. Hope that helps. Cheers.

On Tue, 17 Dec 2019 at 22:43, Sebastian Wolf notifications@github.com wrote:

@savec80 https://github.com/savec80, what distribution were you using to create that patch? I wasn't able to compile it using CentOS 7/mariadb-5.5.64.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/NagiosEnterprises/ndoutils/pull/53?email_source=notifications&email_token=AFIZDZIFYKOA7L6WXQARWRTQZFBW5A5CNFSM4GH2MP5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHEBVCQ#issuecomment-566762122, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFIZDZLD2YTNTKLRFF6ARIDQZFBW5ANCNFSM4GH2MP5A .