thrift-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jordan (JIRA)" <j...@apache.org>
Subject [jira] Reopened: (THRIFT-815) Deserialization of lists is critically broken.
Date Sat, 10 Jul 2010 16:58:51 GMT

     [ https://issues.apache.org/jira/browse/THRIFT-815?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Jordan reopened THRIFT-815:
---------------------------


See my last comment. (I think the provided fix won't fix the case when lists contain objects.
)

> Deserialization of lists is critically broken.
> ----------------------------------------------
>
>                 Key: THRIFT-815
>                 URL: https://issues.apache.org/jira/browse/THRIFT-815
>             Project: Thrift
>          Issue Type: Bug
>          Components: Compiler (JavaScript)
>    Affects Versions: 0.4
>            Reporter: Jordan
>            Assignee: T Jake Luciani
>            Priority: Critical
>             Fix For: 0.4
>
>         Attachments: 813-1.patch
>
>
> Edit the test code that comes with the js language target:
>   var list = [1,2,3]; 
>   var ret = client.testList(list);
>   debugger;
> ret comes back as [3,3,3] when it should be echoed back as [1,2,3]
> The test case never caught this because it only verified the size, and not the contents
of the returned array. 
> I cannot find an immediate workaround, but I will try wrapping it in a dummy type like
ListServiceResponse. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message