Fix for slackdiff.

[Punicaa]

Pre-Request Checklist

• [ ] Passes rubocop code analysis (try rubocop --auto-correct)
• [ ] Tests added or adapted (try rake test)
• [ ] Changes are reflected in the documentation
• [X] User-visible changes appended to CHANGELOG.md

Description

• Updated slackdiff.rb to use slack_ruby_client instead of slack-api.
• Updated default timeout from 20 to 60.

Fixes users experiencing the following error:

ERROR -- : Hook slack (#SlackDiff:0x00005572c9621e20) failed (#<NameError: uninitialized constant Slack::Client>) for event :post_store #2697

[robertcheramy]

I miss the update of the default timeout in the diff. Can you check that? Can you also merge master into your branch (or rebase it, as you which), and modify Changelog so that the changes are under the unreleased Version?

[Punicaa]

Thanks for the comment. Changes from Master pulled and made changes to Changelog and actually changed the timeout.

[robertcheramy]

Something whent terribly wrong here, as many commits where integrated in this PR. Mabe a git rebase would bring some clarity in the PR?

[robertcheramy]

Why should the default timeout be changed? If you don't like it, you can change it in your configuration.

[robertcheramy]

Something went terribly wrong here, as many commits where integrated in this PR. Mabe a git rebase would bring some clarity in the PR?

I've rebased your branch. Be sure to update your branch before submitting more commits. You may need a `git reset --hard.

[Punicaa]

Thanks for fixing the branch. The timeout of 20 seconds is too few for devices with more complicated configurations, like firewalls, or some switches. It's also on the low-end when devices need to be pulled over higher latency. When setting up oxidized in our environment it was "broken" out of the box because of this. I figured this could help some first time users, but I can also leave it out and we'll just go with the slack fix.

[robertcheramy]

You can change the timeout in your configuration file:

timeout: 20

I'll revert the timeout change an merge into master.