[PR #485] [MERGED] feat: add rsync support (fix #301) #519

New issue

Closed

opened 2026-05-07 00:21:19 +02:00 by BreizhHardware · 0 comments

BreizhHardware commented

2026-05-07 00:21:19 +02:00

Owner

📋 Pull Request Information

Original PR: https://github.com/ovh/the-bastion/pull/485
Author: @speed47
Created: 7/5/2024
Status: ✅ Merged
Merged: 9/17/2024
Merged by: @speed47

Base: master ← Head: rsync

📝 Commits (4)

6e09b8f refacto: move special protocols checks into a lib
34fa77a enh: plugins: add --protocol to handle scp, sftp, rsync
cbd945a enh: plugins: add validate_tuple() so a plugin can validate user@host:port independently
1f5cef3 feat: add rsync support to --protocol

📊 Changes

41 files changed (+1515 additions, -1125 deletions)

View changed files

📝 bin/plugin/group-aclkeeper/groupAddServer (+36 -23)
📝 bin/plugin/group-aclkeeper/groupDelServer (+25 -17)
📝 bin/plugin/group-gatekeeper/groupAddGuestAccess (+35 -27)
📝 bin/plugin/group-gatekeeper/groupDelGuestAccess (+31 -24)
➕ bin/plugin/open/rsync (+149 -0)
➕ bin/plugin/open/rsync.json (+4 -0)
📝 bin/plugin/open/scp (+21 -73)
📝 bin/plugin/open/sftp (+14 -67)
📝 bin/plugin/restricted/accountAddPersonalAccess (+37 -24)
📝 bin/plugin/restricted/accountDelPersonalAccess (+29 -21)
📝 bin/plugin/restricted/selfAddPersonalAccess (+36 -23)
📝 bin/plugin/restricted/selfDelPersonalAccess (+23 -15)
📝 bin/shell/osh.pl (+7 -4)
➕ doc/sphinx-plugins-override/rsync.override.rst (+31 -0)
📝 doc/sphinx-plugins-override/scp.override.rst (+1 -1)
📝 doc/sphinx-plugins-override/sftp.override.rst (+1 -1)
📝 doc/sphinx/index.rst (+1 -1)
📝 doc/sphinx/plugins/group-aclkeeper/groupAddServer.rst (+23 -31)
📝 doc/sphinx/plugins/group-aclkeeper/groupDelServer.rst (+15 -29)
📝 doc/sphinx/plugins/group-gatekeeper/groupAddGuestAccess.rst (+17 -30)

...and 21 more files

📄 Description

draft: missing functional tests

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/ovh/the-bastion/pull/485 **Author:** [@speed47](https://github.com/speed47) **Created:** 7/5/2024 **Status:** ✅ Merged **Merged:** 9/17/2024 **Merged by:** [@speed47](https://github.com/speed47) **Base:** `master` ← **Head:** `rsync` --- ### 📝 Commits (4) - [`6e09b8f`](https://github.com/ovh/the-bastion/commit/6e09b8f81f6ee23df7d19950b94cb24966857354) refacto: move special protocols checks into a lib - [`34fa77a`](https://github.com/ovh/the-bastion/commit/34fa77a0d6bf099dc4153d91077475044ccd8750) enh: plugins: add --protocol to handle scp, sftp, rsync - [`cbd945a`](https://github.com/ovh/the-bastion/commit/cbd945aa0cc6be71f17a77b88b975bcffe856a4d) enh: plugins: add validate_tuple() so a plugin can validate user@host:port independently - [`1f5cef3`](https://github.com/ovh/the-bastion/commit/1f5cef371bcfc949cdcee810d11ecfdc671ba8ff) feat: add rsync support to --protocol ### 📊 Changes **41 files changed** (+1515 additions, -1125 deletions) <details> <summary>View changed files</summary> 📝 `bin/plugin/group-aclkeeper/groupAddServer` (+36 -23) 📝 `bin/plugin/group-aclkeeper/groupDelServer` (+25 -17) 📝 `bin/plugin/group-gatekeeper/groupAddGuestAccess` (+35 -27) 📝 `bin/plugin/group-gatekeeper/groupDelGuestAccess` (+31 -24) ➕ `bin/plugin/open/rsync` (+149 -0) ➕ `bin/plugin/open/rsync.json` (+4 -0) 📝 `bin/plugin/open/scp` (+21 -73) 📝 `bin/plugin/open/sftp` (+14 -67) 📝 `bin/plugin/restricted/accountAddPersonalAccess` (+37 -24) 📝 `bin/plugin/restricted/accountDelPersonalAccess` (+29 -21) 📝 `bin/plugin/restricted/selfAddPersonalAccess` (+36 -23) 📝 `bin/plugin/restricted/selfDelPersonalAccess` (+23 -15) 📝 `bin/shell/osh.pl` (+7 -4) ➕ `doc/sphinx-plugins-override/rsync.override.rst` (+31 -0) 📝 `doc/sphinx-plugins-override/scp.override.rst` (+1 -1) 📝 `doc/sphinx-plugins-override/sftp.override.rst` (+1 -1) 📝 `doc/sphinx/index.rst` (+1 -1) 📝 `doc/sphinx/plugins/group-aclkeeper/groupAddServer.rst` (+23 -31) 📝 `doc/sphinx/plugins/group-aclkeeper/groupDelServer.rst` (+15 -29) 📝 `doc/sphinx/plugins/group-gatekeeper/groupAddGuestAccess.rst` (+17 -30) _...and 21 more files_ </details> ### 📄 Description draft: missing functional tests --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>