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

Albums: Exclude private pictures from download as zip #1836

Closed
Christian-Martensson opened this issue Dec 26, 2021 · 2 comments
Closed

Albums: Exclude private pictures from download as zip #1836

Christian-Martensson opened this issue Dec 26, 2021 · 2 comments
Assignees
Labels
bug Something isn't working released Available in the stable release

Comments

@Christian-Martensson
Copy link

What does not work as expected?

Pictures marked as private that are included in an album can be accessed by anyone if the album is shared. The picture won't show up in the web UI, but it will be included when selecting "download all pictures" (as a zip file).

How can we reproduce it?

Steps to reproduce the behavior:

  1. Create an album and add a few pictures
  2. Mark one picture as private
  3. Share the album
  4. Open the album in a private browser window
  5. Verify that the private picture is not visible in the web UI
  6. Download all pictures
  7. Verify that the private picture was included

What behavior do you expect?

Private pictures should in no way be accessible when not logged in.

Can you provide us with example files for testing or screenshots?

What version you are using?

211215-93b26f19-Linux-x86_64

Any other helpful information?

@Christian-Martensson Christian-Martensson added the bug Something isn't working label Dec 26, 2021
@lastzero lastzero self-assigned this Dec 26, 2021
@lastzero lastzero changed the title Bug: Private pictures in shared albums are included on "downloading all" pictures. Albums: Exclude private pictures in shared albums from downloads Jan 2, 2022
@lastzero lastzero changed the title Albums: Exclude private pictures in shared albums from downloads Albums: Exclude pictures in shared albums from downloads Jan 3, 2022
@lastzero lastzero changed the title Albums: Exclude pictures in shared albums from downloads Albums: Exclude private pictures from downloads Jan 3, 2022
@lastzero
Copy link
Member

lastzero commented Jan 3, 2022

Private pictures are now always excluded from downloads, as the primary use for downloads is to share them either directly or indirectly. We may introduce a more fine-grained control with multi-user support.

If I remember correctly, the album download button was added as a "quick improvement" to do someone a favor - we should have rejected it or taken more time to think it through.

@lastzero lastzero added the please-test Ready for acceptance test label Jan 3, 2022
@lastzero
Copy link
Member

lastzero commented Jan 3, 2022

We'll build a Development Preview for testing later today.

@lastzero lastzero changed the title Albums: Exclude private pictures from downloads Albums: Exclude private pictures from download as zip Jan 3, 2022
@graciousgrey graciousgrey added released Available in the stable release and removed please-test Ready for acceptance test labels Jan 7, 2022
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working released Available in the stable release
Projects
Status: Release 🌈
Development

No branches or pull requests

3 participants