[PR #495] [MERGED] fix: regression introduced by `932e72e` for stealth stdout in ssh #526

New issue

Closed

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

BreizhHardware commented

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

Owner

📋 Pull Request Information

Original PR: https://github.com/ovh/the-bastion/pull/495
Author: @speed47
Created: 9/25/2024
Status: ✅ Merged
Merged: 9/25/2024
Merged by: @speed47

Base: master ← Head: part2of2

📝 Commits (1)

6d709e0 fix: regression introduced by 932e72e for stealth stdout in ssh

📊 Changes

2 files changed (+22 additions, -10 deletions)

View changed files

📝 lib/perl/OVH/Bastion.pm (+12 -4)
📝 tests/unit/run.pl (+10 -6)

📄 Description

Before 932e72e, plugin-scoped stealthStdout was ignored, which was fixed by 932e72e which in turn made ssh ignore the pattern-based egress ssh stealthStdout option.

This fix ensures stealthStdout is honored for both plugins and egress ssh.

_{🔄 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/495 **Author:** [@speed47](https://github.com/speed47) **Created:** 9/25/2024 **Status:** ✅ Merged **Merged:** 9/25/2024 **Merged by:** [@speed47](https://github.com/speed47) **Base:** `master` ← **Head:** `part2of2` --- ### 📝 Commits (1) - [`6d709e0`](https://github.com/ovh/the-bastion/commit/6d709e094cb96a583989ce886a424227c46fd36d) fix: regression introduced by 932e72e for stealth stdout in ssh ### 📊 Changes **2 files changed** (+22 additions, -10 deletions) <details> <summary>View changed files</summary> 📝 `lib/perl/OVH/Bastion.pm` (+12 -4) 📝 `tests/unit/run.pl` (+10 -6) </details> ### 📄 Description Before 932e72e, plugin-scoped stealthStdout was ignored, which was fixed by 932e72e which in turn made ssh ignore the pattern-based egress ssh stealthStdout option. This fix ensures stealthStdout is honored for both plugins and egress ssh. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>