Hi Tommi.

Which jsonrpc spec are you following to implement (or intending to
implement) on cxxtools? In some places it seems you're using the spec
1.0, but in others it seems 2.0...

I've changed a bit your code to make it more compatible with the
jsonrpc 2.0 spec and I only tested cxxtools as server.

The clients I successful tested with this server impl were:
- cxxtools client
- rpcjson (ruby)
- php-json-rpc (https://bitbucket.org/jbg/php-json-rpc/src)

Clients I tested but they didn't work:
- python jsonrpclib https://github.com/joshmarshall/jsonrpclib

The code I tested were the calc service and a personal code with some
simple methods. Both worked fine.

The changes I did: add the "jsonrpc": "2.0" pair, remove error key
from successful response, remove result code from error response,
allow the omission of paramaters in request and change the required
content-type of both request and responses to "application/json"
because I can't see any reasons to use other strings.

As the ruby client I tested use a content-type
"application/x-www-form-urlencoded" for requests, I also added it into
code, but I think it can be removed, because who's wrong is probably
the client :-)

I didn't tested the jsonrpc over tcp, without http, but I imagine it
will work fine.

This is not a real patch, because I did it only to see if I'd be able
to talk with clients in another languages from my c++ application. And
it proved it's possible :-)

ps: have you already thought about migrating tntnet and cia. from svn
to git? With git is much easier to clone the repositories, create
patches, etc :-)

-- 
-----
Sent from my Atari

Attachment: 2.0.diff
Description: Binary data

------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
Tntnet-general mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/tntnet-general

Reply via email to