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

Autoupload not working in Andorid 7 #612

Closed
jamfx opened this issue Jan 31, 2017 · 75 comments
Closed

Autoupload not working in Andorid 7 #612

jamfx opened this issue Jan 31, 2017 · 75 comments
Assignees

Comments

@jamfx
Copy link

jamfx commented Jan 31, 2017

Actual behaviour

-Take a Photo. Wait for upload. Die...

Expected behaviour

-Take a Photo. Looking how it loads up. Be happy. :)

Steps to reproduce

  1. Install NextCloud App (even with Beta)
  2. Take a Photo.
  3. Wait for upload, which never happens... :(

Environment data

Android version:
Android 7.0 Patchlevel, 1. December 2016

Device model:
Honor 8, no root.

Stock or customized system:
Nothing customized. No Root. Just a simple Phone. (But also happened on my Nexus 5x with Android 7)

Nextcloud app version:
Nextcloud Beta Version 20170130

Nextcloud server version:
Nextcloud 11.0.0

Surely this issue is reported already, but I couldn't find it. So please feel free to merge / close. But please, do something about this issue soon! Thanks a lot!

@tobiasKaminsky
Copy link
Member

Have you configured AutoUpload the right way?
Can you test a 3rd party camera app? (there are some vendors that cause some problems with their stock camera)

@jamfx
Copy link
Author

jamfx commented Jan 31, 2017

Yes, I configured it right away. I double checked now. Everything as I think it should.
I tried the following Cam Apps:
a) Build-In Camera
b) Google Camera (Extra App from Google-Play Store)
c) Perfectly Clear App

None of these triggered an upload. :(

@bekl101
Copy link

bekl101 commented Feb 1, 2017

Same problem with Galaxy S7 7.0.
No upload action after taking a picture, neither taking a screenshot.
upload is only working when i copy a picture manually to "/storage/emulated/0/DCIM/Screenshots with a file-manager.
there is also no action when the build in camera-app uses the SD-CARD as storage.
tested with nextcloudapp beta 20170130

@jamfx
Copy link
Author

jamfx commented Feb 1, 2017

Nice points! I have my pictues on the SD-Card, too! Screenshots have never been uploaded. Also, the upload is not triggered, if I copy the image somewhere. Upload just never starts.

@KekcuHa
Copy link

KekcuHa commented Feb 1, 2017

Confirm that, no autoupload at nextcloud android app beta 20170130 at SGS7.

@KekcuHa
Copy link

KekcuHa commented Feb 1, 2017

Seems to be double of #498 (link)

@mario
Copy link
Contributor

mario commented Feb 1, 2017

@KekcuHa SGS7 has only Android 6 though, right?

@bekl101
Copy link

bekl101 commented Feb 2, 2017

@mario
No, since last week 7.0 is rolled out by Vodafone Germany on SGS7

@KekcuHa
Copy link

KekcuHa commented Feb 2, 2017

@mario, no, SGS7 now on Android 7.0 as been already pointed by @bekl101
The same situation (no autoupload at beta client) at LG G Pro2 running on Android 5.0.1.

@mario
Copy link
Contributor

mario commented Feb 2, 2017

Thanks for letting me know - there is SGS7 among my phones, but no Android 7 yet on it. Anyway, I'm looking into the issues. As for now, AutoUpload is only on 7 (we hope to bring it to 5 and 6), and 4,5,6 use InstantUpload (older version).

That being said, AutoUpload does have it's issues that I'm currently correcting.

@bekl101
Copy link

bekl101 commented Feb 2, 2017

is there a temporary workaround to sync photos taken on my mobile with the nextcloud server until the Autoupload problem is fixed?
@mario
if you need logs from the beta let me know.

@kubjo
Copy link

kubjo commented Feb 2, 2017

Same issue with LG G5 Android 7.0 Nextcloud Beta 20170130

@mario
Copy link
Contributor

mario commented Feb 2, 2017

Hello,

it looks like I've determined the root cause of the problem, but I have to figure out how/if it's easy to fix or not.

Thank you for your patience!

@daymr
Copy link

daymr commented Feb 2, 2017

Can confirm.. verizon galaxy s7 android 7.0 beta 2 Nov 22 build.. autoupload is ENTIRELY broken and does not function with any camera app or storage location. Please fix. Thanks for the awesome dev work!

@mario
Copy link
Contributor

mario commented Feb 2, 2017

So it seems that the problem is bigger than we thought - some phones (Galaxy ones in particular) just don't emit the right information that we need to make autoupload work. As such, we need to rethink how we're gonna handle uploads in the future and pave a way to true two-way sync.

@jamfx
Copy link
Author

jamfx commented Feb 2, 2017

Hi Mario,
for my Side, I don't need a two-way-sync on my phone. I just would love to have a Picture Upload. Maybe this can be reviewed in a first way?
If I can help for this somehow, let me know.

@mario
Copy link
Contributor

mario commented Feb 2, 2017

I have strong reasons to believe the Auto upload will be fixed soon as I am working on rewriting our upload service, and will be brought to Android 5.x and 6.x as well. 4.x can be expected, but I do now know when - until then, Instant upload can be used there (when it works(.

@mario
Copy link
Contributor

mario commented Feb 2, 2017

Fix ready (#615), we'll hopefully have it in beta soon!

@mario mario added the pr exists label Feb 3, 2017
@mario
Copy link
Contributor

mario commented Feb 5, 2017

Few issues with the fix, working on it!

@daymr
Copy link

daymr commented Feb 5, 2017

************ CAUSE OF ERROR ************

java.lang.IllegalStateException: Apps may not schedule more than 100 distinct jobs
at android.os.Parcel.readException(Parcel.java:1701)
at android.os.Parcel.readException(Parcel.java:1646)
at android.app.job.IJobScheduler$Stub$Proxy.schedule(IJobScheduler.java:158)
at android.app.JobSchedulerImpl.schedule(JobSchedulerImpl.java:42)
at com.owncloud.android.services.FileAlterationMagicListener.onFileCreate(FileAlterationMagicListener.java:103)
at org.apache.commons.io.monitor.FileAlterationObserver.doCreate(FileAlterationObserver.java:379)
at org.apache.commons.io.monitor.FileAlterationObserver.checkAndNotify(FileAlterationObserver.java:345)
at org.apache.commons.io.monitor.FileAlterationObserver.checkAndNotify(FileAlterationObserver.java:304)
at org.apache.commons.io.monitor.FileAlterationMonitor.run(FileAlterationMonitor.java:182)
at java.lang.Thread.run(Thread.java:761)

************ DEVICE INFORMATION ***********
Brand: Verizon
Device: heroqltevzw
Model: SM-G930V
Id: NRD90M
Product: heroqltevzw

************ FIRMWARE ************
SDK: 24
Release: 7.0
Incremental: G930VVRU4ZPKC

@daymr
Copy link

daymr commented Feb 5, 2017

Unrecoverable crash screen appears every 10 seconds! Even on reboot.. guess clear program data?

@mario
Copy link
Contributor

mario commented Feb 5, 2017 via email

@JSoko
Copy link
Member

JSoko commented Feb 6, 2017

I have also that crashscreen on SGS 7 nougat with beta 170204:

************ CAUSE OF ERROR ************

java.lang.IllegalStateException: Apps may not schedule more than 100 distinct jobs
at android.os.Parcel.readException(Parcel.java:1701)
at android.os.Parcel.readException(Parcel.java:1646)
at android.app.job.IJobScheduler$Stub$Proxy.schedule(IJobScheduler.java:158)
at android.app.JobSchedulerImpl.schedule(JobSchedulerImpl.java:42)
at com.owncloud.android.services.FileAlterationMagicListener.onFileCreate(FileAlterationMagicListener.java:103)
at org.apache.commons.io.monitor.FileAlterationObserver.doCreate(FileAlterationObserver.java:379)
at org.apache.commons.io.monitor.FileAlterationObserver.checkAndNotify(FileAlterationObserver.java:345)
at org.apache.commons.io.monitor.FileAlterationObserver.checkAndNotify(FileAlterationObserver.java:304)
at org.apache.commons.io.monitor.FileAlterationMonitor.run(FileAlterationMonitor.java:182)
at java.lang.Thread.run(Thread.java:762)

************ DEVICE INFORMATION ***********
Brand: samsung
Device: herolte
Model: SM-G930F
Id: NRD90M
Product: heroltexx

************ FIRMWARE ************
SDK: 24
Release: 7.0
Incremental: G930FXXU1DPLT

@mario
Copy link
Contributor

mario commented Feb 6, 2017

170206 should fix it.

@bekl101
Copy link

bekl101 commented Feb 6, 2017

crashscreen after installing 170206 with a previous app version (170204) installed.
after removing 170204 and installing 170206 no crashscreen.
autoupload tested with a few new piktures seems to work as expected.

@jamfx
Copy link
Author

jamfx commented Feb 6, 2017

I tried Nextcloud 20170206 beta and still have no auto upload on my Phone. I didn't had crashes in earlier versions... Just no Auto Upload at all.

@mario
Copy link
Contributor

mario commented Feb 6, 2017

That sounds strange @jamfx - have you configured this in the Auto Upload screen?

@jamfx
Copy link
Author

jamfx commented Feb 6, 2017

Yes, where else could this be configured?

@bekl101
Copy link

bekl101 commented Feb 8, 2017

tested 20170208 after clean install.
Autoupload seems to work , no doble uploads like yesterday.
but i didn´t find a log file in the app, on filesystem i found it in storage/6331-6230/Android/data/com.nextcloud.beta/files/nextcloud_log/log.

after saving setting for Autoupload Screenshots errormessage pops up "Nextcloud beta beendet restart app",
second try "Nextcloud beta wird wiederholt beendet App schließen"
Enable AutoUpload for screenshots with defaults by clicking the cloud icon, success, upload works

@jamfx
Copy link
Author

jamfx commented Feb 8, 2017

This Version (20170208) works for me! No double Uploads and Uploads start quite quickly. Nice! Thanks a lot! 💯 🥇

@bekl101
Copy link

bekl101 commented Feb 8, 2017

@jamfx
do you see the log within the app?
sync seems to be stable

@jamfx
Copy link
Author

jamfx commented Feb 8, 2017

As far as I can see, nope... sorry, no logs.

@mario
Copy link
Contributor

mario commented Feb 8, 2017

@bekl101 what does "Nextcloud beta wird wiederholt beendet App schließen" means?

@mario
Copy link
Contributor

mario commented Feb 8, 2017

@bekl101 I just noticed you linked me to this issue, instead of the one relevant for SD card? :)

@bekl101
Copy link

bekl101 commented Feb 8, 2017

That means, there comes a popup which told you that the application stops working several times and will be closed
i don´t know if @jamfx has the same problem. He confirmed only that the app shows no logfile.

@mario
Copy link
Contributor

mario commented Feb 8, 2017

Niiice! :) Will investigate.

@mario
Copy link
Contributor

mario commented Feb 8, 2017

As for logs ... dunno, my code shouldn't affect logs at all.

@bekl101
Copy link

bekl101 commented Feb 8, 2017

you´re right #476 is the right issue.
sorry for confusing you :)

its strange, because in the version 20170207 and before the logfile was display correct in the app

@JSoko
Copy link
Member

JSoko commented Feb 8, 2017

Hello,
on sgs7 nougat 20170208 works for me. Autouploads are working an no double uploads...
Thank You! 👍

The logs window in the app is empty, but the currentLog.txt has 143 kb

@mario mario added this to the Nextcloud App 1.4.2 milestone Feb 8, 2017
@mario mario self-assigned this Feb 8, 2017
@mario
Copy link
Contributor

mario commented Feb 8, 2017

Glad this is working, thanks folks!

There will be some changes tomorrow to auto-upload so please test it as well.

I will close this ticket now, but please feel free to re-open it if issues crop up again.

@mario mario closed this as completed Feb 8, 2017
@RogerSik
Copy link

Sorry for re-opening. When we can count with Version 1.4.2?

@mario
Copy link
Contributor

mario commented Feb 13, 2017

@RogerSik RC1 soon hopefully :)

@mario
Copy link
Contributor

mario commented Feb 13, 2017

@bekl101 new beta!!

https://github.com/nextcloud/android/raw/beta/apks/latest.apk

@bekl101
Copy link

bekl101 commented Feb 13, 2017

@mario
short question to Autoupload.
Is it correct that when i activate "delete file after upload" the taken picture is stored on my NC, in the folder on my local SD-Card and the original file in DCIM is deleted?
If yes maby i have a small issue. Everything works but the file is not deleted.

@tobiasKaminsky
Copy link
Member

@bekl101 if you choose "delete file after upload" there should be no file left on the device after a successful upload. It is the same as uploading it and then delete the local copy AND the original file.

@mario
Copy link
Contributor

mario commented Feb 13, 2017

@tobiasKaminsky it doesn't do that. Can you look into it tomorrow please, if you have the time?

@bekl101
Copy link

bekl101 commented Feb 14, 2017

ok, so it wasn´t my fault.
Same problem if you choose "move file to app folder"
The new picture wasn´t moved to app folder. It stays only in the DCIM folder. But the NC app shows it as synced in the app folder

@tobiasKaminsky
Copy link
Member

Is the source file on an external storage? I remember that this can lead to problems where the file cannot be deleted and fails silently. This would explain why both is not working as the file needs to be deleted in both cases.

@bekl101
Copy link

bekl101 commented Feb 14, 2017

Yes, source is an external storage (sd-card). Later i can test with pictures saved to internal storage.

Changed camera target to internal storage, everything works as expected

@Wikinaut
Copy link
Contributor

Wikinaut commented Mar 1, 2017

Same problem Android 7.0 on HUAWEI P8 LITE 2017 (Auto upload does not work)

@mario
Copy link
Contributor

mario commented Mar 1, 2017

What version @Wikinaut ?

@mario
Copy link
Contributor

mario commented Mar 1, 2017

All should be fixed in beta.

@Wikinaut
Copy link
Contributor

Wikinaut commented Mar 1, 2017

Android Nextcloud 1.4.1

@mario
Copy link
Contributor

mario commented Mar 1, 2017

Cool. Should be fixed in beta and 1.4.2 so worry not :)

@Wikinaut
Copy link
Contributor

Wikinaut commented Mar 1, 2017

@mario I compiled the beta, but the App-PIN entry does not work when trying to contact my instance in the 1.4.2RC1 version.... and so I cannot contact my server with this beta version..... 👎
(filed as #702 )

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

Successfully merging a pull request may close this issue.

10 participants