Download & Extend

Make sure Drupal provides support for IPv6

Project:Drupal core
Version:7.x-dev
Component:base system
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (fixed)
Issue tags:ipv6

Issue Summary

The column 'hostname' in core tables is defined as varchar(128), but the the column 'ip' in {blocked_ips} table is defined as varchar(32), which is a bug in D7, I think.

There's an additional issue that could be reviewed... I'm not sure if there are other places to review to make sure Drupal fully supports IPv6 (validations, preg, etc.), but maybe it would worth to check it out, just in case. The reason for this is: IPv6ActNow which is operated by RIPE NCC itself.

Comments

#3

A few more links:

http://www.ipv6day.org/
http://www.ipv6-to-standard.org/
http://bgp.potaroo.net/index-ale.html

This one is pretty interesting reading:

http://www.potaroo.net/tools/ipv4/

To quote todays report:

- "Projected IANA Unallocated Address Pool Exhaustion: 06-Sep-2011"
- "Projected RIR Unallocated Address Pool Exhaustion: 31-Aug-2012"

IPv4 exhaustion counters:

http://ipv6.he.net/statistics/ (ByeBye v4 Hurricane Electric has a series of applications, widgets and JavaScript snippets to display the estimated exhaustion time and rate of IPv4, and the growing adoption of IPv6.)

#4

The lenght of an IPv6 address is 40 characters, for the record:

$ echo 0000:0000:0000:0000:0000:0000:0000:0001 | wc
1 1 40

#5

I started to tag issues in the queue with the "ipv6" tag. There are only two other issues right now.

#6

Status:active» needs review

Here's a patch that fixes the ip field in the blocked list. I am certain there are other culprits, for example #523286: Cleanup IP blocking fixes the form in a similar way.

AttachmentSizeStatusTest resultOperations
issue_475140_ipv6.patch835 bytesIdlePASSED: [[SimpleTest]]: [MySQL] 17,541 pass(es).View details

#7

actually it's 39, as your command, above, counts the newline that echo writes. here's the correct command:

$ echo -n 0000:0000:0000:0000:0000:0000:0000:0001 | wc -c
39

but if you need to store the newline or a zero terminated character or otherwise, it's safe to allow 40 characters.

#8

Status:needs review» reviewed & tested by the community

This is ok

#9

Status:reviewed & tested by the community» fixed

Committed to CVS HEAD. Thanks.

#10

Status:fixed» closed (fixed)

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