-
Notifications
You must be signed in to change notification settings - Fork 2.2k
(PUP-11326) Make regsubst() sensitive-aware #8799
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
(PUP-11326) Make regsubst() sensitive-aware #8799
Conversation
|
Can one of the admins verify this patch? |
e3d1ea9 to
ed4a58a
Compare
ed4a58a to
520b873
Compare
520b873 to
4ec8679
Compare
4ec8679 to
bd61004
Compare
|
Thanks @cocker-cc you'll want to rebase this PR against #8879 after it's merged to resolve the rspec issue on Ruby 3 |
bd61004 to
b8a49d2
Compare
b8a49d2 to
4a556d2
Compare
4a556d2 to
b998ce7
Compare
320fc93 to
a7dc145
Compare
a7dc145 to
10c8b22
Compare
|
There is one more possible |
cff4ef6 to
6ebddd7
Compare
6ebddd7 to
425a6f3
Compare
425a6f3 to
6d30ad8
Compare
6d30ad8 to
aac6132
Compare
aac6132 to
92f6efd
Compare
92b15f6 to
6b4a0ae
Compare
6b4a0ae to
a580772
Compare
a580772 to
c8975ba
Compare
0addbdb to
550af99
Compare
550af99 to
3997234
Compare
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]. Co-authored-by: Henrik Lindberg <[email protected]> Co-authored-by: Aria Li <[email protected]>
3997234 to
0bf1a6b
Compare
|
Thanks for your contribution @cocker-cc ! |
PUP-11326
Possible Use-Case out of Real-Life: