From 32e798967e962738723f0f56c2b37619ce9bc44c Mon Sep 17 00:00:00 2001 From: Lance Stout Date: Fri, 20 Jul 2012 15:26:18 -0700 Subject: [PATCH 1/3] Fix see-other-host handling if no host is actually given. Also, limit number of consecutive redirection attempts. --- sleekxmpp/basexmpp.py | 20 ++++++++++++++++++++ 1 file changed, 20 insertions(+) diff --git a/sleekxmpp/basexmpp.py b/sleekxmpp/basexmpp.py index d81be016..90134c4d 100644 --- a/sleekxmpp/basexmpp.py +++ b/sleekxmpp/basexmpp.py @@ -72,6 +72,11 @@ class BaseXMPP(XMLStream): self.boundjid = JID(jid) self._expected_server_name = self.boundjid.host + self._redirect_attempts = 0 + + #: The maximum number of consecutive see-other-host + #: redirections that will be followed before quitting. + self.max_redirects = 5 self.session_bind_event = threading.Event() @@ -144,6 +149,8 @@ class BaseXMPP(XMLStream): MatchXPath("{%s}error" % self.stream_ns), self._handle_stream_error)) + self.add_event_handler('session_start', + self._handle_session_start) self.add_event_handler('disconnected', self._handle_disconnected) self.add_event_handler('presence_available', @@ -656,6 +663,10 @@ class BaseXMPP(XMLStream): def getjidbare(self, fulljid): return fulljid.split('/', 1)[0] + def _handle_session_start(self, event): + """Reset redirection attempt count.""" + self._redirect_attempts = 0 + def _handle_disconnected(self, event): """When disconnected, reset the roster""" self.roster.reset() @@ -666,6 +677,15 @@ class BaseXMPP(XMLStream): if error['condition'] == 'see-other-host': other_host = error['see_other_host'] + if not other_host: + log.warning("No other host specified.") + return + + if self._redirect_attempts > self.max_redirects: + log.error("Exceeded maximum number of redirection attempts.") + return + + self._redirect_attempts += 1 host = other_host port = 5222 From 81d3723084b55b95c660776159fd054f1844f5ac Mon Sep 17 00:00:00 2001 From: Lance Stout Date: Fri, 20 Jul 2012 18:07:27 -0700 Subject: [PATCH 2/3] Add event for vCard avatar update. --- sleekxmpp/plugins/xep_0153/vcard_avatar.py | 1 + 1 file changed, 1 insertion(+) diff --git a/sleekxmpp/plugins/xep_0153/vcard_avatar.py b/sleekxmpp/plugins/xep_0153/vcard_avatar.py index 1e32595a..6b70e33e 100644 --- a/sleekxmpp/plugins/xep_0153/vcard_avatar.py +++ b/sleekxmpp/plugins/xep_0153/vcard_avatar.py @@ -106,6 +106,7 @@ class XEP_0153(BasePlugin): return elif data == '' or data != self.api['get_hash'](pres['to']): self._reset_hash(pres['from']) + self.xmpp.event('vcard_avatar_update', pres) # ================================================================= From 474390fa00a4c2ddf63976bdaf95468248ddd12b Mon Sep 17 00:00:00 2001 From: Lance Stout Date: Fri, 20 Jul 2012 18:10:14 -0700 Subject: [PATCH 3/3] Add example for retrieving avatars. --- examples/download_avatars.py | 184 +++++++++++++++++++++++++++++++++++ 1 file changed, 184 insertions(+) create mode 100644 examples/download_avatars.py diff --git a/examples/download_avatars.py b/examples/download_avatars.py new file mode 100644 index 00000000..b621c940 --- /dev/null +++ b/examples/download_avatars.py @@ -0,0 +1,184 @@ +#!/usr/bin/env python +# -*- coding: utf-8 -*- + +""" + SleekXMPP: The Sleek XMPP Library + Copyright (C) 2012 Nathanael C. Fritz + This file is part of SleekXMPP. + + See the file LICENSE for copying permission. +""" + +import sys +import logging +import getpass +import threading +from optparse import OptionParser + +import sleekxmpp +from sleekxmpp.exceptions import XMPPError + + +# Python versions before 3.0 do not use UTF-8 encoding +# by default. To ensure that Unicode is handled properly +# throughout SleekXMPP, we will set the default encoding +# ourselves to UTF-8. +if sys.version_info < (3, 0): + reload(sys) + sys.setdefaultencoding('utf8') +else: + raw_input = input + + +FILE_TYPES = { + 'image/png': 'png', + 'image/gif': 'gif', + 'image/jpeg': 'jpg' +} + + +class AvatarDownloader(sleekxmpp.ClientXMPP): + + """ + A basic script for downloading the avatars for a user's contacts. + """ + + def __init__(self, jid, password): + sleekxmpp.ClientXMPP.__init__(self, jid, password) + self.add_event_handler("session_start", self.start, threaded=True) + self.add_event_handler("changed_status", self.wait_for_presences) + + self.add_event_handler('vcard_avatar_update', self.on_vcard_avatar) + self.add_event_handler('avatar_metadata_publish', self.on_avatar) + + self.received = set() + self.presences_received = threading.Event() + + def start(self, event): + """ + Process the session_start event. + + Typical actions for the session_start event are + requesting the roster and broadcasting an initial + presence stanza. + + Arguments: + event -- An empty dictionary. The session_start + event does not provide any additional + data. + """ + self.send_presence() + self.get_roster() + + print('Waiting for presence updates...\n') + self.presences_received.wait(15) + self.disconnect(wait=True) + + def on_vcard_avatar(self, pres): + print("Received vCard avatar update from %s" % pres['from'].bare) + try: + result = self['xep_0054'].get_vcard(pres['from'], cached=True) + except XMPPError: + print("Error retrieving avatar for %s" % pres['from']) + return + avatar = result['vcard_temp']['PHOTO'] + + filetype = FILE_TYPES.get(avatar['TYPE'], 'png') + filename = 'vcard_avatar_%s_%s.%s' % ( + pres['from'].bare, + pres['vcard_temp_update']['photo'], + filetype) + with open(filename, 'w+') as img: + img.write(avatar['BINVAL']) + + def on_avatar(self, msg): + print("Received avatar update from %s" % msg['from']) + metadata = msg['pubsub_event']['items']['item']['avatar_metadata'] + for info in metadata['items']: + if not info['url']: + try: + result = self['xep_0084'].retrieve_avatar(msg['from'], info['id']) + except XMPPError: + print("Error retrieving avatar for %s" % msg['from']) + return + + avatar = result['pubsub']['items']['item']['avatar_data'] + + filetype = FILE_TYPES.get(metadata['type'], 'png') + filename = 'avatar_%s_%s.%s' % (msg['from'].bare, info['id'], filetype) + with open(filename, 'w+') as img: + img.write(avatar['value']) + else: + # We could retrieve the avatar via HTTP, etc here instead. + pass + + def wait_for_presences(self, pres): + """ + Wait to receive updates from all roster contacts. + """ + self.received.add(pres['from'].bare) + if len(self.received) >= len(self.client_roster.keys()): + self.presences_received.set() + else: + self.presences_received.clear() + + +if __name__ == '__main__': + # Setup the command line arguments. + optp = OptionParser() + optp.add_option('-q','--quiet', help='set logging to ERROR', + action='store_const', + dest='loglevel', + const=logging.ERROR, + default=logging.ERROR) + optp.add_option('-d','--debug', help='set logging to DEBUG', + action='store_const', + dest='loglevel', + const=logging.DEBUG, + default=logging.ERROR) + optp.add_option('-v','--verbose', help='set logging to COMM', + action='store_const', + dest='loglevel', + const=5, + default=logging.ERROR) + + # JID and password options. + optp.add_option("-j", "--jid", dest="jid", + help="JID to use") + optp.add_option("-p", "--password", dest="password", + help="password to use") + opts,args = optp.parse_args() + + # Setup logging. + logging.basicConfig(level=opts.loglevel, + format='%(levelname)-8s %(message)s') + + if opts.jid is None: + opts.jid = raw_input("Username: ") + if opts.password is None: + opts.password = getpass.getpass("Password: ") + + xmpp = AvatarDownloader(opts.jid, opts.password) + xmpp.register_plugin('xep_0054') + xmpp.register_plugin('xep_0153') + xmpp.register_plugin('xep_0084') + + # If you are working with an OpenFire server, you may need + # to adjust the SSL version used: + # xmpp.ssl_version = ssl.PROTOCOL_SSLv3 + + # If you want to verify the SSL certificates offered by a server: + # xmpp.ca_certs = "path/to/ca/cert" + + # Connect to the XMPP server and start processing XMPP stanzas. + if xmpp.connect(): + # If you do not have the dnspython library installed, you will need + # to manually specify the name of the server if it does not match + # the one in the JID. For example, to use Google Talk you would + # need to use: + # + # if xmpp.connect(('talk.google.com', 5222)): + # ... + xmpp.process(block=True) + else: + print("Unable to connect.")