This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author vstinner
Recipients acooke, cloverprince, draghuram, samwyse, vstinner
Date 2010-02-01.20:46:27
SpamBayes Score 0.02940265
Marked as misclassified No
Message-id <1265057188.83.0.684100386803.issue3058@psf.upfronthosting.co.za>
In-reply-to
Content
client_address is not interresting, but I prefer the handler object (SimpleXMLRPCRequestHandler instance). The handler contains more information:
 * handler.address_string() gives the FQDN
 * handler.connection.getpeername() gives the client address
 * handler.connection.getsockanem() gives the server address, useful is the server is listening to multiple addresses
 * etc.

If the server uses SSL, it would be possible to get more information about SSL from handler(.connection).

Your patch breaks the API for all (service) callbacks. Add an option to register_function() would allow a smoother transition (don't break API): you don't have to inherit from a new class. Eg. server.register_function(whoami, with_handler=True).
History
Date User Action Args
2010-02-01 20:46:29vstinnersetrecipients: + vstinner, draghuram, samwyse, acooke, cloverprince
2010-02-01 20:46:28vstinnersetmessageid: <1265057188.83.0.684100386803.issue3058@psf.upfronthosting.co.za>
2010-02-01 20:46:27vstinnerlinkissue3058 messages
2010-02-01 20:46:27vstinnercreate