hacksong / jsonrpc4j

Automatically exported from code.google.com/p/jsonrpc4j
0 stars 0 forks source link

Add JsonRpcServiceExporter configuration properties to AutoJsonRpcServiceExporter #46

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
We needed to define our own ErrorResolver but wanted to use the 
AutoJsonRpcServiceExporter in our project.

So we added the configuration properties of the JsonRpcServiceExporter also to 
the AutoJsonRpcServiceExporter and pass the values to the dynamically created 
JsonRpcServiceExporter beans.

See the attached patch for details.

Any hints to avoid the code duplication between AbstractJsonServiceExporter and 
the new version of AutoJsonRpcServiceExporter are welcome.

Thanks for your great work on this lib and the nice Spring integration!

Original issue reported on code.google.com by knut.san...@gmail.com on 14 Dec 2012 at 1:06

Attachments:

GoogleCodeExporter commented 9 years ago
I'll take a look at the patch, thanks.

Original comment by brian.di...@gmail.com on 30 Jan 2013 at 5:31

GoogleCodeExporter commented 9 years ago
We could probably create a JsonServiceConfiguration object with all of the 
options - but for now i've just applied your patch. Thanks.

Original comment by brian.di...@gmail.com on 16 Feb 2013 at 8:33

GoogleCodeExporter commented 9 years ago

Original comment by brian.di...@gmail.com on 16 Feb 2013 at 8:33