fraschetti / Octoslack

OctoPrint plugin for Slack, Mattermost, Pushbullet, Pushover, Rocket.Chat, Discord, Riot/Matrix, & Microsoft Teams
MIT License
74 stars 34 forks source link

Not uploading timelapses to Slack #77

Closed calonmerc closed 4 years ago

calonmerc commented 4 years ago

For a while now, all of my Octoprint images have been unable to upload timelapses to Slack. The timelapse rendering completes, I get a notification about it, but there's no URL for it. I've double checked my settings and it is supposed to include the URL.

The big problem is that the local timelapses get removed.

fraschetti commented 4 years ago

Hi @calonmerc

Do you mind providing some rough details about your environment? -What device/OS are you running? -How are you connecting to Slack (Webhook or API token)? -Do you mind following the steps below to generate a debug log for analysis?

  1. Enable DEBUG logging
  2. Reproduce the issue
  3. Send Octoprint.log

Instructions to enable DEBUG logging for Octoslack

For OctoPi releases, the log file will be located in: /home/pi/.octoprint/logs/octoprint.log

calonmerc commented 4 years ago

-What device/OS are you running? --Variety of Pi 3B and 3B+, all running Raspbian from the OctoPi image. -How are you connecting to Slack (Webhook or API token)? --Using the API token on all of them. (The same token, if it matters) -Do you mind following the steps below to generate a debug log for analysis? -- I will do these after this print finishes.

calonmerc commented 4 years ago

octoprint.log

calonmerc commented 4 years ago

Probably a year and a half ago I was using imgur or something, but I've been using Slack, with the timelapse uploads, for consistently over a year. I stopped printing much in November, started again at the end of February. That's when I noticed the timelapses weren't showing anymore.

fraschetti commented 4 years ago

Hi @calonmerc

I likely broke the Slack asset upload as part of recent release. There were a number of scenarios where the code might try to use an old upload method if the user was swapping around configurations. The logic to ensure the setup was correct for Slack uploads had a typo. I've submitted a fix which will be included in my next release.

calonmerc commented 4 years ago

Awesome! Any ETA on the release? Or any help running dev code? (I don't see a development branch to pull)

fraschetti commented 4 years ago

Hi @calonmerc Shouldn't be long before I push a new release - likely within the next week or so. As far as the dev branch is concerned, this project has been fairly simple thus far and I haven't needed to break out a dev branch from the main branch. If you want to test the change, you can simply pull the main branch and go from there. If you haven't seen this page before, the steps to register a development plugin can be found here - https://docs.octoprint.org/en/master/plugins/gettingstarted.html

brad-colbert commented 4 years ago

May I "pile on" this thread?

I'm not seeing either snapshots or timelapses posted to Slack. I'm assuming that Octoslack is able to post both directly? If not I apologize for not understanding.

Cheers!

fraschetti commented 4 years ago

Hi @calonmerc I've pushed new Octoslack 2.0.0 release which should address your Slack upload issue. Give it a try - if you still experience issues, feel free to reopen this ticket.

fraschetti commented 4 years ago

Hi @brad-colbert

Assuming you're config is also setup to use Slack to host the snapshots/timelapses, the new release should fix your issue as well. It you were using a different hosting method or are experiencing a different issue, feel free to open another ticket that we can use to investigate your issue further.

brad-colbert commented 4 years ago

Yep, testing now. Will know in roughly 3.5 hours! Thank you for all of your hard work! Fantastic plugin!

Kind regards, Brad

On Sun, May 24, 2020 at 9:03 AM Chris Fraschetti notifications@github.com wrote:

Hi @brad-colbert https://github.com/brad-colbert

Assuming you're config is also setup to use Slack to host the snapshots/timelapses, the new release should fix your issue as well. It you were using a different hosting method or are experiencing a different issue, feel free to open another ticket that we can use to investigate your issue further.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/fraschetti/Octoslack/issues/77#issuecomment-633252554, or unsubscribe https://github.com/notifications/unsubscribe-auth/AK35VRP23TPX2JDWWP3T55DRTFAMJANCNFSM4LWZUDOQ .