-
Notifications
You must be signed in to change notification settings - Fork 159
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
Limit the number of concurrent share invite requests #10227
Comments
Fixed via #10230. Now needs to target the other branches as well. |
Brought the config to ocis master via owncloud/ocis#8056 - also needs to be backported to ocis |
This is still open, any actions we should take from here because p1? |
Depends. Backport to stable-7.1 would still be needed to close this issue if (and only if) the bycs team can handle further patches of the ocis-4.0 minor version. We didn't receive a definitive answer or it didn't reach me. @dragotin could you shed some light on this? For ocis-5.0 the work is done. I'd like to close here because this is not a bug in the general product. |
We can close here IMHO |
Describe the bug
Apparently ocis has issues with concurrent share creations on the same file. To mitigate this quickly, we want to set a (configurable) limit for the number of
share receivers in one inviteconcurrent requests.Steps to reproduce
Expected behavior
Short request/response duration and no failing invites.
Actual behavior
Slow consecutive requests, sometimes failing, if the deployment/instance is slow in general.
Solution
We want to set a configurable number of concurrent requests here
web/packages/web-app-files/src/components/SideBar/Shares/Collaborators/InviteCollaborator/InviteCollaboratorForm.vue
Line 479 in fa19d32
Would like to propose the config.json structure as follows:
Affected versions
Needs to be fixed in
stable-7.1
ANDstable-8.0
so that we can provide a fix forocis-4.0
and the upcomingocis-5.0
.The text was updated successfully, but these errors were encountered: