fix #1155

irc/client.go

@@ -723,16 +723,11 @@ func (client *Client) playReattachMessages(session *Session) {
 //
 
 // Touch indicates that we received a line from the client (so the connection is healthy
-// at this time, modulo network latency and fakelag). `active` means not a PING or suchlike
-// (i.e. the user should be sitting in front of their client).
-func (client *Client) Touch(active bool, session *Session) {
+// at this time, modulo network latency and fakelag).
+func (client *Client) Touch(session *Session) {
 	var markDirty bool
 	now := time.Now().UTC()
 	client.stateMutex.Lock()
-	if active {
-		client.lastActive = now
-		session.lastActive = now
-	}
 	if client.accountSettings.AutoreplayMissed || session.deviceID != "" {
 		client.setLastSeen(now, session.deviceID)
 		if now.Sub(client.lastSeenLastWrite) > lastSeenWriteInterval {

irc/commands.go

@@ -15,7 +15,6 @@ type Command struct {
 	handler        func(server *Server, client *Client, msg ircmsg.IrcMessage, rb *ResponseBuffer) bool
 	oper           bool
 	usablePreReg   bool
-	leaveClientIdle bool // if true, leaves the client active time alone
 	allowedInBatch bool // allowed in client-to-server batches
 	minParams      int
 	capabs         []string
@@ -67,7 +66,7 @@ func (cmd *Command) Run(server *Server, client *Client, session *Session, msg ir
 
 	// TODO: eliminate idletimer entirely in favor of this measurement
 	if client.registered {
-		client.Touch(!cmd.leaveClientIdle, session)
+		client.Touch(session)
 	}
 
 	return exiting
@@ -161,7 +160,6 @@ func init() {
 		"ISON": {
 			handler:   isonHandler,
 			minParams: 1,
-			leaveClientIdle: true,
 		},
 		"JOIN": {
 			handler:   joinHandler,
@@ -246,13 +244,11 @@ func init() {
 			handler:      pingHandler,
 			usablePreReg: true,
 			minParams:    1,
-			leaveClientIdle: true,
 		},
 		"PONG": {
 			handler:      pongHandler,
 			usablePreReg: true,
 			minParams:    1,
-			leaveClientIdle: true,
 		},
 		"PRIVMSG": {
 			handler:        messageHandler,
@@ -351,7 +347,6 @@ func init() {
 		"WHO": {
 			handler:   whoHandler,
 			minParams: 1,
-			leaveClientIdle: true,
 		},
 		"WHOIS": {
 			handler:   whoisHandler,

irc/getters.go

@@ -418,6 +418,14 @@ func (client *Client) detailsNoMutex() (result ClientDetails) {
 	return
 }
 
+func (client *Client) UpdateActive(session *Session) {
+	now := time.Now().UTC()
+	client.stateMutex.Lock()
+	defer client.stateMutex.Unlock()
+	client.lastActive = now
+	session.lastActive = now
+}
+
 func (channel *Channel) Name() string {
 	channel.stateMutex.RLock()
 	defer channel.stateMutex.RUnlock()

irc/handlers.go

@@ -1955,7 +1955,12 @@ func messageHandler(server *Server, client *Client, msg ircmsg.IrcMessage, rb *R
 		return false
 	}
 
-	if rb.session.isTor && utils.IsRestrictedCTCPMessage(message) {
+	isCTCP := utils.IsRestrictedCTCPMessage(message)
+	if histType == history.Privmsg && !isCTCP {
+		client.UpdateActive(rb.session)
+	}
+
+	if rb.session.isTor && isCTCP {
 		// note that error replies are never sent for NOTICE
 		if histType != history.Notice {
 			rb.Notice(client.t("CTCP messages are disabled over Tor"))