Board index Bugs IPv6

Found a bug? Tell us!!
ridcully
 
Posts: 1
Joined: Thu Apr 12, 2007 8:08 am

Postby ridcully » Thu Apr 12, 2007 8:13 am

Die IP-Felder in den Tabellen serendipity_comments und serendipity_visitors sind zu klein. Daher gibt es mit Servern die ausschließlich IPv6 verwenden Probleme, da die IP nicht vollständig gespeichert wird. In den Kommentar-Mails werden die IPv6 richtig gespeichert.

Keine Ahnung, ob noch irgendwo eine IP gespeichert wird. Aber wahrscheinlich dürfte auch an anderen Stellen das IP-Feld zu klein sein...

User avatar
garvinhicking
Core Developer
 
Posts: 29966
Joined: Tue Sep 16, 2003 9:45 pm
Location: Cologne, Germany

Postby garvinhicking » Thu Apr 12, 2007 11:35 am

Hi!

Das stimmt, da wird es vermutlich einige Felder geben bei denen das so ist. Hat jemand Lust einen Patch dafür zu entwickeln und überall zu suchen?

(Code grep nach "REMOTE_ADDR", schauen wofür es verwendet wird und analysieren in welchem DB-Feld es landet)

Grüße,
Garvin
# Garvin Hicking (s9y Developer)
# Did I help you? Consider making me happy: http://wishes.garv.in/
# or use my PayPal account "paypal {at} supergarv (dot) de"
# My "other" hobby: http://flickr.garv.in/

puck
Regular
 
Posts: 14
Joined: Mon Apr 30, 2007 10:55 am
Location: New Zealand

Postby puck » Mon Apr 30, 2007 10:59 am

Hey,

I can't read/speak German but the gist of this message seems familiar.

I can't post comments to my blog anymore because I'm now using an IPv6 address. Looking in the serendipity_comments table in my database I see that the IP column is a character varying(15) which implies to me that there are underlying issues here. Would it be sufficient to change that column type to just a varchar or does the code perform logic on the IP address?

I'm running s9y v1.1. The host is http://blog.etc.gen.nz which is iPv6 enabled if you need somewhere to try it out.

Cheers!

User avatar
garvinhicking
Core Developer
 
Posts: 29966
Joined: Tue Sep 16, 2003 9:45 pm
Location: Cologne, Germany

Postby garvinhicking » Wed May 02, 2007 10:44 am

Hi!

Changing the column to varchar(64) or so in your case should suffice, yes.

I'll patch the upcoming s9y 1.2 version so that this will be the default.

Best regards,
Garvin
# Garvin Hicking (s9y Developer)
# Did I help you? Consider making me happy: http://wishes.garv.in/
# or use my PayPal account "paypal {at} supergarv (dot) de"
# My "other" hobby: http://flickr.garv.in/

puck
Regular
 
Posts: 14
Joined: Mon Apr 30, 2007 10:55 am
Location: New Zealand

Postby puck » Wed May 02, 2007 11:57 pm

garvinhicking wrote:Changing the column to varchar(64) or so in your case should suffice, yes.

I'll patch the upcoming s9y 1.2 version so that this will be the default.


Hey Gavin,

Yes, changing the database seems to have resolved that issue.

Thanks!



Return to Bugs

Who is online

Users browsing this forum: No registered users and 1 guest

cron