diff options
| author | Wim <wim@42.be> | 2019-01-30 23:26:36 +0100 |
|---|---|---|
| committer | Wim <wim@42.be> | 2019-01-30 23:28:37 +0100 |
| commit | 4ae028fe73c9b5217b3b4a403432d18bee18a057 (patch) | |
| tree | 5380a88faf2d208c4a31296ee2a8c33effc12cfc /bridge/slack/handlers.go | |
| parent | 707db950c83e7fe2578ec206c6eeb330cc2f5f5b (diff) | |
| download | matterbridge-msglm-4ae028fe73c9b5217b3b4a403432d18bee18a057.tar.gz matterbridge-msglm-4ae028fe73c9b5217b3b4a403432d18bee18a057.tar.bz2 matterbridge-msglm-4ae028fe73c9b5217b3b4a403432d18bee18a057.zip | |
Optimize handling of very large slack teams. Fixes #695
Stop getting users if we reach 2000 users. Slack will rate-limit us
even if we follow their limits.
This means that we now have to lookup every user that says a message
for the first time. This should be less intensive on the API.
This also disables partly fb713ed91bfb48c0037e489f80be85c54e69953a for now.
ChannelMembers will not be filled.
Diffstat (limited to 'bridge/slack/handlers.go')
| -rw-r--r-- | bridge/slack/handlers.go | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/bridge/slack/handlers.go b/bridge/slack/handlers.go index 5e601eaa..e013b830 100644 --- a/bridge/slack/handlers.go +++ b/bridge/slack/handlers.go @@ -75,8 +75,6 @@ func (b *Bslack) handleSlackClient(messages chan *config.Message) { // When we join a channel we update the full list of users as // well as the information for the channel that we joined as this // should now tell that we are a member of it. - b.populateUsers(false) - b.channelsMutex.Lock() b.channelsByID[ev.Channel.ID] = &ev.Channel b.channelsByName[ev.Channel.Name] = &ev.Channel @@ -202,7 +200,6 @@ func (b *Bslack) handleMessageEvent(ev *slack.MessageEvent) (*config.Message, er func (b *Bslack) handleStatusEvent(ev *slack.MessageEvent, rmsg *config.Message) bool { switch ev.SubType { case sChannelJoined, sMemberJoined: - b.populateUsers(false) // There's no further processing needed on channel events // so we return 'true'. return true |