ikifar2012 / remote-backup-addon

Automatically create and backup HA snapshots using SCP
30 stars 4 forks source link

Issue with spaces in remote path #43

Closed uberboffin closed 1 year ago

uberboffin commented 2 years ago

Version info:

 Add-on version: 2022.5.0
 You are running the latest version of this add-on.
 System: Home Assistant OS 7.6  (amd64 / qemux86-64)
 Home Assistant Core: 2022.4.5
 Home Assistant Supervisor: 2022.05.2

We are using SCP to copy to a remote Mac server. The path needs to be escaped, as it has spaces in it. The add-on does use quotes for the path, so I can simply enter the path into the config file. However, if I do not escape, then the following error is shown:

scp: ambiguous target

If I do escape the string, then the move of the tar to the friendly name then breaks, as you cannot mix quotes and escaped spaces together.

stale[bot] commented 2 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

patman15 commented 2 years ago

fixed in commit https://github.com/ikifar2012/remote-backup-addon/pull/59/commits/a31d17f6fcc4469483cc3c1e064ed42ad11f94b8 PR https://github.com/ikifar2012/remote-backup-addon/pull/59