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

feat: Add new options to rclone backend #139

Merged
merged 1 commit into from
Jan 28, 2024
Merged

feat: Add new options to rclone backend #139

merged 1 commit into from
Jan 28, 2024

Conversation

aawsome
Copy link
Member

@aawsome aawsome commented Jan 27, 2024

This PR adds the following options to the rclone backend:

  • rclone-command - to manually define the rclone command used. Default: "rclone serve restric --addr localhost:0".
  • use-password - whether to generate and use user/password for the rclone rest conneciotn. Default: "true"
  • rest-url - the rest URL to use. Default: Get URL from rclone output

Also adds a better treatment for the spawned thread which outputs rclone errors. This closes #22 and rustic-rs/rustic#1006

@aawsome aawsome force-pushed the rclone-command branch 2 times, most recently from 7378592 to 6b3db8f Compare January 27, 2024 21:36
@simonsan
Copy link
Contributor

simonsan commented Jan 28, 2024

Looks good to me! 🚀

EDIT: Misread something, so it's fine now.

@simonsan simonsan added this pull request to the merge queue Jan 28, 2024
Merged via the queue into main with commit 27499db Jan 28, 2024
56 checks passed
@simonsan simonsan deleted the rclone-command branch January 28, 2024 04:42
github-merge-queue bot pushed a commit that referenced this pull request Jan 29, 2024
* Add more access methods to repository

* fixup access

* make more node properties public

* add caching

* add vfs

* use dav_server

* remove fuse

* fix lints

* update dav-server

* dont't expose cache type

* Add file policy to Vfs

* fixup dump.rs

* fix clippy lints

* Adjust API

* feat: Add options rclone-command, use-password, rest-url to rclone backend (#139)

* docs: fix typo

Signed-off-by: simonsan <[email protected]>

* allow to use FilePolicy in clap

* prevent potential bug about unparitioned vec

* fix seek value after reading

* add some docs

* dprint lint

* refactor: extract constants to own module

Signed-off-by: simonsan <[email protected]>

* fix: typo

Signed-off-by: simonsan <[email protected]>

* docs: add docs for constants

Signed-off-by: simonsan <[email protected]>

* docs: add TODO for missing Errors, add some docs

Signed-off-by: simonsan <[email protected]>

* refactor: error handling in vfs

Signed-off-by: simonsan <[email protected]>

* docs: add error doc

Signed-off-by: simonsan <[email protected]>

* use FilePolicy only in WebDavFs

* docs: add todos for error handling

Signed-off-by: simonsan <[email protected]>

* fix: unreachable pub

Signed-off-by: simonsan <[email protected]>

* remove unwraps in vfs

* fix: clippy

Signed-off-by: simonsan <[email protected]>

* style: cargo fmt

Signed-off-by: simonsan <[email protected]>

* docs: document more errors

Signed-off-by: simonsan <[email protected]>

* fix: sort enum variants alphabetically

Signed-off-by: simonsan <[email protected]>

* fix: sort enum variants alphabetically

Signed-off-by: simonsan <[email protected]>

* docs: more docs

Signed-off-by: simonsan <[email protected]>

* fix: imports

Signed-off-by: simonsan <[email protected]>

* docs: more docs

Signed-off-by: simonsan <[email protected]>

* docs: more docs

Signed-off-by: simonsan <[email protected]>

* docs: more docs

Signed-off-by: simonsan <[email protected]>

* docs: more docs

Signed-off-by: simonsan <[email protected]>

* docs: more docs

Signed-off-by: simonsan <[email protected]>

* docs: more docs

Signed-off-by: simonsan <[email protected]>

* chore: cleanup

Signed-off-by: simonsan <[email protected]>

---------

Signed-off-by: simonsan <[email protected]>
Co-authored-by: simonsan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rclone seems to keep running after backup ends
2 participants