Fix saving `+draft/react` is msgstores v1 APPLIED

Calvin Lee: 1
 Fix saving `+draft/react` is msgstores

 1 files changed, 4 insertions(+), 3 deletions(-)
Export patchset (mbox)
How do I use this?

Copy & paste the following snippet into your terminal to import this patchset into git:

curl -s https://lists.sr.ht/~emersion/soju-dev/patches/46053/mbox | git am -3
Learn more about email & git

[PATCH] Fix saving `+draft/react` is msgstores Export this patch

This commit fixes a bug where only TAGMSGs with a `+react` tag would be
stored, when a `+draft/react` tag is normative. I have kept messages
containing `+react` as well, as they will most likely be used in the

I have also reversed the order of tag inspection, as I found the new
order to be more readable.
 upstream.go | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/upstream.go b/upstream.go
index 80f0517..5bffe3d 100644
--- a/upstream.go
+++ b/upstream.go
@@ -49,7 +49,8 @@ var permanentUpstreamCaps = map[string]bool{
// storableMessageTags is the static list of message tags that will cause
// a TAGMSG to be stored.
var storableMessageTags = map[string]bool{
	"+react": true,
	"+draft/react": true,
	"+react":       true,

type registrationError struct {
@@ -2099,8 +2100,8 @@ func (uc *upstreamConn) appendLog(entity string, msg *irc.Message) (msgID string
	if msg.Command == "TAGMSG" {
		store := false
		for tag := range storableMessageTags {
			if _, ok := msg.Tags[tag]; ok {
		for tag := range msg.Tags {
			if storableMessageTags[tag] {
				store = true
Pushed, thanks!