feat: unban ipv6 address (#3573)

* feat: unban ipv6 address

* chore: update unit tests

Author: sweatybridge

internal/bans/update/update.go

@@ -13,21 +13,17 @@ import (
 
 func validateIps(ips []string) error {
 	for _, ip := range ips {
-		ip := net.ParseIP(ip)
-		if ip.To4() == nil {
-			return errors.Errorf("only IPv4 supported at the moment: %s", ip)
+		if net.ParseIP(ip) == nil {
+			return errors.Errorf("invalid IP address: %s", ip)
 		}
 	}
 	return nil
 }
 
 func Run(ctx context.Context, projectRef string, dbIpsToUnban []string, fsys afero.Fs) error {
 	// 1. sanity checks
-	{
-		err := validateIps(dbIpsToUnban)
-		if err != nil {
-			return err
-		}
+	if err := validateIps(dbIpsToUnban); err != nil {
+		return err
 	}
 
 	// 2. remove bans

internal/bans/update/update_test.go

@@ -8,10 +8,15 @@ import (
 
 func TestPrivateSubnet(t *testing.T) {
 	err := validateIps([]string{"12.3.4.5", "10.0.0.0", "1.2.3.1"})
-	assert.Nil(t, err)
+	assert.NoError(t, err)
 }
 
-func TestIpv4(t *testing.T) {
-	err := validateIps([]string{"12.3.4.5", "2001:db8:abcd:0012::0", "1.2.3.1"})
-	assert.ErrorContains(t, err, "only IPv4 supported at the moment: 2001:db8:abcd:12::")
+func TestIPv6(t *testing.T) {
+	err := validateIps([]string{"2001:db8:abcd:0012::0", "::0"})
+	assert.NoError(t, err)
+}
+
+func TestInvalidAddress(t *testing.T) {
+	err := validateIps([]string{"12.3.4"})
+	assert.ErrorContains(t, err, "invalid IP address: 12.3.4")
 }