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

security(install): resolve possible injections #248

Open
Tracked by #494
ocervell opened this issue Apr 8, 2024 · 0 comments
Open
Tracked by #494

security(install): resolve possible injections #248

ocervell opened this issue Apr 8, 2024 · 0 comments
Assignees
Labels
important security Security issues.

Comments

@ocervell
Copy link
Contributor

ocervell commented Apr 8, 2024

class SourceInstaller:
	"""Install a tool from source."""

	@classmethod
	def install(cls, install_cmd):
		"""Install from source.

		Args:
			cls: ToolInstaller class.
			install_cmd (str): Install command.

		Returns:
			bool: True if install is successful, False otherwise.
		"""
		ret = Command.execute(install_cmd, cls_attributes={'shell': True})
		return ret.return_code == 0
@ocervell ocervell self-assigned this Apr 8, 2024
@ocervell ocervell added security Security issues. important labels Apr 8, 2024
@ocervell ocervell changed the title security: Possible command injection on install commands security(install): resolve possible injections Nov 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
important security Security issues.
Projects
None yet
Development

No branches or pull requests

1 participant