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.
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.