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

Unable to create/specify Git repo #150

Closed
clach04 opened this issue Feb 3, 2024 · 10 comments · Fixed by #161
Closed

Unable to create/specify Git repo #150

clach04 opened this issue Feb 3, 2024 · 10 comments · Fixed by #161
Labels
bug Something isn't working Git repos Related to Git syncing

Comments

@clach04
Copy link

clach04 commented Feb 3, 2024

Using Android 10 phone.

I can generate an ssh key fine without problem, so slightly different from #58

But selecting the git repo does nothing. If I select Directory or WebDAV both bring up a dialog, Git brought up an Android permissions request for file access which I granted but now does nothing. Seen with FDroid install and the separate APK (orgzly-revived-1.8.14.apk) from https://github.com/orgzly-revived/orgzly-android-revived/releases/tag/v1.8.14

Is this user error or a bug (it feels like the later).

@clach04 clach04 added the bug Something isn't working label Feb 3, 2024
@amberin amberin added the Git repos Related to Git syncing label Feb 4, 2024
@amberin
Copy link
Member

amberin commented Feb 4, 2024

Hello, thank you for reporting. What Android version are you on?

It's definitely not a user error. :)

@clach04
Copy link
Author

clach04 commented Feb 4, 2024

Android 10 (I'll go back and update the original report).

@amberin
Copy link
Member

amberin commented Feb 5, 2024

Will see if I can recreate. If you can check the device logs (e.g. with an app such as Logcat Reader) and filter for orgzly, that would be great. I will suggest adding cloning errors to the "developer major events log".

@wilderjds
Copy link

I have a similar issue; git cloning fails silently; logcat is attached
logcat_02-04-2024_23-31-12.txt

@amberin
Copy link
Member

amberin commented Feb 5, 2024

I have a similar issue; git cloning fails silently; logcat is attached logcat_02-04-2024_23-31-12.txt

Great, thank you. Which Android version are you on?

This error from JGit is something we have not seen before. I hope it might be an R8/minify issue.

@wilderjds
Copy link

Android 11 here. Please let me know if I can be of more help.

@amberin
Copy link
Member

amberin commented Feb 11, 2024

Seems to be this bug: https://bugs.eclipse.org/bugs/show_bug.cgi?id=582423

I will look at downgrading JGit to 6.6.1 for now.

@amberin
Copy link
Member

amberin commented Feb 12, 2024

I think I have resolved the issues on Android 11, but on 10 it looks worse. It seems that our current solution for external storage permission (0d0dc29) does not work on Android 10, and I don't know if it has ever worked.

The permission check always gets a negative result, so we always end up in the "request permission" logic, but then nothing happens once you have answered the prompt one time. The Android GUI says that the app has "storage" permission, but I'm not sure what that really means.

If I change the logic to launch the activity, cloning fails because of missing write permissions.

Hopefully, we can get rid of this type of issues if we tackle #21.

@clach04
Copy link
Author

clach04 commented Feb 13, 2024

Thanks for the detailed update @amberin as an Android 10 (potential) user that's a real pain 😿.

I know a few authors of Android apps that did not want to deal with android storage api, have ended up using the Android app "home" directory (which it sounds like #21 maybe suggesting?) as it was least hassle to develop. It doesn't rule out real permissions/API in the future.

@amberin
Copy link
Member

amberin commented Feb 13, 2024

I think I found and solved the permission problem on Android 10. I will open a PR after some testing.

@amberin amberin linked a pull request Feb 13, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Git repos Related to Git syncing
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants