TUN-7065: Revert Ingress Rule check for named tunnel configurations

sudarshan-reddy · sudarshan-reddy · commit b8e610a06739 · 2023-02-01T10:08:10.000Z
Named Tunnels can exist without Ingress rules (They would default to
8080). Moreover, having this check also prevents warp tunnels from
starting since they do not need ingress rules.
diff --git a/cmd/cloudflared/tunnel/configuration.go b/cmd/cloudflared/tunnel/configuration.go
@@ -258,9 +258,6 @@ func prepareTunnelConfig(
 			return nil, nil, err
 		}
 	}
-	if ingressRules.IsEmpty() {
-		return nil, nil, ingress.ErrNoIngressRules
-	}
 
 	protocolSelector, err := connection.NewProtocolSelector(transportProtocol, cfg.WarpRouting.Enabled, namedTunnel, protocolFetcher, supervisor.ResolveTTL, log, c.Bool("post-quantum"))
 	if err != nil {

Original file line number	Diff line number	Diff line change
`@@ -258,9 +258,6 @@ func prepareTunnelConfig(`
`258`	`258`	`return nil, nil, err`
`259`	`259`	`}`
`260`	`260`	`}`
`261`		`- if ingressRules.IsEmpty() {`
`262`		`- return nil, nil, ingress.ErrNoIngressRules`
`263`		`- }`
`264`	`261`
`265`	`262`	`protocolSelector, err := connection.NewProtocolSelector(transportProtocol, cfg.WarpRouting.Enabled, namedTunnel, protocolFetcher, supervisor.ResolveTTL, log, c.Bool("post-quantum"))`
`266`	`263`	`if err != nil {`