From 0a17e2111921ad45251b72864cc713a120658849 Mon Sep 17 00:00:00 2001 From: Qais Patankar Date: Mon, 30 Nov 2020 05:57:16 +0000 Subject: Remove WebhookURL support (discord) --- bridge/discord/discord.go | 22 ++++++++++++++-------- bridge/discord/transmitter/transmitter.go | 2 +- 2 files changed, 15 insertions(+), 9 deletions(-) (limited to 'bridge/discord') diff --git a/bridge/discord/discord.go b/bridge/discord/discord.go index 1a3af929..78c579ae 100644 --- a/bridge/discord/discord.go +++ b/bridge/discord/discord.go @@ -43,14 +43,7 @@ func New(cfg *bridge.Config) bridge.Bridger { b.nickMemberMap = make(map[string]*discordgo.Member) b.channelInfoMap = make(map[string]*config.ChannelInfo) - // If WebhookURL is set to anything, we assume preference for autoWebhooks - // - // Legacy note: WebhookURL used to have an actual webhook URL that we would edit, - // but we stopped doing that due to Discord making rate limits more aggressive. - // - // We're keeping the same setting for now, and we will late deprecate this setting - // in favour of a new setting, something like "AutoWebhooks=true" - b.useAutoWebhooks = b.GetString("WebhookURL") != "" + b.useAutoWebhooks = b.GetBool("AutoWebhooks") if b.useAutoWebhooks { b.Log.Debug("Using automatic webhooks") } @@ -147,6 +140,19 @@ func (b *Bdiscord) Connect() error { return err } + // Legacy note: WebhookURL used to have an actual webhook URL that we would edit, + // but we stopped doing that due to Discord making rate limits more aggressive. + // + // Even older: the same WebhookURL used to be used by every channel, which is usually unexpected. + // This is no longer possible. + if b.GetString("WebhookURL") != "" { + message := "The global WebhookURL setting has been removed. " + message += "You can get similar \"webhook editing\" behaviour by replacing this line with `AutoWebhooks=true`. " + message += "If you rely on the old-OLD (non-editing) behaviour, can move the WebhookURL to specific channel sections." + b.Log.Errorln(message) + return fmt.Errorf("use of removed WebhookURL setting") + } + // Initialise webhook management b.transmitter = transmitter.New(b.c, b.guildID, "matterbridge", b.useAutoWebhooks) b.transmitter.Log = b.Log diff --git a/bridge/discord/transmitter/transmitter.go b/bridge/discord/transmitter/transmitter.go index 41ed055b..e6d52a98 100644 --- a/bridge/discord/transmitter/transmitter.go +++ b/bridge/discord/transmitter/transmitter.go @@ -111,7 +111,7 @@ func (t *Transmitter) HasWebhook(id string) bool { } // AddWebhook allows you to register a channel's webhook with the transmitter. -func (t *Transmitter) AddWebhook(channelID string, webhook *discordgo.Webhook) (replaced bool) { +func (t *Transmitter) AddWebhook(channelID string, webhook *discordgo.Webhook) bool { t.Log.Debugf("Manually added webhook %#v to channel %#v", webhook.ID, channelID) t.mutex.Lock() defer t.mutex.Unlock() -- cgit v1.2.3