Hacktoberfest 2026: the issues maintainers tagged for October, open and beginner-friendly. Browse Hacktoberfest issues

BREAKING: Rework run() arguments

Open
#6 1 comment 0 reactions 1 assignee View on GitHub

@jmgate is already working on this.

Since May 9, 2024.

Assessment

This issue has not been assessed yet.

Description

Type: Refactor

Tasks

  • Revert 520b68e18c7e492ecec529d4d0a64e2f04b180af.
  • Switch the first argument to the run() method from a string to a list of strings to protect against command injection in the shell.
  • Make any associated changes to bring everything in sync.

Note: This will be a breaking change, but it will be worthwhile to remove the possibility of this unsafe behavior.

Dominant language
Python
Stars
23
Forks
2
Avg merge
5d 4h
Merged PRs (30d)
3

Getting set up

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from sandialabs/staged-script

All issues in sandialabs/staged-script

Similar issues

More Python issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.