Tzu Gwo
Tzu Gwo
I will try to reproduce it, thanks
Thriftpy was not supported any more, it will be merged into [thriftpy2](https://github.com/Thriftpy/thriftpy2/pull/9), thanks!
Yes, I think Thriftpy should validate the arguments are all passed, even it breaks the compatibility.
@iamsudip Thank you, I think we can modify the function `thrift.py:init_func_generator`, and change all required field from keyword arguments to normal arguments, and please make sure that the argument sequence...
Thriftpy parser depends definition sequence, please move the definition of `TProgressUpdateResp` front of `TGetOperationStatusResp`, it's a known defect, but `Thriftpy` was not supported any more, please move to [thriftpy2](https://github.com/Thriftpy/thriftpy2) and...
Hi all, I tried to fix it in [thriftpy2](https://github.com/Thriftpy/thriftpy2/pull/21) and I have already tested thrift file be gave above, expect for your feedback, thanks!
it may be caused by https://github.com/Thriftpy/thriftpy2/pull/158, could you please try to install the master branch of thriftpy2?
not 0.4.14, please install the master branch directly from the source code, the fixed code of #158 is not released yet, but if you could make a quick verification.
hi @Maxsparrow @Rosadosa could you please give me a test case to reproduce this issue?
@Maxsparrow Sorry I am not familiar with samba or thrift-sasl, maybe it is better to have more exactly info to point out that it is caused by thriftpy2.