Data too long for column 'AllIPAddresses'

[JasonBarnabe]

A user has reported to me that when they try to sign in to my forum, they get

Fatal Error in Gdn_Database.Query();
Data too long for column 'AllIPAddresses' at row 1
update GDN_User `User` set DateLastActive = :DateLastActive, LastIPAddress = :LastIPAddress, CountVisits = :CountVisits, AllIPAddresses = :AllIPAddresses where UserID = :UserID
The error occurred on or near: /home/www/forum/library/database/class.database.php

I'm unsure what this field represents (all IP addresses used ever by this user?), but it seems like this should be truncated at the field size.

[linc]

Hi Jason, what version are you using?

[JasonBarnabe]

2.1.

[linc]

Thanks!

[JasonBarnabe]

After setting the column to null again for the problematic user, the problem went away for a bit then happened again for him. It has also occurred for another user.

[JasonBarnabe]

As a workaround, I've changed the schema:

ALTER TABLE GDN_User MODIFY COLUMN AllIPAddresses VARCHAR(500);

[austins]

The error Data too long for column 'AllIPAddresses' still happens in Vanilla 2.1.1 for some users.

In this case, the AllIPAddresses column is set to the default varchar(100) NULL type. The users who receive this error while signing in have a long string of IP addresses in that column (some of which are truncated), whereas users who aren't affected have a short string of IP addresses.

This only occurs if MySQL is set to strict mode (sql-mode setting contains STRICT_TRANS_TABLES), so a workaround would be to disable STRICT_TRANS_TABLES mode for MySQL or change the column character limit like @JasonBarnabe posted above.

[JasonBarnabe]

Use of ipFlood can cause this...

[mike-pt]

This still happens in v 2.1.3, ideally the code should clean the last ip while adding the new ones (if X ips are already there ofc)

[JasonBarnabe]

See PR #2493.