search

neo-project / neo-modules

MIT License
60 stars 100 forks source link

RpcServer should support missing params request property #769

Closed devhawk closed 2 years ago

devhawk commented 2 years ago

Fixes #768

Note, there are no tests for RpcServer currently. Should I add one for this functionality?

roman-khimov commented 2 years ago

See also #747.

devhawk commented 2 years ago

See also #747.

I think this is a cleaner fix than #747, but I'm cool with merging either. @shargon make the call

shargon commented 2 years ago

See also #747.

I think this is a cleaner fix than #747, but I'm cool with merging either. @shargon make the call

Agree

superboyiii commented 2 years ago

Tested OK.

superboyiii commented 2 years ago

@erikzhang Could you have a look?