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

Improve handling of invalid federated (and regular) shares #7513

Open
enoch85 opened this issue Dec 14, 2017 · 2 comments
Open

Improve handling of invalid federated (and regular) shares #7513

enoch85 opened this issue Dec 14, 2017 · 2 comments
Labels

Comments

@enoch85
Copy link
Member

enoch85 commented Dec 14, 2017

As @rullzer mentioned here we should improve the way closed servers with federated shares are handled. The issue in #7001 was fixed with caching and it works, but could be done "in a smarter" way.

I suggest that:

  • When a URL for the server of a federated share has been inactive (aka 404) for 30 days, then remove the federated share automatically with cron.php. A solution could be to ping the domain of the receiver once a day for example and after 30 days of "reached time out" remove the share to that specific receiver. Also, when a share is expired - notify the admin when that happens.
  • The federated shares are performance checked, so when a federated share performs bad, inform the user with a notification. (As a stage 2 step)
  • List all federated share users in the Files app, in addition to "Shared by link". So for example "Federated shares" and list all the users.

I'm sure you guys have even better ideas.

@MorrisJobke
Copy link
Member

cc @nextcloud/sharing

@MorrisJobke
Copy link
Member

Most likely nothing for 14 -> moving to 15.

@nextcloud-bot nextcloud-bot removed the stale Ticket or PR with no recent activity label Jul 24, 2018
@MorrisJobke MorrisJobke modified the milestones: Nextcloud 15, Nextcloud 16 Nov 5, 2018
@rullzer rullzer modified the milestones: Nextcloud 17.0.4, Nextcloud 17.0.5 Mar 11, 2020
@rullzer rullzer modified the milestones: Nextcloud 17.0.5, Nextcloud 17.0.6 Mar 23, 2020
@rullzer rullzer removed this from the Nextcloud 17.0.6 milestone Apr 22, 2020
@rullzer rullzer added this to the Nextcloud 17.0.7 milestone Apr 22, 2020
@skjnldsv skjnldsv removed this from the Nextcloud 17.0.7 milestone Apr 27, 2020
@skjnldsv skjnldsv added the 0. Needs triage Pending check for reproducibility or if it fits our roadmap label Apr 27, 2020
@skjnldsv skjnldsv added 1. to develop Accepted and waiting to be taken care of and removed 0. Needs triage Pending check for reproducibility or if it fits our roadmap labels Aug 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants