Skip to content
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

Remove securesystemslib.process module #505

Merged
merged 1 commit into from
Feb 16, 2023

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 <lukas.puehringer@nyu.edu>
@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