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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Bastlynn’s picture

Status: Active » Needs review
FileSize
1.39 KB
927 bytes

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

Liam McDermott’s picture

Status: Needs review » Fixed

Thanks 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! :)

Bastlynn’s picture

Whoa - yay! I'd actually completely forgotten I got this patch out there. Thanks! :)

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.