[ http://issues.apache.org/jira/browse/XMLRPC-117?page=all ]

Jimisola Laursen updated XMLRPC-117:
------------------------------------

    Attachment: XMLRPC-117.diff

Added instance variables for AuthenticationHandler, 
RequestProcessorFactoryFactory and TypeConverterFactory to XmlRpcServer and 
then created delegate methods in XmlRpcServlet.

However, I am not sure whether the instance variables should in fact be in 
XmlRpcServer or if they can be moved up to XmlRpcServlet. 

Notes:

1.

This patch does not handle the case of changing AuthenticationHandler, 
RequestProcessorFactoryFactory and/or TypeConverterFactory after the init 
method has been called by the servlet container.

I'll leave it up to you to decide if it should be handled. I think that is it 
improper use and perhaps it is enough to use document it.

2.

This patch honestly haven't been tested. There is not much to test and I 
believe that you looking at the code and me running it once the its available 
as a Maven SNAPSHOT should be sufficient.

> XmlRpcServlet  should allow for usage of AuthenticationHandler, 
> RequestProcessorFactoryFactory and TypeConverterFactory implementations to be 
> configurable
> ----------------------------------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: XMLRPC-117
>                 URL: http://issues.apache.org/jira/browse/XMLRPC-117
>             Project: XML-RPC
>          Issue Type: Improvement
>            Reporter: Jimisola Laursen
>         Attachments: XMLRPC-117.diff
>
>
> The XmlRpcServlet should have setters for
>  - AuthenticationHandler 
>  - RequestProcessorFactoryFactory
>  - TypeConverterFactory 
> and it shall use this accordingly.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to