[PR #396] [MERGED] feat: add dryrun in access_modify() and widest prefix precondition check #448

New issue

Closed

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

BreizhHardware commented

2026-05-07 00:20:57 +02:00

Owner

📋 Pull Request Information

Original PR: https://github.com/ovh/the-bastion/pull/396
Author: @speed47
Created: 5/31/2023
Status: ✅ Merged
Merged: 6/1/2023
Merged by: @speed47

Base: master ← Head: selfAddOwnOnly

📝 Commits (6)

ab6b14a feat: add dryrun in access_modify() and widest prefix precondition check
ec3d2ed feat: plugins: add loadConfig parameter & config validator support
ffed347 feat: add 2 configurable knobs to (self|account)AddPersonalAccess
ca4b8b7 chore: doc: adding plugin configuration autogeneration
56aedc9 chg: remove Debian 9 tests and dockerfile
56ed15c chore: fix GitHub actions under freebsd

📊 Changes

15 files changed (+373 additions, -77 deletions)

View changed files

📝 .github/workflows/freebsd.yml (+3 -3)
📝 .github/workflows/tests.yml (+1 -1)
📝 bin/helper/osh-accountModifyPersonalAccess (+30 -10)
📝 bin/plugin/restricted/accountAddPersonalAccess (+64 -3)
📝 bin/plugin/restricted/selfAddPersonalAccess (+63 -10)
📝 doc/sphinx/build-plugins-help.sh (+23 -1)
📝 doc/sphinx/plugins/restricted/accountAddPersonalAccess.rst (+32 -0)
📝 doc/sphinx/plugins/restricted/selfAddPersonalAccess.rst (+31 -0)
➖ docker/Dockerfile.debian9 (+0 -30)
📝 lib/perl/OVH/Bastion.pm (+4 -3)
📝 lib/perl/OVH/Bastion/Plugin.pm (+19 -6)
📝 lib/perl/OVH/Bastion/Plugin/ACL.pm (+0 -0)
📝 lib/perl/OVH/Bastion/allowkeeper.inc (+47 -7)
📝 lib/perl/OVH/Bastion/configuration.inc (+22 -3)
📝 tests/functional/tests.d/340-selfaccesses.sh (+34 -0)

📄 Description

No description provided

_{🔄 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/396 **Author:** [@speed47](https://github.com/speed47) **Created:** 5/31/2023 **Status:** ✅ Merged **Merged:** 6/1/2023 **Merged by:** [@speed47](https://github.com/speed47) **Base:** `master` ← **Head:** `selfAddOwnOnly` --- ### 📝 Commits (6) - [`ab6b14a`](https://github.com/ovh/the-bastion/commit/ab6b14abebb929f378f24f381d88db02f0abfc45) feat: add dryrun in access_modify() and widest prefix precondition check - [`ec3d2ed`](https://github.com/ovh/the-bastion/commit/ec3d2ed33dc96b1dbcf6189d278a3ee4ab20c901) feat: plugins: add loadConfig parameter & config validator support - [`ffed347`](https://github.com/ovh/the-bastion/commit/ffed347dad7b89cae7d52daf3dbed35bb0da7de8) feat: add 2 configurable knobs to (self|account)AddPersonalAccess - [`ca4b8b7`](https://github.com/ovh/the-bastion/commit/ca4b8b7e72b7bdb80c00e7df476633da9ac6a5a3) chore: doc: adding plugin configuration autogeneration - [`56aedc9`](https://github.com/ovh/the-bastion/commit/56aedc964d61719a588ab879ac88fec21976ae40) chg: remove Debian 9 tests and dockerfile - [`56ed15c`](https://github.com/ovh/the-bastion/commit/56ed15c89f00d7c4f124f10487659c7977de4bde) chore: fix GitHub actions under freebsd ### 📊 Changes **15 files changed** (+373 additions, -77 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/freebsd.yml` (+3 -3) 📝 `.github/workflows/tests.yml` (+1 -1) 📝 `bin/helper/osh-accountModifyPersonalAccess` (+30 -10) 📝 `bin/plugin/restricted/accountAddPersonalAccess` (+64 -3) 📝 `bin/plugin/restricted/selfAddPersonalAccess` (+63 -10) 📝 `doc/sphinx/build-plugins-help.sh` (+23 -1) 📝 `doc/sphinx/plugins/restricted/accountAddPersonalAccess.rst` (+32 -0) 📝 `doc/sphinx/plugins/restricted/selfAddPersonalAccess.rst` (+31 -0) ➖ `docker/Dockerfile.debian9` (+0 -30) 📝 `lib/perl/OVH/Bastion.pm` (+4 -3) 📝 `lib/perl/OVH/Bastion/Plugin.pm` (+19 -6) 📝 `lib/perl/OVH/Bastion/Plugin/ACL.pm` (+0 -0) 📝 `lib/perl/OVH/Bastion/allowkeeper.inc` (+47 -7) 📝 `lib/perl/OVH/Bastion/configuration.inc` (+22 -3) 📝 `tests/functional/tests.d/340-selfaccesses.sh` (+34 -0) </details> ### 📄 Description _No description provided_ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>