Issue with named parameters in the request.params about pyramid_rpc HOT 9 CLOSED

Nevermind, no more than 30 seconds after submitting this I realized that pyramid_rpc is taking the arguments a step further than I expected.

For example, if the request is:

{"jsonrpc":"2.0","method":"service_render","id":1,"params":{"test":"1","test2":"2"}}

then the method signature will need to be:

@jsonrpc_method(endpoint='api')
def service_render(request, data, data2):

and then pyramid_rpc will map "1" to data and "2" to data2.

This works, but is slightly unintuitive. I will propose some additional documentation around this.

Thank you and hopefully this issue will help someone else in the future.

from pyramid_rpc.

I am making a mess of this issue now, but it still appears to not be working correctly. Even with my understanding from my comment above, the values for data and data2 are not the values of the named parameters but rather the name.

As I said the values were "1" and "2" for data and data2 respectively. This is not happening, the values for data and data2 are actually "test" and "test2"

Sorry for the confusion but I think I have identified the problem. Hopefully we can find a solution.

from pyramid_rpc.

Ok, last post I promise :-)

So it turns out pyramid_rpc is SUPER smart.

It maps the named parameters to their associated parameter names... Meaning in order to call:

def service_render(request, data, data2):

the params should be:

{"jsonrpc":"2.0","method":"service_render","id":1,"params":{"data":"1","data2":"2"}}

Then the values are correct inside the method.

Sorry for the mess.

from pyramid_rpc.

Named parameters are not supported with pyramid_rpc right now. The fact that its working for you is lucky because the params dict is unordered.

I haven't seen a compelling reason to support named parameters. In all of the servers I've looked at while implementing JSON-RPC I have never seen one that supported it. If you know of one that does, let me know and I'll think about it. Same goes for clients. XML-RPC doesn't support them and gets along just fine because in reality it is RPC where you are calling a function with a defined signature, so it's on you (the client) to provide the arguments to match the signature in the correct order.

from pyramid_rpc.

thank you for the follow up mmerickel.

Named parameters isn't that important to me. I think the only reason I ventured down that path was because I was getting inconsistent or unintended data being passed to my json-rpc exposed functions.

As I was saying before, I was getting the name of the parameters and not the values.

I would propose changing line 126 in jsonrpc.py from:

request.rpc_args = tuple(body.get('params', []))

to

request.rpc_args = tuple(body.get('params', []).values())

I very well may not understand the full repercussions of this change, but in my anecdotal scenario it fixed my issues.

from pyramid_rpc.

I will think about adding support for named parameters, but like I said: it's not supported right now. Thus, you should not be sending a dictionary as the params argument.

from pyramid_rpc.

I spent about one hour today to figure out, why my named parameters sent by a java-jsonrpc-client do not make it to my rpc_method in pyramid. Actually i came here to have a look at the code, but then found this Issue here first.

Since named parameters are supported in the spec proposal ( http://en.wikipedia.org/wiki/JSON-RPC#Version_2.0_.28Specification_Proposal.29 ) I would love to see them supported by pyramid_rpc.

from pyramid_rpc.

implemented in pull request #17

from pyramid_rpc.

The trunk now supports named parameters. This will be in the next release.

from pyramid_rpc.