[Pidgin] #7034: Insufficient info for "SSL Certificate Verification"
Pidgin
trac at pidgin.im
Tue Jul 8 02:53:05 EDT 2014
#7034: Insufficient info for "SSL Certificate Verification"
--------------------------------------+-------------------------
Reporter: kiilerix | Owner: darkrain42
Type: defect | Status: new
Milestone: | Component: libpurple
Version: 2.5.1 | Resolution:
Keywords: certificate verification |
--------------------------------------+-------------------------
Comment (by Mark Doliner <mark@…>):
(In [89bda8c0750d]):[[BR]]
Copy changes from datallah's 2948449ffd12 from the
release-2.x.y branch to default.
I dropped these changes when I merged that branch into default because
the code had moved and it didn't seem appropriate to re-do the change
in the merge commit.
The only remaining change from that commit is to add a "View Issuer
Certificate" button to the accept dialog. I think it's a good idea to
give users a way to view info about the cert chain, but I'm not going
to spend the time to figure out how to do it in default. If anyone else
wants to, feel free.
If anyone is keeping a list of starter tasks for aspiring patch writers,
this is a great one.
Also, why the hell is the x509_display_string() function in the
library-specific code? The function is duplicated and almost 100%
identical
in ssl-gnutls.c and ssl-nss.c. Ugh.
Refs #7034
https://developer.pidgin.im/ticket/7034
--
Ticket URL: <https://developer.pidgin.im/ticket/7034#comment:12>
Pidgin <https://pidgin.im>
Pidgin
More information about the Tracker
mailing list