ircevent: allow message truncation by default

Question

ircevent: allow message truncation by default

Closed this issue 3 years ago · 2 comments

After the dust settled with the truncation changes (#45, #48), we made LineBytesStrict return ErrorBodyTooLong if the message is truncated.

ircevent does not check for this error, so truncation causes outgoing messages to be dropped. This is probably not the behavior bot authors expect. We should add a config variable like DisallowTruncation (i.e., truncation is allowed by default), and then check it and check that error value.

Answer 1 · 2021-04-20T13:12:22.000Z

We should also expose a normalizer: thoj/go-ircevent#127

Answer 2 · 2021-04-23T15:38:57.000Z

I went with AllowTruncation (i.e., truncation disabled by default).