summaryrefslogtreecommitdiffstats
path: root/bridge
diff options
context:
space:
mode:
authorQais Patankar <qaisjp@gmail.com>2020-11-30 05:57:16 +0000
committerWim <wim@42.be>2020-12-13 23:19:48 +0100
commit0a17e2111921ad45251b72864cc713a120658849 (patch)
tree84804e5ba84e8ec6fb9c3d89465b2f317e154681 /bridge
parent52e2f926f423295dbf95463218bed6abd94d574a (diff)
downloadmatterbridge-msglm-0a17e2111921ad45251b72864cc713a120658849.tar.gz
matterbridge-msglm-0a17e2111921ad45251b72864cc713a120658849.tar.bz2
matterbridge-msglm-0a17e2111921ad45251b72864cc713a120658849.zip
Remove WebhookURL support (discord)
Diffstat (limited to 'bridge')
-rw-r--r--bridge/discord/discord.go22
-rw-r--r--bridge/discord/transmitter/transmitter.go2
2 files changed, 15 insertions, 9 deletions
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()