Re: [aur-dev] [PATCH] Store last login address as plain text

On Mon, 2016-03-14 at 10:12 +0100, Marcel Korpel wrote:

* Lukas Fleischer (Sun, 13 Mar 2016 11:49:01 +0100):

...

- LastLoginIPAddress INTEGER UNSIGNED NOT NULL DEFAULT 0, + LastLoginIPAddress VARCHAR(40) NULL DEFAULT NULL,

VARCHAR(40) is too short, I think, see http://stackoverflow.com/q/166132/258127

According to Linux header files, the maximum length of an IPv6 address is 45 characters.

Regards, Marcel

I'm not 100% sure how this actually stores the address, but if it's stored in its hex form, it can only be as large as 39 characters (4*8+7) making 40 characters more than enough. Mark Weiman