[PR #220] [MERGED] fix: accountCreate --uid-auto: rare case where a free UID couldn't be found #320

New issue

Closed

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

BreizhHardware commented

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

Owner

📋 Pull Request Information

Original PR: https://github.com/ovh/the-bastion/pull/220
Author: @speed47
Created: 7/13/2021
Status: ✅ Merged
Merged: 7/19/2021
Merged by: @speed47

Base: master ← Head: nextuid

📝 Commits (1)

92828ca fix: accountCreate --uid-auto: rare case where a free UID couldn't be found

📊 Changes

2 files changed (+30 additions, -4 deletions)

View changed files

📝 bin/helper/osh-accountCreate (+1 -1)
📝 lib/perl/OVH/Bastion/allowkeeper.inc (+29 -3)

📄 Description

This happened when a free UID was found, along with a corresponding GID,
but the corresponding GID for the ttyrec group of the account was not
available. Now this is checked directly in get_next_available_uid()

_{🔄 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/220 **Author:** [@speed47](https://github.com/speed47) **Created:** 7/13/2021 **Status:** ✅ Merged **Merged:** 7/19/2021 **Merged by:** [@speed47](https://github.com/speed47) **Base:** `master` ← **Head:** `nextuid` --- ### 📝 Commits (1) - [`92828ca`](https://github.com/ovh/the-bastion/commit/92828cabbd773ccb45504c12b6284c3a687a5316) fix: accountCreate --uid-auto: rare case where a free UID couldn't be found ### 📊 Changes **2 files changed** (+30 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `bin/helper/osh-accountCreate` (+1 -1) 📝 `lib/perl/OVH/Bastion/allowkeeper.inc` (+29 -3) </details> ### 📄 Description This happened when a free UID was found, along with a corresponding GID, but the corresponding GID for the ttyrec group of the account was not available. Now this is checked directly in get_next_available_uid() --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>