Review Request 108791: Handle Channel.Type.ServerTLSConnection

[Daniele E. Domenichelli]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/108791/#review26691
-----------------------------------------------------------

Ship it!


The patch looks good, but it causes the same issues of the KWallet stuff (see Bug 300062)
We should probably discuss the problem upstream on the telepathy mailing list.

Anyway it's a ship it from me, just add a comment to that bug and open a new one so that we don't remember to fix both these problems.


main.cpp
<http://git.reviewboard.kde.org/r/108791/#comment20270>

    You should change this to 2.
    Yes it's probably a stupid method to do it, feel free change it.


- Daniele E. Domenichelli


On Feb. 4, 2013, 11:17 p.m., Dan Vrátil wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/108791/
> -----------------------------------------------------------
> 
> (Updated Feb. 4, 2013, 11:17 p.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Description
> -------
> 
> Uses QCA2 to verify the certificate chain. CA certs are read from KSslCertificateManager. Exception rules are stored in KSslCertificateManager too. This means that we now share certificates and exceptions with the rest of KDE \o/. However in order to use KSslCertificateManager and KSslInfoDialog I had to import several private headers from kdelibs.
> 
> The dialogs are copied from KIO::TcpSlaveBase, so they look exactly like kio_http.
> 
> If you set a "remember forever" rule during testing and you want to remove it, remove it from ~/.kde/share/config/ksslcertificatemanager and restart KDED.
> 
> I had to remove -DQT_NO_CAST_FROM_ASCII and -DQT_NO_KEYWORDS because QCA2 and kdelibs headers wouldn't compile otherwise.
> 
> 
> This addresses bug 265315.
>     http://bugs.kde.org/show_bug.cgi?id=265315
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 417554a 
>   kssl/kssl.h PRE-CREATION 
>   kssl/ksslcertificatemanager.h PRE-CREATION 
>   kssl/ksslinfodialog.h PRE-CREATION 
>   kssl/ksslsettings.h PRE-CREATION 
>   main.cpp 3350658 
>   tls-cert-verifier-op.h 121cf41 
>   tls-cert-verifier-op.cpp c94999e 
> 
> Diff: http://git.reviewboard.kde.org/r/108791/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Dan Vrátil
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20130205/d2063163/attachment.html>