Fix: Add reason, move role change to debug level

Signed-off-by: Knut Ahlers <knut@ahlers.me>
This commit is contained in:
Knut Ahlers 2021-08-26 00:55:26 +02:00
parent eee8ba9374
commit 8ff38fe99c
Signed by: luzifer
GPG key ID: 0066F03ED215AD7D

View file

@ -100,7 +100,7 @@ func (m modLiveRole) handlePresenceUpdate(d *discordgo.Session, p *discordgo.Pre
if err := exitFunc(p.GuildID, p.User.ID, member.Roles); err != nil { if err := exitFunc(p.GuildID, p.User.ID, member.Roles); err != nil {
logger.WithError(err).Error("Unable to update live-streamer-role") logger.WithError(err).Error("Unable to update live-streamer-role")
} }
logger.Info("Updated live-streamer-role") logger.Debug("Updated live-streamer-role")
} }
}() }()
@ -116,7 +116,7 @@ func (m modLiveRole) handlePresenceUpdate(d *discordgo.Session, p *discordgo.Pre
if activity == nil { if activity == nil {
// No streaming activity: Remove role // No streaming activity: Remove role
exitFunc = m.removeLiveStreamerRole exitFunc = m.removeLiveStreamerRole
logger = logger.WithField("action", "remove") logger = logger.WithFields(log.Fields{"action": "remove", "reason": "no activity"})
return return
} }
@ -124,14 +124,14 @@ func (m modLiveRole) handlePresenceUpdate(d *discordgo.Session, p *discordgo.Pre
if err != nil { if err != nil {
logger.WithError(err).WithField("url", activity.URL).Warning("Unable to parse activity URL") logger.WithError(err).WithField("url", activity.URL).Warning("Unable to parse activity URL")
exitFunc = m.removeLiveStreamerRole exitFunc = m.removeLiveStreamerRole
logger = logger.WithField("action", "remove") logger = logger.WithFields(log.Fields{"action": "remove", "reason": "broken activity URL"})
return return
} }
if u.Host != "www.twitch.tv" { if u.Host != "www.twitch.tv" {
logger.WithError(err).WithField("url", activity.URL).Warning("Activity is not on Twitch") logger.WithError(err).WithField("url", activity.URL).Warning("Activity is not on Twitch")
exitFunc = m.removeLiveStreamerRole exitFunc = m.removeLiveStreamerRole
logger = logger.WithField("action", "remove") logger = logger.WithFields(log.Fields{"action": "remove", "reason": "activity not on twitch"})
return return
} }
@ -147,13 +147,13 @@ func (m modLiveRole) handlePresenceUpdate(d *discordgo.Session, p *discordgo.Pre
if err != nil { if err != nil {
logger.WithError(err).WithField("user", strings.TrimLeft(u.Path, "/")).Warning("Unable to fetch streams for user") logger.WithError(err).WithField("user", strings.TrimLeft(u.Path, "/")).Warning("Unable to fetch streams for user")
exitFunc = m.removeLiveStreamerRole exitFunc = m.removeLiveStreamerRole
logger = logger.WithField("action", "remove") logger = logger.WithFields(log.Fields{"action": "remove", "reason": "error in getting streams"})
return return
} }
if len(streams.Data) > 0 { if len(streams.Data) > 0 {
exitFunc = m.addLiveStreamerRole exitFunc = m.addLiveStreamerRole
logger = logger.WithField("action", "add") logger = logger.WithFields(log.Fields{"action": "add", "reason": "stream found"})
} }
} }