[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