Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 207677

Summary: [msn] NotificationSession throws StringIndexOutOfBoundsException on login MSG
Product: [RT] ECF Reporter: tonys <tony.seebregts>
Component: ecf.protocolsAssignee: Remy Suen <remy.suen>
Status: RESOLVED WONTFIX QA Contact:
Severity: major    
Priority: P3 CC: remy.suen, slewis
Version: 1.2.0Keywords: helpwanted
Target Milestone: ---   
Hardware: All   
OS: All   
Whiteboard:

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.