search

boredazfcuk / docker-icloudpd

An Alpine Linux container for the iCloud Photos Downloader command line utility
1.57k stars 149 forks source link

Docker Container keeps stopping after downloading files #578

Closed Palito93 closed 3 weeks ago

Palito93 commented 3 weeks ago

Hello,

I have 2 docker containers which have been running without any issues for weeks now.

But lately both docker containers keep shutting down after downloading pictures. Here's a screenshot of the debug:

Screenshot 2024-06-10 071829

Leckbesteck commented 3 weeks ago

I have the exact same problem.

dunezilla commented 3 weeks ago

Same here. Same debug.

boredazfcuk commented 3 weeks ago

What notification method do you have configured, if any?

Palito93 commented 3 weeks ago

I have no notification method enabled (and never had if that matters)

On Mon, 10 Jun 2024 at 19:03, boredazfcuk @.***> wrote:

What notification method do you have configured, if any?

— Reply to this email directly, view it on GitHub https://github.com/boredazfcuk/docker-icloudpd/issues/578#issuecomment-2158882178, or unsubscribe https://github.com/notifications/unsubscribe-auth/BJCCXRUGWGOQXKVI4OWQJO3ZGXL7ZAVCNFSM6AAAAABJBSEYQ6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJYHA4DEMJXHA . You are receiving this because you authored the thread.Message ID: @.***>

boredazfcuk commented 3 weeks ago

Yeah, that's what I was thinking because it usually states the notification type in the log entry, but yours appears to be blank. The logic should check to make sure the notification type is set before sending one, so seems the logic is flawed, so a genuine bug. I will check shortly

Palito93 commented 3 weeks ago

Thank you very much!

This started happening maybe a week ago if i had to guess a time.

On Mon, 10 Jun 2024 at 19:34, boredazfcuk @.***> wrote:

Yeah, that's what I was thinking because it usually states the notification type in the log entry, but yours appears to be blank. The logic should check to make sure the notification type is set before sending one, so seems the logic is flawed, so a genuine bug. I will check shortly

— Reply to this email directly, view it on GitHub https://github.com/boredazfcuk/docker-icloudpd/issues/578#issuecomment-2158933874, or unsubscribe https://github.com/notifications/unsubscribe-auth/BJCCXRQYHGFS4B7Q4RFGO6DZGXPUDAVCNFSM6AAAAABJBSEYQ6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJYHEZTGOBXGQ . You are receiving this because you authored the thread.Message ID: @.***>

dunezilla commented 3 weeks ago

Same. No notificationsThanks,Lee ThomasOn Jun 10, 2024, at 2:07 PM, Palito93 @.***> wrote: Thank you very much!

This started happening maybe a week ago if i had to guess a time.

On Mon, 10 Jun 2024 at 19:34, boredazfcuk @.***> wrote:

Yeah, that's what I was thinking because it usually states the

notification type in the log entry, but yours appears to be blank. The

logic should check to make sure the notification type is set before sending

one, so seems the logic is flawed, so a genuine bug. I will check shortly

—

Reply to this email directly, view it on GitHub

https://github.com/boredazfcuk/docker-icloudpd/issues/578#issuecomment-2158933874,

or unsubscribe

https://github.com/notifications/unsubscribe-auth/BJCCXRQYHGFS4B7Q4RFGO6DZGXPUDAVCNFSM6AAAAABJBSEYQ6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJYHEZTGOBXGQ

.

You are receiving this because you authored the thread.Message ID:

@.***>

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: @.***>

boredazfcuk commented 3 weeks ago

This was the result of a PR I merged which added support for SMTP notifications. Previously, it would only match if notifications were configured. After the PR was merged, it would match notifications that were not SMTP, so including unconfigured notifications.

I've built and pushed a new container to Dockerhub