You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "T Jake Luciani (JIRA)" <ji...@apache.org> on 2010/07/10 17:00:52 UTC
[jira] Resolved: (THRIFT-815) Deserialization of lists is
critically broken.
[ https://issues.apache.org/jira/browse/THRIFT-815?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
T Jake Luciani resolved THRIFT-815.
-----------------------------------
Assignee: T Jake Luciani
Fix Version/s: 0.4
Resolution: Fixed
Fixed, thanks for finding this.
> 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.