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

Error resolving conflict: not shown correctly when app protection is enabled #7000

Open
styx-tdo opened this issue Sep 24, 2020 · 16 comments
Open
Labels
approved bug feature: authentication Authentication or accounts related feature: transfers [WIP] Any uploading / downloading matters that aren't auto upload or sync specific hotspot: sync conflicts Sync conflict management matters hotspot: uploads tab and transfer notifications The uploads tab and upload + download related notifications stable-3.13

Comments

@styx-tdo
Copy link

Steps to reproduce

  1. create a conflict
  2. click on notification "File upload conflict"

Expected behaviour

.... conflict dialogue

Actual behaviour

get toast: Error creating conflict dialogue!

Can you reproduce this problem on https://try.nextcloud.com?

  • Please create a test demo account and see if this still happens there.
  • If yes, please open up a bug report
  • If not, please verify server setup and ask for help on forum

Environment data

Android version: 9
Device model: BV9900Pro

Stock or customized system: stock

Nextcloud app version: 3.13.1

Nextcloud server version: 18

Reverse proxy: N/A

Logs

Web server error log

N/A

@styx-tdo styx-tdo added the bug label Sep 24, 2020
@nursoda
Copy link

nursoda commented Sep 30, 2020

Same here. Conflict dialog is not shown after tapping the notification. Instead, the error "Error creating conflict dialogue!" is shown (as toast). Device OnePlus 6T, Android 10, NC Android App 3.13.1.

Yet, some version ago it WAS shown but either choice led to the conflict not being resolved and the dialog being displayed again. For that (other) issue, see #6963

@tobiasKaminsky tobiasKaminsky added the needs info Waiting for info from user(s). Issues with this label will auto-stale. label Oct 16, 2020
@tobiasKaminsky
Copy link
Member

Do you have any idea what kind of conflict file this is?

@styx-tdo
Copy link
Author

File being updated on server and client.. that this is weird for a photo folder is a question for another ticket, though.

@nursoda
Copy link

nursoda commented Oct 16, 2020

Do you have any idea what kind of conflict file this is?

In my case this happens when AutoUpload is set to COPY immediately via Wifi (don't delete locally) and I edit (crop most of the time) the image (e.g. screenshot).

@tobiasKaminsky
Copy link
Member

tobiasKaminsky commented Oct 19, 2020

Can you test #7102 ?

@stale

This comment was marked as outdated.

@stale stale bot added the stale label Nov 21, 2020
@giox069
Copy link

giox069 commented Dec 4, 2020

Same problem here, with latest version (3.14.1) available on playstore of Nextcloud client for android. Phone is Huawei P10 Lite.
It started some months ago: suddenly a lot of conflicts started to appear on the InstantUpload Camera folder for no reason.
But it's impossible to open the conflict dialog/window from the system notification:
immagine

@stale stale bot removed the stale label Dec 4, 2020
@nursoda
Copy link

nursoda commented Dec 6, 2020

Sorry, I was/am busy. Just hit the part of the error again I think. I'm on a OnePlus 6T using NC app 3.14.1 (Google Play Beta Channel) on Android 10 against NC 20.0.2. Actions to reproduce:

  • set NC app to auto-upload screenshots (if resticted to only-when-charging, then plug-in cable) and to require "device credentials" ("Geräte-Anmeldedaten") to open it
  • take a screenshot (I use two-button-press hotkey set in my phone settings)
  • Note: On my phone, newly taken screenshots are displayed on the lower right display corner with advanced features such as "exteded screenshot" (scrolls current page down to its end and captures all of it). There, I select "edit" to crop the image (in order to share the cropped in an IM).
  • edit the just-take screenshot right away (in my case crop). I see that the Nextcloud upload of the original screenshot starts in mobile phones top status bar while cropping it. Save the cropped screenshot. It's saved to the SAME filename obviously, which is intended/expected behaviour since I decided to crop it "in-place" using the above mentioned phone's feature.
  • after saving, I get an NC app conflict notification. When I tap it, the NC app opens (and asks for my fingerprint) along with a toast that the "conflict dialog" could not be "created" ("Konfiktdialog konnte nicht erstellt werden" on my german phone) – and the notification disappears
  • I open the NC app and head via menu to "Uploads" section. There, on the picture previews I see that the uncropped file was uploaded fine, and that the modified file is an unresolved conflict.
  • tapping the conflict yields the conflict dialog (that should have displayed in the first place, so maybe it's a "dialog cannot be opened via toast when NC app access protection is enabled" issue?
  • I select "new file" and "OK". The new file is uploaded and replaces the original file, so THAT part of the bug IS solved.

@github-actions

This comment was marked as outdated.

@github-actions github-actions bot added the stale label Mar 1, 2021
@nursoda
Copy link

nursoda commented Mar 2, 2021

I don't think there is anything I could do better here since the test request above is obsolete since the subsequent reports are based on a client that already had the improvements merged.

@tobiasKaminsky
Copy link
Member

Can you test again with app protection disabled?

@nursoda
Copy link

nursoda commented Mar 4, 2021

Completely without app protection, and with PIN, I always get the proper conflict dialog and file handling.
With app protection, it depends on timing and whether the app is open before tapping the conflict notification:

  • very quick edits → edited file is uploaded right away
  • medium speed → typically one conflict dialog, but I managed to get two: first was cropped+full shot, second was zero-byte+cropped shot :/

Steps to reproduce:

  • set app to app protection, close app, remove app from recent apps
  • take shot, crop it right away while it gets uploaded, conflict notification appears
  • don't tap notification yet, open NC app, unlock protection, tap notification
  • dialog is displayed properly, cropped shot is uploaded and replaces full shot

So, yes, perfect like that … except that the preview pic NC app shows doesn't match cropped but full shot.

Here's a screenshot taken after removing protection to show the zero byte case (in uploads history):

Screenshot_20210304-222046

@github-actions github-actions bot removed the stale label Mar 4, 2021
@tobiasKaminsky tobiasKaminsky added approved and removed needs info Waiting for info from user(s). Issues with this label will auto-stale. labels Mar 5, 2021
@tobiasKaminsky tobiasKaminsky changed the title Error resolving conflict - no Error resolving conflict: not shown correctly when app protection is enabled Mar 5, 2021
@detrout
Copy link

detrout commented May 6, 2021

I am experiencing this problem (or at least one very similar).

I had used auto upload to upload a bunch of images from my phone to nextcloud, but left them on my phone. Later I decided to change my nextcloud site name from something like example.org/nextcloud to nextcloud.example.org which seemed to require that I delete and recreate my account.

I recreated my account and turned instant upload on and the android client attempted to reupload all 700ish files. I unfortunately didn't pick a default of assume server side is correct so it want's to ask which version to keep.

However when I click on the "File upload conflict" notification I get "Error creating conflict dialog"

I'm on a Galaxy S9 SM-G960U1, using Android 10 build QP1A.190711.020.G960U1UES9FUC1 with SE Enforcing on.

I'm using Nextcloud from f-droid version 3.14.1

Also I think something about managing the notification window with the 700ish photos is causing a power drain as after I turned this on my battery life seems to have declined.

@detrout
Copy link

detrout commented May 6, 2021

I was finally able to find how get to the conflict resolution dialog. I had to go to uploads in the nextcloud app and in there if I tapped a photo the dialog would pop up. I canceled the pending uploads first, before changing the state, but I'm not sure if that's necessary.

@cnmicha
Copy link

cnmicha commented Jun 10, 2021

Potential duplicate of #6027

@joshtrichards joshtrichards added feature: authentication Authentication or accounts related feature: transfers [WIP] Any uploading / downloading matters that aren't auto upload or sync specific labels Oct 10, 2023
@joshtrichards joshtrichards added the hotspot: sync conflicts Sync conflict management matters label Nov 16, 2023
@strk
Copy link

strk commented Jun 18, 2024

Same problem here now.
Model: moto g¹⁰ (XT2127-2)
Android 11 build number RRBS31.Q1-3-48-22
App version: 3.29.0

I confirm @detrout workaround works: #7000 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved bug feature: authentication Authentication or accounts related feature: transfers [WIP] Any uploading / downloading matters that aren't auto upload or sync specific hotspot: sync conflicts Sync conflict management matters hotspot: uploads tab and transfer notifications The uploads tab and upload + download related notifications stable-3.13
Projects
None yet
Development

No branches or pull requests

8 participants