Comment 12 for bug 1468422

Revision history for this message
Mike Rylander (mrylander) wrote :

Bill,

The new service looks good. In scanning the patch set, you use auth_internal for the service name, but register the method with a name containing auth-internal (_ vs -). I personally prefer - but I know XML::RPC is not a fan of that spelling. And, of course, it'll work just fine as it is right now, but may confuse devs calling the service.