Log and return instead of panicing due to dead containers

deviantintegral · deviantintegral · commit 160236dae63b · 2022-10-03T14:21:11.000-04:00
diff --git a/forwarder.go b/forwarder.go
@@ -155,7 +155,10 @@ func (f *Forwarder) startForwarding(id string) {
 					// The container has been removed, so we can't do anything to get logs.
 					return
 				}
-				panic(err)
+				// The other common error is that the container is marked for removal. Unfortunately, the
+				// SDK doesn't send a specific error message, so we log and return.
+				log.Println(err)
+				return
 			}
 
 			// Good reference at https://www.linode.com/docs/guides/developing-udp-and-tcp-clients-and-servers-in-go/

Original file line number	Diff line number	Diff line change
`@@ -155,7 +155,10 @@ func (f *Forwarder) startForwarding(id string) {`
`155`	`155`	`// The container has been removed, so we can't do anything to get logs.`
`156`	`156`	`return`
`157`	`157`	`}`
`158`		`- panic(err)`
	`158`	`+ // The other common error is that the container is marked for removal. Unfortunately, the`
	`159`	`+ // SDK doesn't send a specific error message, so we log and return.`
	`160`	`+ log.Println(err)`
	`161`	`+ return`
`159`	`162`	`}`
`160`	`163`
`161`	`164`	`// Good reference at https://www.linode.com/docs/guides/developing-udp-and-tcp-clients-and-servers-in-go/`