Bug 207677 - [msn] NotificationSession throws StringIndexOutOfBoundsException on login MSG
Summary: [msn] NotificationSession throws StringIndexOutOfBoundsException on login MSG
Status: RESOLVED WONTFIX
Alias: None
Product: ECF
Classification: RT
Component: ecf.protocols (show other bugs)
Version: 1.2.0   Edit
Hardware: All All
: P3 major (vote)
Target Milestone: ---   Edit
Assignee: Remy Suen CLA
QA Contact:
URL:
Whiteboard:
Keywords: helpwanted
Depends on:
Blocks:
 
Reported: 2007-10-27 14:13 EDT by tonys CLA
Modified: 2014-02-12 14:11 EST (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description tonys CLA 2007-10-27 14:13:09 EDT
Build ID: M20070921-1145
ECF     : 1.20v20071019-1300

Split out from bug 207670 and refiled at Remy's request.

Steps To Reproduce:

The bug occurs about 25% of the time when logging in to MSN and depends on the MSG sent by the MSN switchboard when logging in. The easiest way to repeat it is to split out NotificationSession.read() event parsing code into a seperate method and get the method to parse the following input string:

MSG Hotmail Hotmail 289
Content-Type:text/x-msmsgsinitialmdatanotification;charset=UTF-8 
                                          
Mail-Data:<MD><E><I>0</I><IU>0</IU><O>0</O><OU>0</OU></E><Q><QTM>409600</QTM><QNM>204800</QNM></Q></MD
Inbox-URL: /cgi-bin/HoTMaiL                                          
Folders-URL: /cgi-bin/folders                                          
Post-URL: http://www.hotmail.com
CHG 6 NLN 268435488

More information:

The error occurs because the event parser in
NotificationSession.read() does not recognise incoming MSG packets and picks up the NLN on the last line of the login response from the MSN server.

The problem is more general in that the current implementation of the event parser in NotificationSession.read() will trigger on any embedded string that look like MSN commands in a MSG body (or in any otherwise unprocessed MSN command).
Comment 1 Scott Lewis CLA 2008-05-23 14:55:14 EDT
Is this fixed or irrelevant at this point?
Comment 2 Remy Suen CLA 2008-05-23 17:23:18 EDT
As far as I can tell, this is still relevant.
Comment 3 Scott Lewis CLA 2008-05-23 17:35:49 EDT
changing target to 2.0.0 release
Comment 4 Scott Lewis CLA 2011-08-09 00:54:58 EDT
Remy...as the keeper of the ECF msn provider...any plans to address this bug?
Comment 5 Scott Lewis CLA 2014-02-12 14:11:48 EST
Seems as though the msn provider is no longer supported.  If this is wrong or if new resources are made available, then please reopen.