[jira] Commented: (AXISCPP-609) Multi chunked http responses miss some bytes randomly

["Fred Preston (JIRA)" <ax...@ws.apache.org>]

    [ http://issues.apache.org/jira/browse/AXISCPP-609?page=comments#action_12315484 ] 

Fred Preston commented on AXISCPP-609:
--------------------------------------

Hi Bernd,
I've had a look at your changes and was trying to get the heart of what you have changed.  I supposed the first question is, does it solve your 'missing byte' problems?  The next one asumes that the answer to the first question is 'yes' is, what exactly is the difference that makes it work?  All I can see is that in the following code segment

// make sure we have read at least some part of the message
if( endOfChunkData == std::string::npos)
{
  iIterationCountdown = 100;

  do
  {
    m_pszRxBuffer [0] = '\0';
    *m_pActiveChannel >> m_pszRxBuffer;

    if( strlen( m_pszRxBuffer) == 0)
    {
      iIterationCountdown--;
    }
    else
    {
      iIterationCountdown = 100;
    }

    m_strReceived = m_pszRxBuffer;
    endOfChunkData = m_strReceived.find( ASCII_S_CRLF );
  } while( endOfChunkData == std::string::npos && iIterationCountdown > 0);
					}
You have changed 'm_strReceived = m_pszRxBuffer;' to 'm_strReceived += m_pszRxBuffer;' which is interesting, but if you have a complete description that would be better.

I guess the next step is to get these changes put into the FVT suite for testing before committing.

Regards,

Fred Preston.

> Multi chunked http responses miss some bytes randomly
> -----------------------------------------------------
>
>          Key: AXISCPP-609
>          URL: http://issues.apache.org/jira/browse/AXISCPP-609
>      Project: Axis-C++
>         Type: Bug
>   Components: Transport (axis3)
>     Versions: unspecified
>  Environment: The issue is not specific of a particular environment
>     Reporter: Arcangelo Di Balsamo
>  Attachments: HTTPTransport.cpp, HTTPTransport.cpp, HTTPTransport.cpp, HTTPTransport.cpp, HTTPTransport.cpp, HTTPTransport.dll, HTTPTransport.hpp, HTTPTransport.zip, axis-c-1-5_transport.zip
>
> Multi chunked http responses miss some bytes randomly.
> The issue is partially related to the AXISCPP-518 reported by myself too.
> The bug is in the getBytes method of HTTPTransport class.
> I made a tentative fix, that I've deeply tested.
> In the fix I did, I've changed a little bit the implementation logic.
> Splitting the getBytes method in more methods should be a nice to have.
> I'm attaching the modified file.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira