[Pidgin] #247: MSN Direct File Transfers
Pidgin
trac at pidgin.im
Tue Dec 15 12:09:27 EST 2009
#247: MSN Direct File Transfers
-----------------------------+----------------------------------------------
Reporter: gbil | Owner: khc
Type: enhancement | Status: new
Milestone: Patches welcome | Component: MSN
Version: 2.0 | Resolution:
Keywords: |
-----------------------------+----------------------------------------------
Comment(by kukkerman):
Hi!
It turned out that the direct file transfer is not the only problem here.
I was able to negotiate a direct connection between the two parties and
the file data is sent through it just fine. However if there's an
estabilished direct connection between two parties then the official
clients send other invitations (emoticon, user display picture for
example) on this channel as well to lighten the load of MSN servers I
suppose. This doesn't mean that the transferred file will be corrupted
because these messages can easily be spearated from the file data. But...
they can't directly be fed back to the existing handlers because of the
slighty different format and the response generated by handlers will be
sent through the MSN servers by default rather than the direct connection.
So the direct connection has to be integrated into the slplink and its
corresponding slpcalls. I've come to this conclusion a bit late and
realized that some of my code is redundant because with the appropriate
modification the existing slplink/slpcall routines will be able to deal
with direct connections as well.
Okay that's enough talk... I've attached the patch which enables some
'''limited direct file transfer'''. The caveats are:
* You'll be able to receive/send one file at a time '''with a specific
party'''.
* Invite messages (emoticon, user display picture for example) won't be
received during the file transfer.
If I find some time I'll work on the proper direct file transfer so don't
panic.
Cheers,
Kukkerman
--
Ticket URL: <http://developer.pidgin.im/ticket/247#comment:48>
Pidgin <http://pidgin.im>
Pidgin
More information about the Tracker
mailing list