syncthing-android icon indicating copy to clipboard operation
syncthing-android copied to clipboard

Forced run - split head, make clear "UI" vs. "daemon"

Open licaon-kter opened this issue 1 year ago • 14 comments

...forced start is not doing what's expected, the UI messages are confused/ing.

1.26.0.2 F-Droid

The Status UI page says: "Syncthing is not running" and lower "reason: Syncthing is running because you forced it to start regardless..."

No sync happens though.

188a1c25_C5ujqPtHSyqY6G8Cp3qx7g

To fix it I need to exit it, then restart it.

licaon-kter avatar Nov 20 '23 05:11 licaon-kter

Attached picture to OP

licaon-kter avatar Nov 20 '23 15:11 licaon-kter

@licaon-kter

Is the screenshot above the result of the native crashing? This was reported since v1.26.x releases and upstream has merged a PR to fix this, hopefully to be released soon. I agree the messages are weird, but regarding that the native crashing should not be daily matter, I suggest it's okay.

The message "Syncthing is not running" is sent by the native executor which also detects the crash. The message in the "reason" section is just to sum up the set run conditions. Confusing yes, but at least we now know why it is ;-).

Maybe we could hide the reasons if a crash was detected.

Catfriend1 avatar Nov 21 '23 21:11 Catfriend1

I don't see visible Android crashes, so if it's the daemon under the hood I'd need to check logcat.

licaon-kter avatar Nov 21 '23 21:11 licaon-kter

oh so you have in app logging, will capture one of those when it happens again (of course I tried now and it didn't crash)

fyi, when testing before, I needed to press restart or kill 2-3 times until it actually was able to start :(

licaon-kter avatar Nov 22 '23 10:11 licaon-kter

@licaon-kter you could also try the latest preview build from the github release page . It contains upstream's fix.

Catfriend1 avatar Nov 22 '23 22:11 Catfriend1

After last update this also happens to me too. Syncthing-fork not working anymore for me. Even if device on AC.

EuPhobos avatar Nov 23 '23 10:11 EuPhobos

@EuPhobos try preview?

Catfriend1 avatar Nov 23 '23 10:11 Catfriend1

@Catfriend1 yep, trying this file com.github.catfriend1.syncthingandroid_github_v1.26.1.0_2ee4de2c.apk, same problem. Before i has f-droid version running fine, today i update app and faced with a this problem. I export settings, remove f-droid verion, install github version, import settings - same problem. I can provide logs, if you need.

EuPhobos avatar Nov 23 '23 10:11 EuPhobos

@EuPhobos

There is a newer release 1.27.0.0

Catfriend1 avatar Nov 23 '23 10:11 Catfriend1

@Catfriend1 Oh, my inattention. 1.27 work fine now. Thank you very much! You're best

EuPhobos avatar Nov 23 '23 10:11 EuPhobos

Thanks for reporting back

Catfriend1 avatar Nov 23 '23 11:11 Catfriend1

I think this is okay for me, let the head split when the native crashes. If this ever happens, the user is doomed and other parts of the app may also be inaccurate or non-working.

Catfriend1 avatar Nov 23 '23 11:11 Catfriend1

For correctness: it gets a verified bug label 😊

Catfriend1 avatar Nov 23 '23 11:11 Catfriend1

let the head split when the native crashes.

No problem with that, maybe rework the strings a bit to separate "UI" vs "daemon" so they don't both identify themselves as "Syncthing"?

licaon-kter avatar Nov 23 '23 11:11 licaon-kter