communi / libcommuni

A cross-platform IRC framework for Qt
https://communi.github.io
BSD 3-Clause "New" or "Revised" License
83 stars 37 forks source link

Verify ssl peer certificates #51

Open ManiacTwister opened 8 years ago

ManiacTwister commented 8 years ago

Hello,

Currently the ssl certificate aren't verified. In my opinion you should change this behaviour because only if the peer certificate is verified the connection can be secure. As a first step, the sslsockets verifymode should be changed to QSslSocket::VerifyPeer.

Ideally there should also be a way to specify ca certificates and fingerprints on a per server basis.