search

theforeman / foreman_ansible_inventory

Foreman dynamic inventory script for ansible - Now merged into Ansible itself
GNU General Public License v3.0
70 stars 31 forks source link

Suppress warning when SSL cert verification is off #18

Closed arielsalvo closed 8 years ago

arielsalvo commented 8 years ago

How about suppressing all these warnings? I'm using self-signed certs so I get a lot of nagging. I think it can be expected that turning off SSL certificate verification will make the request not be very secure.

agx commented 8 years ago

On Tue, Jul 26, 2016 at 02:11:24AM -0700, Ariel Salvo wrote:

How about suppressing all these warnings? I'm using self-signed certs so I get a lot of nagging. I think it can be expected that turning off SSL certificate verification will make the request not be very secure.

I'm petty much in favour of leaving the warning in place. Self signed certs can be used without warnings too.

Cheers, -- Guido

arielsalvo commented 8 years ago

Eventually, I'll set up a proper certificate so it is not a big issue. I just find it inconvenient that the warning comes up so often when I already accepted not to do cert verification. How about printing just a single warning to stderr in that same if clause?

dLobatog commented 8 years ago

I think it's alright to show the warning too - the warning can go away if you have a CA in the system that trusts the cert. https://gist.github.com/kyledrake/d7457a46a03d7408da31 for some help on how to do that

arielsalvo commented 8 years ago

Sounds good, I'm going to close this one. Thanks!