Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Logging seems broken #697

Closed
boredazfcuk opened this issue Sep 30, 2023 · 3 comments · Fixed by #698 or #699
Closed

Logging seems broken #697

boredazfcuk opened this issue Sep 30, 2023 · 3 comments · Fixed by #698 or #699
Labels

Comments

@boredazfcuk
Copy link
Contributor

Overview

In my container, when I run icloudpd, I output the logs to stdout and stderr for further parsing. This worked in previous versions, but in release 16.2, logging seems broken.

Steps to Reproduce

  1. ./icloudpd --directory /icloud --cookie-directory /config --domain com --username [email protected] --auto-delete --folder-structure {:%Y/%m} 2>/tmp/icloudpd/icloudpd_download_error | tee /tmp/icloudpd/icloudpd_sync.log

Expected Behavior

stdout is passed to tee which pushes the logs to stdout and also to /tmp/icloudpd/icloudpd_sync.log

Actual Behavior

Nothing goes to tee, so no logging to screen. Everything ends up in stderr.

@AndreyNikiforov
Copy link
Collaborator

What version did it work? Can you highlight differences in behavior in two versions, pls? I tried running the following in 1.6.1 and 1.15.1 and then seem to produce same ./d:

icloudpd -u [email protected] -d ./ --dry-run --recent 10 --no-progress-bar 2> ./d

@AndreyNikiforov
Copy link
Collaborator

Maybe related to #664 when we switched to using standard python logging.

@boredazfcuk
Copy link
Contributor Author

Just confirming that I've now tested v1.16.2 and it's working as expected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants