LDAP for POSIX accounts extension

Hi all,

I am trying to use Keycloak for logging in to an Open OnDemand Linux cluster frontend. The catch is that I would like to do all user management from Keycloak, but Open OnDemand requires complete POSIX account information, including a unique uidNumber. This can be solved mostly using LDAP user federation, but I could not find a way to automatically set the uidNumber and the home directory, so I created an extension (extending AbstractLDAPStorageMapper) to do this:

This approach appears to work in a local test environment, but before deploying it I just wanted to double-check here if this is an acceptable way of doing this, and that I’m not missing any obvious caveats?