Skip to content

Conversation

@lukpueh
Copy link
Member

@lukpueh lukpueh commented Feb 10, 2023

This is a BREAKING CHANGE for in-toto, must coordinate release! (cc in-toto/in-toto#544)


This module was originally developed in in-toto and transferred to securesystemslib in #174, primarily as Py2/Py3-agnostic wrapper around stdlib's subprocess.run, to run the gpg command. In #502 we switched to using subprocess.run directly.

Another wrapper around subprocess.run, provided by the module, allows capturing standard streams AND write them to a terminal. It was developed as specific in-toto-run feature and does not need to be public API in sslib. in-toto/in-toto#544 moves the function back to in-toto.

closes #345,
transfers #337 (to in-toto)

This module was originally developed in in-toto and transferred to
securesystemslib in secure-systems-lab#174, primarily as Py2/Py3-agnostic wrapper
around stdlib's `subprocess.run`, to run the `gpg` command. In secure-systems-lab#502
we switched to using `subprocess.run` directly.

Another wrapper around `subprocess.run`, provided by the module,
allows capturing standard streams AND write them to a terminal. It
was developed as specific `in-toto-run` feature and does not need
to be public API in sslib. in-toto/in-toto#544 moves the function
back to in-toto.

closes secure-systems-lab#345,
transfers secure-systems-lab#337 (to in-toto)

Signed-off-by: Lukas Puehringer <[email protected]>
@lukpueh
Copy link
Member Author

lukpueh commented Feb 10, 2023

cc @trishankatdatadog, @ofek

@trishankatdatadog
Copy link
Contributor

cc @trishankatdatadog, @ofek

Noted, thanks, shouldn't affect us unless we upgrade @yzhan289 @iliakur

Copy link
Member

@adityasaky adityasaky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving on behalf of in-toto/in-toto.

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.

Default timeout setting helper in process module does not work as intended

3 participants