newhinton / Round-Sync

An android cloud file manager, powered by rclone. Visit https://roundsync.com for more information!
https://roundsync.com
GNU General Public License v3.0
1.31k stars 51 forks source link

Union remote with spaces in remote names #36

Open 43qcc2cn opened 2 years ago

43qcc2cn commented 2 years ago

What version of RCX are you using (About -> App version)?

2.0.3-dev (v1.58.1-rcx)

What is your Android version, phone model and manufacturer?

LineageOS 19 (Android 12) on Samsung Galaxy 10

Which steps are required to reproduce this issue?

  1. Create remotes (in my case all Nextcloud WebDAV) with names that contain spaces.
  2. Create a Union remote using the remotes above.
  3. Open Union remote. Returns "Error retrieving directory content."

Does the same issue also occur when using the same configuration on a PC or in Termux?

Same configuration on Linux (Fedora) works fine.

What are the contents of Android/data/de.felixnuesse.extract/files/logs/log.log?

log.txt (click to expand)
2022-09-17 12:40:13 - 2022/09/17 11:40:13 DEBUG : Home directory lookup failed and cannot be used as configuration location: exec: "getent": executable file not found in $PATH
2022/09/17 11:40:13 DEBUG : Setting default for local-no-set-modtime="true" from environment variable RCLONE_LOCAL_NO_SET_MODTIME
2022/09/17 11:40:13 DEBUG : rclone: Version "v1.58.1-rcx" starting with parameters ["/data/app/~~OLvFjjPNbrKXcvmKNC98eA==/de.felixnuesse.extract-vM9oaX5iZyUlOYBjyyeRCw==/lib/arm64/librclone.so" "--cache-chunk-path" "/data/user/0/de.felixnuesse.extract/cache" "--cache-db-path" "/data/user/0/de.felixnuesse.extract/cache" "--config" "/data/user/0/de.felixnuesse.extract/files/rclone.conf" "-vvv" "lsjson" "Nextcloud:"]
2022/09/17 11:40:13 DEBUG : Creating backend with remote "Nextcloud:"
2022/09/17 11:40:13 DEBUG : Using config file from "/data/user/0/de.felixnuesse.extract/files/rclone.conf"
2022/09/17 11:40:13 DEBUG : Creating backend with remote "Digital:"
2022/09/17 11:40:13 DEBUG : Creating backend with remote "Hosting:"
2022/09/17 11:40:13 DEBUG : Creating backend with remote "Tab:ro"
2022/09/17 11:40:13 DEBUG : found headers: 
2022/09/17 11:40:13 DEBUG : Creating backend with remote "Webo:ro"
2022/09/17 11:40:13 Failed to create file system for "Nextcloud:": didn't find section in config file

2022-09-17 12:40:13 - 2022/09/17 11:40:13 DEBUG : Home directory lookup failed and cannot be used as configuration location: exec: "getent": executable file not found in $PATH
2022/09/17 11:40:13 DEBUG : Setting default for local-no-set-modtime="true" from environment variable RCLONE_LOCAL_NO_SET_MODTIME

2022-09-17 12:40:13 - 2022/09/17 11:40:13 DEBUG : Home directory lookup failed and cannot be used as configuration location: exec: "getent": executable file not found in $PATH
2022/09/17 11:40:13 DEBUG : Setting default for local-no-set-modtime="true" from environment variable RCLONE_LOCAL_NO_SET_MODTIME

The multiple "Creating backend with remote" lines for each remote don't look correct to me.

43qcc2cn commented 2 years ago

Completely forgot that, out of habit, I always quote things that have spaces when using Linux.

If I manually edit the extRact config and quote the remotes in the Union section it works fine.