Merge pull request #42 from ocampeau/master

Fix bug where instance only has either public or private ip.
gjbae1212 · Aug 29, 2021 · 9cadf4f · 9cadf4f
2 parents e65a494 + 2f18966
commit 9cadf4f
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/cmd/internal.go b/cmd/internal.go
@@ -501,10 +501,10 @@ func findInstanceIdByIp(sess *session.Session, region, ip string) (string, error
 	}
 	for _, rv := range output.Reservations {
 		for _, inst := range rv.Instances {
-			if inst.PublicIpAddress == nil || inst.PrivateIpAddress == nil {
-				continue
+			if inst.PublicIpAddress != nil && ip == *inst.PublicIpAddress{
+				return *inst.InstanceId, nil
 			}
-			if ip == *inst.PublicIpAddress || ip == *inst.PrivateIpAddress {
+			if inst.PrivateIpAddress != nil && ip == *inst.PrivateIpAddress {
 				return *inst.InstanceId, nil
 			}
 		}