search

Miserlou / django-easy-timezones

Easy timezones for Django based on GeoIP
http://gun.io/blog/django-easy-timezones/
Apache License 2.0
203 stars 44 forks source link

added IPV6 support #7

Closed lucas03 closed 8 years ago

robbyt commented 9 years ago

I think that this should be implemented as a new middleware, instead of updating the existing middlware. The user can easily select which middleware is needed.

lucas03 commented 9 years ago

About exception instead of None, you are probably right. If I get some time I will try to set up IPv6 on my computer and test it. But I do not think it should be implemented as new middleware. I guess it would be better for compativility with old version. But importing two middleware is unnecessary, if it does the same thing (getting location from ip, no matter if its IPv4 or IPv6).

Miserlou commented 8 years ago

Fixed in 0.6+