Skip to content
This repository has been archived by the owner on Sep 25, 2020. It is now read-only.

Function returns parser error with version 12.4.2 #42

Open
AudLaw opened this issue Sep 15, 2015 · 2 comments
Open

Function returns parser error with version 12.4.2 #42

AudLaw opened this issue Sep 15, 2015 · 2 comments

Comments

@AudLaw
Copy link

AudLaw commented Sep 15, 2015

Issue with using tcurl to hit this endpoint M::N defined as below:

struct Params {
1: required string uuid
2: optional string id
}

service M {
bool N(
1: required Params params
)

Get this assertion error:

AssertionError: "Can't get reader/writer for definition with unknown type undefined" undefined "undefined"
at Thrift.resolve (/usr/lib/node_modules/tcurl/node_modules/tchannel/node_modules/thriftrw/thrift.js:237:16)
at ThriftField.link (/usr/lib/node_modules/tcurl/node_modules/tchannel/node_modules/thriftrw/struct.js:59:27)
at ThriftStruct.link (/usr/lib/node_modules/tcurl/node_modules/tchannel/node_modules/thriftrw/struct.js:184:28)
at Thrift.link (/usr/lib/node_modules/tcurl/node_modules/tchannel/node_modules/thriftrw/thrift.js:199:32)
at new Thrift (/usr/lib/node_modules/tcurl/node_modules/tchannel/node_modules/thriftrw/thrift.js:69:10)
at new TChannelAsThrift (/usr/lib/node_modules/tcurl/node_modules/tchannel/as/thrift.js:46:17)
at asThrift (/usr/lib/node_modules/tcurl/index.js:324:18)
at onIdentified (/usr/lib/node_modules/tcurl/index.js:278:13)
at finish (/usr/lib/node_modules/tcurl/node_modules/tchannel/peer.js:266:9)
at Array.onIdentified as 1

Not seeing the problem after switching back to tcurl 4.12.1 from 4.12.2

@prashantv
Copy link
Contributor

@kriskowal thriftrw issue?

@AudLaw
Copy link
Author

AudLaw commented Sep 15, 2015

Should have mentioned, think may be thriftrw/thriftrw-node#58

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants