janus-ssp / janus

Fully featured metadata registration administration module built on top of simpleSAMLphp.
Other
13 stars 8 forks source link

IPv6 support for Janus #395

Closed surfnet-niels closed 10 years ago

surfnet-niels commented 10 years ago

At various places in the database the IP address of the user changing the entity is recorded. The database however only seems to support IPv4 as IPv6 addresses get recorded, but are then truncated. See below an example of a IPv4 and an IPv6 record in the table 'janus__allowedEntity'

eid, revisionid, remoteeid, created, ip

30, 5, 0, 2011-06-14T10:17:54+02:00, 2001:610:188:43 36, 1, 0, 2011-06-14T10:49:40+02:00, 145.96.3.65

The following tables need fixing:

janusallowedEntity janusarp janusblockedEntity janusdisableConsent janusentity janushasEntity janusmessage janusmetadata janussubscription janususerData

lucasvanlierop commented 10 years ago

This was already fixed in Janus before the original WAYF version was merged with the Surfnet version. For surfnet this was fixed in Serviceregistry patch: https://github.com/OpenConext/OpenConext-serviceregistry/blob/master/database/patch/patch-0017.sql