mattermost-plugin-google-calendar icon indicating copy to clipboard operation
mattermost-plugin-google-calendar copied to clipboard

Improve error handling in `completeCalendar` to avoid panic

Open mickmister opened this issue 4 years ago • 1 comments

PR https://github.com/mattermost/mattermost-plugin-google-calendar/pull/7 was created to be clear about configuring the plugin properly, as a remedy to avoid a panic in the plugin. The plugin should instead be resilient to this and handle the case of this error. See this comment for details https://github.com/mattermost/mattermost-plugin-google-calendar/pull/7#issuecomment-618610885

There was an index out of range error occurring here: https://github.com/mattermost/mattermost-plugin-google-calendar/blob/f85d05be8bdfdb8d7a9b52353a19a6a42f9f1b91/server/api.go#L83

mickmister avatar Apr 23 '20 19:04 mickmister