When Openid Connect module want to generate username during the connexion of a not existing user, if username length is higher than limitation (USERNAME_MAX_LENGTH = 60), then a fatal error occur.
In case username generated is higher than max length, we should change it so it can exceed 60 characters.
Two options :
1/ reduce username to length < 60 and suffix by an iterative number while username is not available.
2/ crypt username to a string of 60 characters
I have a preference for 2/ which is more generic.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | openid_connect-valid_username-2872912-4.patch | 2.78 KB | pjcdawkins |
| #3 | openid-connect-2872912-3.patch | 578 bytes | nod_ |
| #2 | username_too_long-2872912-2.patch | 885 bytes | goz |
Comments
Comment #2
goz commentedComment #3
nod_Solved it a different way (D8 Patch). md5 the id given back by the auth server, length is predictable that way.
Comment #4
pjcdawkins commentedThe rules for valid usernames are quite complex. This adds a function that sanitizes the new username according to Drupal core's rules.
Comment #6
sanduhrsCommitted, thanks!