Yeah it's more cleaner. I have changed it.
I checked the function signature in the library and you're right, "de" is setting the default region to Germany if no region code is provided in the number itself.
I saw same code pattern in AuthenticationController.java (which I have not touched) there was this exact same code to parse the phone number before sending PIN.
I am changing it to the much softer phrase: "Attempt to claim existing username {} by {}". How does it sound now?
For "undocumented", I agreed that it was slightly confusing terminology. Since the entries table maps a JID to a phone number, if an entry doesn't exist, it is "unmapped". So, I am changing the...
The regex "^(?![0-9]+$)[a-zA-Z0-9]+$" does two things: