fix: workspace token mentions

.env.example

@@ -1,2 +1,2 @@
 SLACK_TOKEN=
-SLACK_WORKSPACE_TOKEN=
+SLACK_SIGNING_SECRET=

remote/slack/helper.go

@@ -458,17 +458,17 @@ func populateBotUsers(slackUsers []slack.User, bot *models.Bot) {
 // populateUserGroups populates slack user groups
 func populateUserGroups(bot *models.Bot) {
 	userGroups := make(map[string]string)
-	wsAPI := slack.New(bot.SlackToken)
-	ugroups, err := wsAPI.GetUserGroups()
+	api := slack.New(bot.SlackToken)
+
+	ugroups, err := api.GetUserGroups()
 	if err != nil {
-		bot.Log.Debugf("Unable to retrieve usergroups: %s", err.Error())
-		bot.Log.Debug("Please double check your Slack Workspace token")
+		bot.Log.Debugf("unable to retrieve usergroups: %s - confirm you have usergroups:read permission set", err.Error())
 	}
+
 	for _, usergroup := range ugroups {
 		userGroups[usergroup.Handle] = usergroup.ID
 	}
-	// we don't need API anymore
-	wsAPI = nil
+
 	bot.UserGroups = userGroups
 }
 

utils/access_check_test.go

@@ -37,7 +37,6 @@ func TestCanTrigger(t *testing.T) {
 		{"User not in ignore list", args{"jane.doe", "F123456", models.Rule{IgnoreUsers: []string{"john.doe", "jack.jill"}}, testBot}, true},
 		{"User is allowed but ignored", args{"jane.doe", "F123456", models.Rule{AllowUsers: []string{"jane.doe"}, IgnoreUsers: []string{"jane.doe", "jack.jill"}}, testBot}, false},
 		{"User is not allowed and ignored", args{"john.doe", "F123456", models.Rule{AllowUsers: []string{"jane.doe"}, IgnoreUsers: []string{"john.doe", "jack.jill"}}, testBot}, false},
-		{"Group - Workspace Token not supplied", args{"jane.doe", "F123456", models.Rule{AllowUserGroups: []string{"admins"}}, testBot}, false},
 		{"Group - Discord - Not supported", args{"jane.doe", "F123456", models.Rule{AllowUserGroups: []string{"admins"}}, discordBot}, false},
 		{"Group - Chat network not supported", args{"jane.doe", "F123456", models.Rule{AllowUserGroups: []string{"admins"}}, strangeBot}, false},
 		// TODO: figure out how to test this below: