Improve receive message error handling #158
Comments
portante
added a commit
to portante/gosmee
that referenced
this issue
Apr 27, 2023
See r3labs/sse#158, "Improve receive message error handling." Fixes chmouel#77, "`ERROR gosmee bufio.Scanner: token too long`".
portante
added a commit
to portante/gosmee
that referenced
this issue
Apr 27, 2023
See r3labs/sse#158, "Improve receive message error handling." Fixes chmouel#77, "`ERROR gosmee bufio.Scanner: token too long`".
portante
added a commit
to portante/gosmee
that referenced
this issue
Apr 27, 2023
See r3labs/sse#158, "Improve receive message error handling." Fixes chmouel#77, "`ERROR gosmee bufio.Scanner: token too long`".
portante
added a commit
to portante/gosmee
that referenced
this issue
Apr 29, 2023
See r3labs/sse#158, "Improve receive message error handling." Fixes chmouel#77, "`ERROR gosmee bufio.Scanner: token too long`".
chmouel
pushed a commit
to chmouel/gosmee
that referenced
this issue
May 2, 2023
See r3labs/sse#158, "Improve receive message error handling." Fixes #77, "`ERROR gosmee bufio.Scanner: token too long`".
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
See also #97 and chmouel/gosmee#77.
If a very large payload is published, one that is bigger than the 65K default buffer size, the client silently re-subscribes without any indication of an error.
Adding some kind of
OnError()event handling describing the failure mode would have saved days of debugging (yes,straceis your friend).The text was updated successfully, but these errors were encountered: