-
-
Notifications
You must be signed in to change notification settings - Fork 4.7k
Labels
0. Needs triagePending check for reproducibility or if it fits our roadmapPending check for reproducibility or if it fits our roadmap31-feedbackbug
Description
⚠️ This issue respects the following points: ⚠️
- This is a bug, not a question or a configuration/webserver/proxy issue.
- This issue is not already reported on Github OR Nextcloud Community Forum (I've searched it).
- Nextcloud Server is up to date. See Maintenance and Release Schedule for supported versions.
- I agree to follow Nextcloud's Code of Conduct.
Bug description
Typical usage creates 429 errors in Talk app, e.g. after sending 20 chat messages with attachments, no further such messages can be sent.
The bug in the app is merely the misleading "You are not allowed to share files" message displayed. The root of the problem appears to be in core - the default rate limits being too low, by about a factor of 10.
See #50905 (backports: v30 #52205 v31 #52206)
Steps to reproduce
- Enter Room in Nextcloud Talk
- Send a message with 20 image attachments
- Send another attachment within 10 minutes
- Observe (unclear) error
Expected behavior
Sending another message should work. Typical human interaction via the web interface should not be hampered by mitigations against misuse or broken apps.
Nextcloud Server version
- actually reproduced on: 31.0.13
- earliest v31 that has blamed patch: v31.0.4
- earliest v30 that has blamed patch: v30.0.10
What user-backends are you using?
- Default user-backend (database)
- LDAP/ Active Directory
- SSO - SAML
- Other
Additional info
Note that in the example of the Talk app occurs after the expensive PUT has been processed.
Its only later when cheaply POSTing the image to the Chatroom that the rate limit kicks in.
Metadata
Metadata
Assignees
Labels
0. Needs triagePending check for reproducibility or if it fits our roadmapPending check for reproducibility or if it fits our roadmap31-feedbackbug