Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
IPv6 can be up to 40 characters long, the table is only 15. This can be a critical error for users posting and interacting with the site as they'll hit an SQL error with every action, effectively locking them out using the site. All we should need for this is an update to lengthen the field.
Comment | File | Size | Author |
---|---|---|---|
#1 | user_stats-d6-1789120-1.patch | 927 bytes | Bastlynn |
#1 | user_stats-d7-1789120-1.patch | 1.39 KB | Bastlynn |
Comments
Comment #1
Bastlynn CreditAttribution: Bastlynn commentedPatches attached, I looked over the rest of the code to see if length would impact it and it doesn't look like it would. But let me know if otherwise.
Comment #2
Liam McDermott CreditAttribution: Liam McDermott commentedThanks very much for the patches, this is a very welcome change!
The D7 version of your patch was committed with a change to remove the Drupal 6 update function (I already removed it for another issue).
The D6 version I committed verbatim. Thanks again for the patches! :)
Comment #3
Bastlynn CreditAttribution: Bastlynn commentedWhoa - yay! I'd actually completely forgotten I got this patch out there. Thanks! :)