Log and return instead of panicing due to dead containers (#3)

deviantintegral · web-flow · commit 40b3fecb8702 · 2022-10-03T14:35:52.000-04:00
diff --git a/forwarder.go b/forwarder.go
@@ -152,7 +152,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
`@@ -152,7 +152,10 @@ func (f *Forwarder) startForwarding(id string) {`
`152`	`152`	`// The container has been removed, so we can't do anything to get logs.`
`153`	`153`	`return`
`154`	`154`	`}`
`155`		`- panic(err)`
	`155`	`+ // The other common error is that the container is marked for removal. Unfortunately, the`
	`156`	`+ // SDK doesn't send a specific error message, so we log and return.`
	`157`	`+ log.Println(err)`
	`158`	`+ return`
`156`	`159`	`}`
`157`	`160`
`158`	`161`	`// Good reference at https://www.linode.com/docs/guides/developing-udp-and-tcp-clients-and-servers-in-go/`