search

NarekAproyan / openrtb

Automatically exported from code.google.com/p/openrtb
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

IPv6 in 1.9c draft #9

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
In the 1.9c draft spec there is the IP address attribute in the device object. 
But it is only specified as 15 chars long, which is fine for IPv4, but it 
should be extended to utilize IPv6 as well, which is 39 characters. Perhaps it 
make sense to have two different fields?

Patrik

Original issue reported on code.google.com by patrik.o...@admeta.com on 17 May 2011 at 10:34

GoogleCodeExporter commented 9 years ago 
Hi,

In general, we are considering taking all the the string limits off the spec, 
since there are many issues like this.

Original comment by willard....@gtempaccount.com on 17 May 2011 at 1:44

GoogleCodeExporter commented 9 years ago 
I'm leery about removing all length restrictions from fields - or unbounded 
fields in general. Past bad experiences with garbage data entering systems, 
especially given that many of our data types will effectively end up as 
basically being not much more than "String" of some sort or another.

In this particular case, 39 characters is I think a nice flag that the field 
can contain either IPv4 or IPv6 values, since else I guarantee you we'll see a 
bunch of people only setup to expect/parse IPv4, regardless of what the docs 
say.

Original comment by j...@shao.org on 18 May 2011 at 10:28

GoogleCodeExporter commented 9 years ago

Original comment by js...@pulsepoint.com on 18 May 2011 at 10:29

GoogleCodeExporter commented 9 years ago

Original comment by js...@pulsepoint.com on 18 May 2011 at 10:29