Merge pull request #1084 from slingamn/monoffline

fix #1076
This commit is contained in:
Shivaram Lingamneni 2020-05-30 20:59:31 -07:00 committed by GitHub
commit 404e3c2ed8
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
4 changed files with 9 additions and 7 deletions

View File

@ -1332,7 +1332,7 @@ func (client *Client) destroy(session *Session) {
// alert monitors
if registered {
client.server.monitorManager.AlertAbout(client, false)
client.server.monitorManager.AlertAbout(details.nick, details.nickCasefolded, false)
}
// clean up monitor state
client.server.monitorManager.RemoveAll(client)

View File

@ -25,9 +25,7 @@ func (mm *MonitorManager) Initialize() {
}
// AlertAbout alerts everyone monitoring `client`'s nick that `client` is now {on,off}line.
func (manager *MonitorManager) AlertAbout(client *Client, online bool) {
cfnick := client.NickCasefolded()
nick := client.Nick()
func (manager *MonitorManager) AlertAbout(nick, cfnick string, online bool) {
var watchers []*Client
// safely copy the list of clients watching our nick
manager.RLock()
@ -42,7 +40,7 @@ func (manager *MonitorManager) AlertAbout(client *Client, online bool) {
}
for _, mClient := range watchers {
mClient.Send(nil, client.server.name, command, mClient.Nick(), nick)
mClient.Send(nil, mClient.server.name, command, mClient.Nick(), nick)
}
}

View File

@ -92,7 +92,11 @@ func performNickChange(server *Server, client *Client, target *Client, session *
}
if target.Registered() {
client.server.monitorManager.AlertAbout(target, true)
newCfnick := target.NickCasefolded()
if newCfnick != details.nickCasefolded {
client.server.monitorManager.AlertAbout(details.nick, details.nickCasefolded, false)
client.server.monitorManager.AlertAbout(assignedNickname, newCfnick, true)
}
target.nickTimer.Touch(rb)
} // else: these will be deferred to the end of registration (see #572)
return nil

View File

@ -276,7 +276,7 @@ func (server *Server) tryRegister(c *Client, session *Session) (exiting bool) {
// count new user in statistics
server.stats.Register(c.HasMode(modes.Invisible))
server.monitorManager.AlertAbout(c, true)
server.monitorManager.AlertAbout(c.Nick(), c.NickCasefolded(), true)
server.playRegistrationBurst(session)
return false