Simon Ser
Simon Ser
https://www.rfc-editor.org/rfc/rfc5258.html
https://www.rfc-editor.org/rfc/rfc5182.html
https://www.rfc-editor.org/rfc/rfc4731.html
This is harmless, but clutters the logs. See https://github.com/emersion/go-imap/pull/398
## Broken The way channels are used makes it so updates are only sent to the first client. See https://github.com/emersion/go-imap/pull/336 ## Racy We read from the connection context: https://github.com/emersion/go-imap/blob/3bc38e360e3c75d52e3e2c60f3597369154ec4b4/server/server.go#L331 However...
This is https://github.com/emersion/go-imap/pull/300 rebased against master. The AutoLogout test has a race: ``` ================== WARNING: DATA RACE Write at 0x00c000172800 by goroutine 51: github.com/emersion/go-imap/server_test.TestLogin_AutoLogout() /home/simon/src/go-imap/server/cmd_noauth_test.go:133 +0x134 testing.tRunner() /usr/lib/go/src/testing/testing.go:909 +0x199 Previous...
This PR contains the memory backend changes extracted from https://github.com/emersion/go-imap/pull/300. cc @Neopallium
String constants are declared this way: ```go const ( // SetFlags replaces existing flags by new ones. SetFlags FlagsOp = "FLAGS" // AddFlags adds new flags. AddFlags = "+FLAGS" //...