d7b9cb0019
Before this change, if an event fails to send on the first attempt, subsequent attempts will fail with context.Cancelled because the the caller of publish passes a cancellable timeout, which the publisher uses to send the event. The publisher returns immediately if the send fails, but adds the event to an async queue to try again. Meanwhile the caller will return cancelling the context. Additionally, subsequent attempts may fail to send because the timeout was expected to be for a single request but the queue sleeps for `attempt*time.Second`. In the shim service, the timeout was set to 5s, which means the send will fail with context.DeadlineExceeded before it reaches `maxRequeue` (which is currently 5). This change moves the timeout to the publisher so each send attempt gets its own timeout. Signed-off-by: Brian Goff <cpuguy83@gmail.com> |
||
|---|---|---|
| .. | ||
| publisher.go | ||
| shim_darwin.go | ||
| shim_freebsd.go | ||
| shim_linux.go | ||
| shim_test.go | ||
| shim_unix.go | ||
| shim_windows.go | ||
| shim.go | ||
| util_unix.go | ||
| util_windows.go | ||
| util.go | ||