bug: fix unintentional provisioning of {Allow,Deny}CIDR

mohammed90 · Mar 28, 2023 · 8d3ac56 · 8d3ac56
1 parent e3e4cbf
commit 8d3ac56
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/http.go b/http.go
@@ -85,11 +85,11 @@ func (t *HTTP) provisionOpts() error {
 		t.opts = append(t.opts, config.WithMetadata(t.Metadata))
 	}
 
-	if t.AllowCIDR != nil {
+	if len(t.AllowCIDR) > 0 {
 		t.opts = append(t.opts, config.WithAllowCIDRString(t.AllowCIDR...))
 	}
 
-	if t.DenyCIDR != nil {
+	if len(t.DenyCIDR) > 0 {
 		t.opts = append(t.opts, config.WithDenyCIDRString(t.DenyCIDR...))
 	}
 

diff --git a/tcp.go b/tcp.go
@@ -54,11 +54,11 @@ func (t *TCP) provisionOpts() error {
 		t.opts = append(t.opts, config.WithMetadata(t.Metadata))
 	}
 
-	if t.AllowCIDR != nil {
+	if len(t.AllowCIDR) > 0 {
 		t.opts = append(t.opts, config.WithAllowCIDRString(t.AllowCIDR...))
 	}
 
-	if t.DenyCIDR != nil {
+	if len(t.DenyCIDR) > 0 {
 		t.opts = append(t.opts, config.WithDenyCIDRString(t.DenyCIDR...))
 	}
 

diff --git a/tls.go b/tls.go
@@ -65,11 +65,11 @@ func (t *TLS) provisionOpts() error {
 		t.opts = append(t.opts, config.WithMetadata(t.Metadata))
 	}
 
-	if t.AllowCIDR != nil {
+	if len(t.AllowCIDR) > 0 {
 		t.opts = append(t.opts, config.WithAllowCIDRString(t.AllowCIDR...))
 	}
 
-	if t.DenyCIDR != nil {
+	if len(t.DenyCIDR) > 0 {
 		t.opts = append(t.opts, config.WithDenyCIDRString(t.DenyCIDR...))
 	}