Skip to content

Commit

Permalink
nit
Browse files Browse the repository at this point in the history
  • Loading branch information
scarlettperry committed Apr 19, 2021
1 parent 3e6dc73 commit 2613955
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions backend/module/bot/slackbot/slackbot.go
Original file line number Diff line number Diff line change
Expand Up @@ -92,8 +92,8 @@ func (m *mod) Event(ctx context.Context, req *slackbotv1.EventRequest) (*slackbo
return &slackbotv1.EventResponse{}, nil
}

// TODO: (sperry) Slack currently still supports this verification type but they plan to deprecate this inf the future &
// we want to check the signing secret instead so this is a short-term solution (need to change API design first)
// TODO: (sperry) Slack plans to deprecate this form of verification in the future. we'll refactor to check the
// signing secret instead (need to change API design)
func verifySlackRequest(token, requestToken string) error {
t := slackevents.TokenComparator{VerificationToken: token}
ok := t.Verify(requestToken)
Expand Down

0 comments on commit 2613955

Please sign in to comment.