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

Non-expiring shared links #260

Closed
Wdavery opened this issue Apr 15, 2021 · 3 comments
Closed

Non-expiring shared links #260

Wdavery opened this issue Apr 15, 2021 · 3 comments
Milestone

Comments

@Wdavery
Copy link
Contributor

Wdavery commented Apr 15, 2021

As a simple alternative to having a permission based user system (like #188, which would be great) could we implement non-expiring shared links? Currently the share doesn't get created if you set a 0 in the Valid: selection.

This would enable some password protection to shared folders that are restrictive at least to a single parent folder.

                                  Gallery
          /               |                     |                \
         /                |                     |                  \
   Fam.A.Pics         Fam.B.Pics            Personal            Wedding A+B
   (Fam. A)            (Fam. B)            (not shared)          (Fam A & B)

In this example the (shared links) would be permanent and both Fam. A and Fam. B would be given two links each with passwords without having to worry about expiration date.

Not a perfect system but it would allow a bit more flexibility beyond needing to host separate instances for, in this case, 3 libraries.

@gklebus
Copy link
Contributor

gklebus commented Apr 15, 2021

As a quick workaround, you could create links with a ridiculously long expiration date (e.g., 1200 months = 100 years). Practically permanent.

@Wdavery
Copy link
Contributor Author

Wdavery commented Apr 15, 2021

Definitely functionally equivalent, and what I'll be doing for now.
Just seems like wasted logic for determining whether a link is expired when you never intend it to be.

bpatrik added a commit that referenced this issue Apr 19, 2021
It works by setting the expirity to -1 that setts the expire dat to 01/01/9999. Now the UI does not show expiration if it is longer than 10 years
@bpatrik
Copy link
Owner

bpatrik commented Apr 19, 2021

I added this. (Explanation in the commit comment). Basically you can choose now: Validity: Forever that sets the validity date to 01/01/9999. And I'm not showing "time" left if that is longer than 10 years.

I hope by 01/01/9999, it wont be an issue :)

@bpatrik bpatrik added this to the v1.9 milestone Apr 19, 2021
@Wdavery Wdavery closed this as completed Apr 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants