[PR #189] [MERGED] feat: add groupDestroy command for owners #300

Closed
opened 2026-05-07 00:20:15 +02:00 by BreizhHardware · 0 comments

📋 Pull Request Information

Original PR: https://github.com/ovh/the-bastion/pull/189
Author: @speed47
Created: 5/31/2021
Status: Merged
Merged: 6/2/2021
Merged by: @speed47

Base: masterHead: groupDestroy


📝 Commits (2)

  • 99663e9 feat: add groupDestroy command for owners
  • 3b700e9 fix: connect.pl: decode 2K bytes of the ttyrec instead of 1K to not miss messages

📊 Changes

9 files changed (+134 additions, -27 deletions)

View changed files

📝 bin/helper/osh-groupDelete (+21 -14)
bin/plugin/group-owner/groupDestroy (+81 -0)
bin/plugin/group-owner/groupDestroy.json (+9 -0)
📝 bin/plugin/restricted/groupDelete (+3 -0)
📝 bin/shell/connect.pl (+2 -2)
📝 doc/sphinx/plugins/group-owner/index.rst (+1 -0)
📝 doc/sphinx/plugins/restricted/groupDelete.rst (+3 -0)
📝 etc/sudoers.group.template.d/500-base.sudoers (+5 -0)
📝 tests/functional/tests.d/350-groups.sh (+9 -11)

📄 Description

This command deletes a group, as groupDelete does, but works
for owners so that they can delete their own group.
groupDelete remains as a restricted command, able to delete any group.


🔄 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/189 **Author:** [@speed47](https://github.com/speed47) **Created:** 5/31/2021 **Status:** ✅ Merged **Merged:** 6/2/2021 **Merged by:** [@speed47](https://github.com/speed47) **Base:** `master` ← **Head:** `groupDestroy` --- ### 📝 Commits (2) - [`99663e9`](https://github.com/ovh/the-bastion/commit/99663e9462143fcf874f4af320b32794533b49d8) feat: add groupDestroy command for owners - [`3b700e9`](https://github.com/ovh/the-bastion/commit/3b700e9696a9f9345d0e47bd7cc69752dca2dc4b) fix: connect.pl: decode 2K bytes of the ttyrec instead of 1K to not miss messages ### 📊 Changes **9 files changed** (+134 additions, -27 deletions) <details> <summary>View changed files</summary> 📝 `bin/helper/osh-groupDelete` (+21 -14) ➕ `bin/plugin/group-owner/groupDestroy` (+81 -0) ➕ `bin/plugin/group-owner/groupDestroy.json` (+9 -0) 📝 `bin/plugin/restricted/groupDelete` (+3 -0) 📝 `bin/shell/connect.pl` (+2 -2) 📝 `doc/sphinx/plugins/group-owner/index.rst` (+1 -0) 📝 `doc/sphinx/plugins/restricted/groupDelete.rst` (+3 -0) 📝 `etc/sudoers.group.template.d/500-base.sudoers` (+5 -0) 📝 `tests/functional/tests.d/350-groups.sh` (+9 -11) </details> ### 📄 Description This command deletes a group, as `groupDelete` does, but works for owners so that they can delete their own group. `groupDelete` remains as a restricted command, able to delete any group. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
BreizhHardware 2026-05-07 00:20:15 +02:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/the-bastion#300
No description provided.