Skip to content

Make regsubst() sensitive-aware#354

Open
cocker-cc wants to merge 1 commit intoOpenVoxProject:mainfrom
cocker-cc:Make_regsubst_sensitive-aware
Open

Make regsubst() sensitive-aware#354
cocker-cc wants to merge 1 commit intoOpenVoxProject:mainfrom
cocker-cc:Make_regsubst_sensitive-aware

Conversation

@cocker-cc
Copy link
Contributor

This commit updates regsubst() to take in Sensitive type targets. Specifically, regsubst() can now take targets that are either Sensitive String, Sensitive arrays that have a combination of String and/or Sensitive[String], or non-Sensitive arrays that have combination of String and/or Sensitive[String].

@cocker-cc
Copy link
Contributor Author

My Commit was already 2024 merged to Puppet.

@bastelfreak
Copy link
Contributor

Why is this missing in openvox? 🤔

@bastelfreak
Copy link
Contributor

@cocker-cc can you rebase please? I fixed some CI issues

@cocker-cc cocker-cc force-pushed the Make_regsubst_sensitive-aware branch from f438841 to 435b757 Compare March 5, 2026 10:53
@corporate-gadfly
Copy link
Contributor

@cocker-cc Also the commit needs a verified signature, TY.

This commit updates regsubst() to take in Sensitive type targets. Specifically,
regsubst() can now take targets that are either Sensitive String, Sensitive
arrays that have a combination of String and/or Sensitive[String], or
non-Sensitive arrays that have combination of String and/or Sensitive[String].
@cocker-cc cocker-cc force-pushed the Make_regsubst_sensitive-aware branch from 435b757 to b6ebaf4 Compare March 5, 2026 19:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants