From b1f403165d6df6af1010ab49305a5968a8a8c4b6 Mon Sep 17 00:00:00 2001 From: Wim Date: Sun, 24 Oct 2021 22:17:46 +0200 Subject: [PATCH] Fix panic in msteams. Fixes #1588 (#1622) --- bridge/msteams/msteams.go | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/bridge/msteams/msteams.go b/bridge/msteams/msteams.go index 87a15a7a..27d7bee0 100644 --- a/bridge/msteams/msteams.go +++ b/bridge/msteams/msteams.go @@ -19,8 +19,10 @@ import ( "golang.org/x/oauth2" ) -var defaultScopes = []string{"openid", "profile", "offline_access", "Group.Read.All", "Group.ReadWrite.All"} -var attachRE = regexp.MustCompile(``) +var ( + defaultScopes = []string{"openid", "profile", "offline_access", "Group.Read.All", "Group.ReadWrite.All"} + attachRE = regexp.MustCompile(``) +) type Bmsteams struct { gc *msgraph.GraphServiceRequestBuilder @@ -50,7 +52,7 @@ func (b *Bmsteams) Connect() error { b.Log.Errorf("Couldn't save sessionfile in %s: %s", tokenCachePath, err) } // make file readable only for matterbridge user - err = os.Chmod(tokenCachePath, 0600) + err = os.Chmod(tokenCachePath, 0o600) if err != nil { b.Log.Errorf("Couldn't change permissions for %s: %s", tokenCachePath, err) } @@ -168,7 +170,7 @@ func (b *Bmsteams) poll(channelName string) error { } // skip non-user message for now. - if msg.From.User == nil { + if msg.From == nil || msg.From.User == nil { continue }