Resolve some basic comments

This commit is contained in:
Owen
2025-06-22 11:38:43 -04:00
parent 040f5375a6
commit 5f14ff3a07
2 changed files with 8 additions and 5 deletions

View File

@@ -75,7 +75,6 @@ func IsWithinHostNetwork(socketPath string, targetAddress string, targetPort int
// Always enforce network validation
containers, err := ListContainers(socketPath, true)
if err != nil {
return false, err
}
@@ -142,7 +141,7 @@ func ListContainers(socketPath string, enforceNetworkValidation bool) ([]Contain
hostContainer, err := getHostContainer(ctx, cli)
if enforceNetworkValidation && err != nil {
return nil, fmt.Errorf("network validation enforced, cannot validate due to: %v", err)
return nil, fmt.Errorf("network validation enforced, cannot validate due to: %w", err)
}
// We may not be able to get back host container in scenarios like running the container in network mode 'host'
@@ -262,4 +261,4 @@ func getHostContainer(dockerContext context.Context, dockerClient *client.Client
}
return &hostContainer, nil
}
}

View File

@@ -792,7 +792,11 @@ func updateTargets(pm *proxy.ProxyManager, action string, tunnelIP string, proto
if action == "add" {
targetAddress := parts[1]
targetPort, _ := strconv.Atoi(parts[2])
targetPort, err := strconv.Atoi(parts[2])
if err != nil {
logger.Info("Invalid target port: %s", parts[2])
continue
}
combinedAddress := targetAddress + ":" + parts[2]
// Call updown script if provided
@@ -807,7 +811,7 @@ func updateTargets(pm *proxy.ProxyManager, action string, tunnelIP string, proto
}
// Only remove the specific target if it exists
err := pm.RemoveTarget(proto, tunnelIP, port)
err = pm.RemoveTarget(proto, tunnelIP, port)
if err != nil {
// Ignore "target not found" errors as this is expected for new targets
if !strings.Contains(err.Error(), "target not found") {