On Sun, Jan 01, 2006 at 02:42:34PM +0200, Norman Rasmussen wrote:
> Some debugging gems:
> 
> 2006/01/01 14:33 SAST [XmlStream,client] Traceback (most recent call last):
[snip]
>      File "/home/norman/src/svn/PyMSNt.trunk/src/misciq.py", line 132,
> in incomingIq
>        froj = jid.intern(fro)
>    exceptions.AttributeError: 'module' object has no attribute 'intern'
> 
> and I also get quite a few of these:
> 
> 2006/01/01 14:33 SAST [XmlStream,client] WARN ::  :: Dropping IQ
> because of stringprep error :: onIq :: <disco.ServerDiscovery instance
> at 0xb76fc44c> :: {'el': <twisted.xish.domish.Element object at
> 0xb76d4b6c>, 'e': <exceptions.AttributeError instance at 0xb76d49ec>,
> 'fro': u'darkskies.za.net', 'self': <disco.ServerDiscovery instance at
> 0xb76fc44c>, 'iqType': u'get', 'ulang': None, 'to':
> u'msn.jabber.darkskies.za.net', 'ID': u'mydhc4mg'}

I think the exception and the warning are due to the same issue. It
seems that newer versions of Twisted Words has had the intern() function
in twisted.words.protocols.jabber.jid renamed to internJID().

To fix this issue, run the following command:
find src/ -iname '*.py' -exec sed -i 's/jid.intern/jid.internJID/' {} \;

Best regards,
Lars
From [EMAIL PROTECTED]  Sun Jan  1 20:22:50 2006
From: [EMAIL PROTECTED] (Stian B. Barmen)
Date: Sun Jan  1 20:22:53 2006
Subject: [py-transports] Preliminary file transfer!
In-Reply-To: <[EMAIL PROTECTED]>
References: <[EMAIL PROTECTED]>
        <[EMAIL PROTECTED]>
        <[EMAIL PROTECTED]>
        <[EMAIL PROTECTED]>
Message-ID: <[EMAIL PROTECTED]>

Skipped content of type multipart/alternative-------------- next part 
--------------
A non-text attachment was scrubbed...
Name: smiley-3.png
Type: image/png
Size: 819 bytes
Desc: not available
Url : 
http://modevia.com/pipermail/py-transports/attachments/20060101/3c56b0b1/smiley-3.png
From [EMAIL PROTECTED]  Sun Jan  1 20:23:21 2006
From: [EMAIL PROTECTED] (Lars T. Mikkelsen)
Date: Sun Jan  1 20:23:27 2006
Subject: [py-transports] Preliminary file transfer!
In-Reply-To: <[EMAIL PROTECTED]>
References: <[EMAIL PROTECTED]>
Message-ID: <[EMAIL PROTECTED]>

On Sun, Jan 01, 2006 at 11:08:00PM +1100, James Bunton wrote:
> Could people please help me test things. I'm looking for regressions in 
> particular. A lot of code has been completely rewritten. So things like 
> groupchat, clientcaps, avatars and even ordinary messaging still need a 
> bit of edge-case testing.

So far I found two minor issues with avatars:

1) The 0.11 branch allowed one to configure the getAllAvatars behaviour in
config.xml. In trunk getAllAvatars is present in src/config.py and
config-example.xml, but config.getAllAvatars is not actually used. The attached
patch replaces GETALLAVATARS with config.getAllAvatars.

2) The avatar hash is not always properly base64-decoded, and thus the
AvatarCache fails to fetch avatars:

2006/01/01 20:47 CET [NotificationClient,client] INFO ::  ::  :: 
contactAvatarChanged :: <tlib.msn.msnw.NotificationClient instance at 
0xb75b1a6c> :: {'self': <tlib.msn.msnw.NotificationClient instance at 
0xb75b1a6c>, 'hash': '2st/sKrrNgIW+/sHsFeE3FT0QQ8=', 'userHandle': '<contact's 
passport>'}
2006/01/01 20:47 CET [NotificationClient,client] INFO :: <my JID> ::  :: 
contactAvatarChanged :: <legacy.glue.LegacyConnection instance at 0xb77b8c8c> 
:: {'self': <legacy.glue.LegacyConnection instance at 0xb77b8c8c>, 'hash': 
'2st/sKrrNgIW+/sHsFeE3FT0QQ8=', 'userHandle': '<contact's passport>'}
2006/01/01 20:47 CET [NotificationClient,client] INFO ::  :: Avatar not found. 
:: getAvatarData :: <avatar.AvatarCache instance at 0xb7600e0c> :: {'self': 
<avatar.AvatarCache instance at 0xb7600e0c>, 'key': 
'2st/sKrrNgIW+/sHsFeE3FT0QQ8=', 'filename': 
'[...]/avatars/2st/2st/sKrrNgIW+/sHsFeE3FT0QQ8='}
2006/01/01 20:47 CET [NotificationClient,client] INFO :: <my JID> ::  :: 
sendAvatarRequest :: <legacy.glue.LegacyConnection instance at 0xb77b8c8c> :: 
{'self': <legacy.glue.LegacyConnection instance at 0xb77b8c8c>, 'userHandle': 
'<contact's passport>'}

Best regards,
Lars
-------------- next part --------------
Index: src/tlib/msn/msnw.py
===================================================================
--- src/tlib/msn/msnw.py        (revision 77)
+++ src/tlib/msn/msnw.py        (working copy)
@@ -12,11 +12,11 @@
 # Local imports
 from debug import LogEvent, INFO, WARN, ERROR
 from tlib.msn import msn
+import config
 
 
 MAXMESSAGESIZE     = 1400
 SWITCHBOARDTIMEOUT = 30.0*60.0
-GETALLAVATARS      = False
 
 
 """
@@ -126,7 +126,7 @@
 
                LogEvent(INFO, self.ident)
                if not self.notificationClient: return
-               if GETALLAVATARS:
+               if config.getAllAvatars:
                        self._ensureSwitchboardSession(userHandle)
                sb = self.switchboardSessions.get(userHandle)
                if sb: return sb.sendAvatarRequest()
From [EMAIL PROTECTED]  Sun Jan  1 20:41:00 2006
From: [EMAIL PROTECTED] (Stian B. Barmen)
Date: Sun Jan  1 20:41:00 2006
Subject: [py-transports] Preliminary file transfer!
In-Reply-To: <[EMAIL PROTECTED]>
References: <[EMAIL PROTECTED]>
        <[EMAIL PROTECTED]>
        <[EMAIL PROTECTED]>
        <[EMAIL PROTECTED]>
        <[EMAIL PROTECTED]>
Message-ID: <[EMAIL PROTECTED]>

Skipped content of type multipart/alternative-------------- next part 
--------------
A non-text attachment was scrubbed...
Name: smiley-3.png
Type: image/png
Size: 819 bytes
Desc: not available
Url : 
http://modevia.com/pipermail/py-transports/attachments/20060101/2f2d867e/smiley-3.png
From [EMAIL PROTECTED]  Sun Jan  1 22:09:34 2006
From: [EMAIL PROTECTED] (Lars T. Mikkelsen)
Date: Sun Jan  1 22:09:45 2006
Subject: [py-transports] Preliminary file transfer!
In-Reply-To: <[EMAIL PROTECTED]>
References: <[EMAIL PROTECTED]>
Message-ID: <[EMAIL PROTECTED]>

On Sun, Jan 01, 2006 at 11:08:00PM +1100, James Bunton wrote:
> Could people please help me test things. I'm looking for regressions in 
> particular. A lot of code has been completely rewritten. So things like 
> groupchat, clientcaps, avatars and even ordinary messaging still need a 
> bit of edge-case testing.

If I'm logged in with one resource (iChat) and log in with a second
resource (Psi) I get the following traceback:

Traceback (most recent call last):
  File "[...]/src/tlib/msn/msn.py", line 1117, in handle_CHG
    if not self._fireCallback(id, params[1]):
  File "[...]/src/tlib/msn/msn.py", line 752, in _fireCallback
    self.ids[id][0].callback(args)
  File "/usr/lib/python2.4/site-packages/twisted/internet/defer.py", line 254, 
in callback
    self._startRunCallbacks(result)
  File "/usr/lib/python2.4/site-packages/twisted/internet/defer.py", line 315, 
in _startRunCallbacks
    self._runCallbacks()
--- <exception caught here> ---
  File "/usr/lib/python2.4/site-packages/twisted/internet/defer.py", line 328, 
in _runCallbacks
    self.result = callback(self.result, *args, **kw)
  File "[...]/src/tlib/msn/msnw.py", line 192, in cb
    self.changeCount += 1
exceptions.AttributeError: LegacyConnection instance has no attribute 
'changeCount'

Similar tracebacks starting in handle_UUX and handle_PRP also occur. I
think this is due to a race condition in MSNConnection.changeStatus(),
if presence packets are received simultaneously from multiple resources.

Best regards,
Lars
From [EMAIL PROTECTED]  Mon Jan  2 07:34:44 2006
From: [EMAIL PROTECTED] (James Bunton)
Date: Mon Jan  2 07:34:54 2006
Subject: [py-transports] Preliminary file transfer!
In-Reply-To: <[EMAIL PROTECTED]>
References: <[EMAIL PROTECTED]>
        <[EMAIL PROTECTED]>
Message-ID: <[EMAIL PROTECTED]>

The current trunk should have less debugging information.

I've turned off some of the MSN protocol debugging information for now.

---

James


On 01/01/2006, at 11:14 PM, Norman Rasmussen wrote:

> Have you figured out how to reduce the amount that is written to the
> log files?
>
> At the moment I have a rm /var/log/pymsnt.log.?? in a daily cron job
> to try and keep down the amount of data that is stored - before this
> the logs grew to several hundred megs before I noticed!
>
> On 1/1/06, James Bunton <[EMAIL PROTECTED]> wrote:
>> Hi all.
>>
>> Just thought I'd let you know. svn://delx.cjb.net/pymsnt/trunk has
>> preliminary file transfer support.
>>
>> You should be able to receive files with jabber:x:oob and
>> jabber:iq:oob. That means, at worst you will get a clickable link to
>> download the file.
>> Sending is not yet supported. socks5 bytestreams are not yet 
>> supported.
>>
>> They will be coming, in that order :)
>>
>> Could people please help me test things. I'm looking for regressions 
>> in
>> particular. A lot of code has been completely rewritten. So things 
>> like
>> groupchat, clientcaps, avatars and even ordinary messaging still need 
>> a
>> bit of edge-case testing.
>>
>> It all seems to work for me right now, but I wouldn't recommend this
>> for a production server.
>>
>> Also, could I have some feedback on how people are going with the
>> installation of PyMSNt? Any suggestions on how to improve things would
>> be greatly appreciated.
>>
>> Finally. I'm away from home now, so I'm only connected by dialup, and
>> only occasionally at that. So please forgive any long response time.
>>
>> svn co svn://delx.cjb.net/pymsnt/trunk
>>
>> Enjoy! :)
>>
>> ---
>>
>> James
>>
>> _______________________________________________
>> py-transports mailing list
>> py-transports@blathersource.org
>> http://www.modevia.com/cgi-bin/mailman/listinfo/py-transports
>>
>
>
> --
> - Norman Rasmussen
>  - Email: [EMAIL PROTECTED]
>  - Home page: http://norman.rasmussen.co.za/
> _______________________________________________
> py-transports mailing list
> py-transports@blathersource.org
> http://www.modevia.com/cgi-bin/mailman/listinfo/py-transports
>

Reply via email to