[Pidgin] #13050: Avatars not showing on Yahoo

Pidgin trac at pidgin.im
Fri Jan 13 16:04:15 EST 2012


#13050: Avatars not showing on Yahoo
---------------------+------------------------------------------------------
 Reporter:  FireDK   |        Owner:  sulabh.dev         
     Type:  defect   |       Status:  closed             
Milestone:  v2.10.1  |    Component:  Yahoo!/Yahoo! JAPAN
  Version:  2.9.0    |   Resolution:  fixed              
 Keywords:           |  
---------------------+------------------------------------------------------

Comment(by thijsalkemade at gmail.com):

 (In [78f01024866db9d9208f8cc09b8321c7ed04c140]):[[BR]]
 *** Plucked rev be5e66abad2af29604bc794cc4c6600ab12751f3
 (rekkanoryo at pidgin.im):
 Remove a bogus short-circuit check intended to prevent downloading the
 same icon
 every time a buddy logs in.  There are three problems with the check:
   * The fact that we already have an icon for the buddy doesn't mean it
 hasn't
     changed on the server since we last downloaded.
   * We should really be checking against the server-provided checksum, but
 we
     don't know how to checksum the same way the server does.
   * We no longer receive a checksum at the YMSG protocol level, so we'd
 have to
     parse such a checksum out of the picture URL, which is going to be
 fragile
     and seems just plain stupid to me.

 It seems better to me all around to just axe the check and potentially
 waste
 some bandwidth.  If someone wants to figure out a bandwidth-saving fix for
 this,
 feel free.

 At any rate, this fixes #13050.

-- 
Ticket URL: <http://developer.pidgin.im/ticket/13050#comment:18>
Pidgin <http://pidgin.im>
Pidgin


More information about the Tracker mailing list