fix: Consolidate Hetzner support under SFTP provider

- Updated the rclone connection logic to treat Hetzner as an SFTP provider, simplifying the configuration process.
- Removed redundant case handling for Hetzner, streamlining the code and enhancing maintainability.
This commit is contained in:
StarFleetCPTN committed 2025-04-08 20:33:32 -07:00
1 parent de97ad6b50
commit bcd09af2d6
1 file changed
+5 -12
+5 -12
View File
@@ -90,6 +90,10 @@ func TestRcloneConnection(config db.TransferConfig, providerType string, dbInsta
rclonePath = "rclone" rclonePath = "rclone"
} }
if provider == "hetzner" {
provider = "sftp"
}
createArgs := []string{ createArgs := []string{
"config", "create", remoteName, provider, "config", "create", remoteName, provider,
"--config", tempConfigPath, "--config", tempConfigPath,
@@ -105,7 +109,7 @@ func TestRcloneConnection(config db.TransferConfig, providerType string, dbInsta
var createCmd *exec.Cmd var createCmd *exec.Cmd
switch provider { switch provider {
case "sftp": case "sftp", "hetzner":
createArgs = append(createArgs, "host", host, "user", user) createArgs = append(createArgs, "host", host, "user", user)
if port != 0 { if port != 0 {
createArgs = append(createArgs, "port", fmt.Sprintf("%d", port)) createArgs = append(createArgs, "port", fmt.Sprintf("%d", port))
@@ -116,17 +120,6 @@ func TestRcloneConnection(config db.TransferConfig, providerType string, dbInsta
if keyFile != "" { if keyFile != "" {
createArgs = append(createArgs, "key_file", keyFile) createArgs = append(createArgs, "key_file", keyFile)
} }
case "hetzner":
createArgs = append(createArgs, "host", host, "user", user, "pass", pass, "key_file", keyFile)
if port != 0 {
createArgs = append(createArgs, "port", fmt.Sprintf("%d", port))
}
if pass != "" {
createArgs = append(createArgs, "pass", pass)
}
if keyFile != "" {
createArgs = append(createArgs, "key_file", keyFile)
}
case "s3": case "s3":
createArgs = append(createArgs, "provider", "AWS", "env_auth", "false") createArgs = append(createArgs, "provider", "AWS", "env_auth", "false")
if accessKey != "" { if accessKey != "" {