Tags: nolanlum/tanya
Tags
Use new Slack methods, use go mod, migrate to conversation API (#66) Co-authored-by: Meff <[email protected]> Co-authored-by: Nolan Lum <[email protected]>
Fix potential usage of uninitialized reverse mappings. If we receive a `user_change` event before `regenerateReverseMappings` is called at the end of `bootstrapMappings`, we could potentially attempt to modify these `nil` maps. Since I can't remember any good reasons why these shouldn't be initialized to empty maps (and all the code seems to assume these will be non-nil anyway), initialize these to empty maps on SlackClient creation.
PreviousNext