Enable message-tags only when all upstreams support it v1 APPLIED

delthas: 1
 Enable message-tags only when all upstreams support it

 1 files changed, 1 insertions(+), 1 deletions(-)

[PATCH] Enable message-tags only when all upstreams support it Export this patch

Previously, we would always advertise mesasge-tags. This made
downstreams believe they could send TAGMSG to the upstream, even though
the upstream did not support it.
---
 downstream.go | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/downstream.go b/downstream.go
index 09f352c..eae4cdd 100644
--- a/downstream.go
+++ b/downstream.go
@@ -229,7 +229,6 @@ var permanentDownstreamCaps = map[string]string{
 	"cap-notify":    "",
 	"echo-message":  "",
 	"invite-notify": "",
-	"message-tags":  "",
 	"server-time":   "",
 	"setname":       "",
 
@@ -247,6 +246,7 @@ var needAllDownstreamCaps = map[string]string{
 	"away-notify":    "",
 	"chghost":        "",
 	"extended-join":  "",
+	"message-tags":   "",
 	"multi-prefix":   "",
 
 	"draft/extended-monitor": "",

base-commit: abe5291b62aa9616a1fa6c83ec75823ad14700e4
-- 
2.17.1

Simon Ser <contact@emersion.fr>

1 year, 1 month ago

Pushed, thanks!