icewind1991 / files_markdown

Nextcloud markdown editor
GNU Affero General Public License v3.0
293 stars 28 forks source link

Text editor loses connection regularly #187

Closed JordanDIG closed 10 months ago

JordanDIG commented 2 years ago

Nextcloud version: 23.0.2 Operating system and version: Raspbian

I am using Apache2 PHP version: 8.0

The issue you are facing: If I edit markdown documents via the embedded web texteditor, it loses regularly the sync:

file could not be loaded. Please check your internet connection (retry)

This occurs right after a few minutes and I guess a tip faster if you are on a edit. This error occurs on different internet connections.

Steps to replicate it:

Open Files app on your cloud webinterface Click a textfile to edit Start writing to the opened file and wait till the above poppup message appears Click rety and note, that nothing will reload. So reload the whole page

OccultWarlock commented 2 years ago

I have this same issue running on my Asustor NAS

kousu commented 1 year ago

I am seeing this too but I'm not sure what the cause is. I've put what debugging I've done so far in https://help.nextcloud.com/t/text-editor-loses-connection-regularly/96787/5.

EDIT: the cause, for me, was this cron entry I added to support renewing my letsencrypt certs:

*/5 * * * * systemctl restart nginx

:facepalm:

But I think there is still room for improvement here: richdocuments (the Collabora editor for .ods,.odt,.docx,.xlsx, etc) doesn't suffer from this. It gives errors in the Javascript console while nginx is restarting but only for a few seconds; after, it reconnects and functions fine. No one on my site ever noticed there was a problem.

But this app shows this prompt

492fc2a82e7791a6ed898b7b96949ca66fa3a70e

and stops working until the user intervenes. It's not the end of the world, especially now that I know the cause and I've throttled way back on how eager I was to reset nginx, but it could be better.

Could you keep the "Offline, changes will be saved when online" text, but drop the button; instead, just show a loading spinner that retries and retries (perhaps with exponential backoff) until it reconnects?

dbolton commented 10 months ago

Likely fixed by https://github.com/nextcloud/text/pull/5126