Skip to content

fix(auto-upload): skip check - #17401

Open
alperozturk96 wants to merge 1 commit into
masterfrom
fix/auto-upload-skip-check
Open

fix(auto-upload): skip check#17401
alperozturk96 wants to merge 1 commit into
masterfrom
fix/auto-upload-skip-check

Conversation

@alperozturk96

Copy link
Copy Markdown
Collaborator

Issue

Worker might mark entity as handled but this is not enough worker may get killed and never reached to the actual upload thus oc upload records should be checked as well.

Fixes

#15573

@github-actions

Copy link
Copy Markdown

APK file: https://github.com/nextcloud/android/actions/runs/30632262721/artifacts/8794164956
To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.
qrcode (please click on link to get QR code displayed)

@github-actions

Copy link
Copy Markdown

@DricomDragon

Copy link
Copy Markdown

I installed the QA app, logged in, gave permission to manage all files, (disabled auto-upload on stock APK), configured the auto upload of video and photograph in camera (after having renamed the previous folder to Camera1 to prevent mass upload), Wi-Fi is enabled, I plugged my phone, took a photograph and video : no effect.

Unplug from charger, plug to my computer, capture logs, go to uploads, pause / play, go to manage auto uploads : I can see my picture and video in the auto-upload configurator, but nothing automatically uploaded yet. Maybe unrelated to your patch, though.

Here are the logs :
qa-logs-redacted.txt

I will use this QA version from now on, I will notice you about uploads or skips when noticed.

Thanks for your amazing work, have a good day !

@alperozturk96

Copy link
Copy Markdown
Collaborator Author

/backport to stable-34.1.x

Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown

Codacy

SpotBugs

CategoryBaseNew
Bad practice3232
Correctness6464
Dodgy code215215
Experimental11
Malicious code vulnerability22
Multithreaded correctness2525
Performance4040
Security1919
Total398398

@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown

APK file: https://github.com/nextcloud/android/actions/runs/31084696925/artifacts/8961049995
To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.
qrcode (please click on link to get QR code displayed)

@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown

blue-Light-Screenshot test failed, but no output was generated. Maybe a preliminary stage failed.

@robdupas-svg

Copy link
Copy Markdown

Setup:

  • Uninstalled F-Droid stable, installed QA build fresh
  • No pre-existing Camera rule (no photos yet taken); a default, disabled ScreenRecorder rule was auto-created from recordings made the previous day
  • Created a manual Camera rule pointing at DCIM/Camera

Test sequence and results:

  1. Un-installed F-Droid app
  2. Installed the QA version
    • Once opened, I was prompted to
    • Allow notifications
    • Allow access to all files
  3. Started to create the Camera rule
    • recommendation to turn on background autostart (did this)
    • No default Camera rule (I had taken no photos yet)
    • Created it
    • Took longer than usual to discover the folders on my server
    • A default ScreenRecorder rule was already setup
    • I have taken some recordings yesterday
    • I left it disabled
  4. Closed Nextcloud
  5. Took a photo
    • Waited 60 seconds and it still did not automatically upload to the server
  6. Opened Nextcloud app
    • Nothing automatically uploaded
    • 2 Camera rules now exist.
    • The one I created shows 5 photos (4 of which I deleted yesterday)
    • The Nextcloud generated Camera rule
  7. Deleted the manually created rule
  8. Configured and enabled the Nextcloud Camera rule
    • The photo uploaded automatically after I did this
  9. Closed Nextcloud
  10. Took another photo
  11. Waited another minute
  12. Nothing uploaded
  13. Checked the app state
    • Apps\Manage apps\Nextcloud
    • Seems to be running as “Force stop” is not greyed out
  14. Opened Nextcloud
    • Both photos I took appear as icons in the rule
    • Toggling upload had no effect
  15. Created a new Camera rule
    • Photo uploaded automatically
  16. Closed Nextcloud
  17. Took another photo with both rules active
  18. Waited another minute
  19. No automatic upload so I opened Nextcloud
    • The custom rule still shows several photos I deleted yesterday + newly created photos
  20. Still no automatic upload so I deleted the custom rule
  21. No upload
  22. Created a custom new rule again
  23. No upload … nothing will upload again

Conclusion:
It appears the app is running in the background (13) above. The last time I tried the test, the Nextcloud camera rule was already created when I first open Auto upload. I can’t remember if I had photos on the phone already. However, it does appear the act of creating a custom folder (either before or after the Nextcloud rule) triggers the upload. This appears to only work once though as my uploads are now frozen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

AutoUpload files skipped or upload not starting

3 participants