From a9f9f725b1e56099932cb6aba7d1548d6d16b9c7 Mon Sep 17 00:00:00 2001
From: Shivaram Lingamneni <slingamn@cs.stanford.edu>
Date: Tue, 5 Feb 2019 04:29:32 -0500
Subject: [PATCH] review fixes

---
 irc/responsebuffer.go | 2 ++
 irc/server.go         | 2 +-
 oragono.yaml          | 4 ++++
 3 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/irc/responsebuffer.go b/irc/responsebuffer.go
index 7b769c65..3fccca01 100644
--- a/irc/responsebuffer.go
+++ b/irc/responsebuffer.go
@@ -47,6 +47,8 @@ func (rb *ResponseBuffer) Add(tags *map[string]ircmsg.TagValue, prefix string, c
 	if rb.finalized {
 		rb.target.server.logger.Error("internal", "message added to finalized ResponseBuffer, undefined behavior")
 		debug.PrintStack()
+		// TODO(dan): send a NOTICE to the end user with a string representation of the message,
+		// for debugging purposes
 		return
 	}
 
diff --git a/irc/server.go b/irc/server.go
index 0ae58875..37cfbf6e 100644
--- a/irc/server.go
+++ b/irc/server.go
@@ -158,7 +158,7 @@ func (server *Server) setISupport() (err error) {
 	isupport.Add("CASEMAPPING", "ascii")
 	isupport.Add("CHANMODES", strings.Join([]string{modes.Modes{modes.BanMask, modes.ExceptMask, modes.InviteMask}.String(), "", modes.Modes{modes.UserLimit, modes.Key}.String(), modes.Modes{modes.InviteOnly, modes.Moderated, modes.NoOutside, modes.OpOnlyTopic, modes.ChanRoleplaying, modes.Secret}.String()}, ","))
 	if config.History.Enabled && config.History.ChathistoryMax > 0 {
-		isupport.Add("CHATHISTORY", strconv.Itoa(config.History.ChathistoryMax))
+		isupport.Add("draft/CHATHISTORY", strconv.Itoa(config.History.ChathistoryMax))
 	}
 	isupport.Add("CHANNELLEN", strconv.Itoa(config.Limits.ChannelLen))
 	isupport.Add("CHANTYPES", "#")
diff --git a/oragono.yaml b/oragono.yaml
index 49aaad86..f8659b4e 100644
--- a/oragono.yaml
+++ b/oragono.yaml
@@ -470,6 +470,10 @@ fakelag:
 # message history tracking, for the RESUME extension and possibly other uses in future
 history:
     # should we store messages for later playback?
+    # the current implementation stores messages in RAM only; they do not persist
+    # across server restarts. however, you should disable this unless you understand
+    # how it interacts with the GDPR and/or any data privacy laws that apply
+    # in your country and the countries of your users.
     enabled: true
 
     # how many channel-specific events (messages, joins, parts) should be tracked per channel?