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

regression in being able to resend stuck messages #2523

Closed
ara4n opened this issue Oct 31, 2016 · 5 comments
Closed

regression in being able to resend stuck messages #2523

ara4n opened this issue Oct 31, 2016 · 5 comments
Labels
P1 S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect X-Regression

Comments

@ara4n
Copy link
Member

ara4n commented Oct 31, 2016

i don't have full repro steps, but i am fairly sure that sending messages has got less reliable when on bad connections. for instance, when the server stops responding for a few minutes after an IO explosion, i have reports of people getting stuck with loads of unsent messages, but no way of triggering a resend

@ara4n ara4n added T-Defect P1 S-Major Severely degrades major functionality or product features, with no satisfactory workaround X-Regression type:composer labels Nov 6, 2016
@ara4n
Copy link
Member Author

ara4n commented Jan 9, 2017

I just had this after today's IO meltdowns. Messages were stacking up despite the server otherwise being back, but no UI to resend them; they were stuck in grey. Logs attached.
bug-2523.txt

@ara4n
Copy link
Member Author

ara4n commented Jan 16, 2017

...and just had this again after DC1 firewall meltdown.

@ara4n
Copy link
Member Author

ara4n commented Mar 23, 2017

Just had this again. Errors are:

bundle.js:20633 Persisting sync data up to  s118136230_177531992_510334_18178459_186038_63906_600364_394212
2017-03-23 16:08:57.418 bundle.js:49086 Uncaught Error: ETIMEDOUT
    at too_late (bundle.js:71681)
2017-03-23 16:09:03.870 bundle.js:63328 Uncaught Error: updatePendingEventStatus called on an event which is not a local echo.
    at Room.updatePendingEvent (bundle.js:63328)
    at _updatePendingEventStatus (bundle.js:54126)
    at MatrixScheduler._procFn (bundle.js:53297)
    at _processQueue (bundle.js:70190)
    at bundle.js:70212

@ara4n
Copy link
Member Author

ara4n commented Mar 23, 2017

it looks like a single request which gets stuck and fails with some error may wedge all subsequent ones.

@lampholder lampholder modified the milestones: RW002, RW003 - candidates, RW002 - candidates Apr 3, 2017
@lampholder lampholder modified the milestones: RW003, RW004 - candidates, RW005 - candidates May 18, 2017
@kittykat
Copy link
Contributor

I'm going to close this issue for now as I was testing this a few weeks ago and re-sending messages worked fine (was testing for notifications, which are also fixed now). If it is still an issue for anyone, please reopen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P1 S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect X-Regression
Projects
None yet
Development

No branches or pull requests

4 participants