Skip to content

Commit

Permalink
Ignore B404 issue on update.py
Browse files Browse the repository at this point in the history
  • Loading branch information
ericof committed Nov 8, 2023
1 parent 4311c19 commit 4f4714e
Showing 1 changed file with 3 additions and 12 deletions.
15 changes: 3 additions & 12 deletions src/pas/plugins/oidc/locales/update.py
Original file line number Diff line number Diff line change
Expand Up @@ -38,10 +38,7 @@ def locale_folder_setup(domain: str):
f"--input={locale_path}/{domain}.pot "
f"--output={locale_path}/{lang}/LC_MESSAGES/{domain}.po"
)
subprocess.call(
cmd,
shell=True,
)
subprocess.call(cmd, shell=True) # nosec B404

Check failure on line 41 in src/pas/plugins/oidc/locales/update.py

View workflow job for this annotation

GitHub Actions / Bandit comments

Test: subprocess_popen_with_shell_equals_true id: B602

subprocess call with shell=True identified, security issue. more info https://bandit.readthedocs.io/en/1.7.5/plugins/b602_subprocess_popen_with_shell_equals_true.html


def _rebuild(domain: str):
Expand All @@ -50,21 +47,15 @@ def _rebuild(domain: str):
f"--exclude {excludes} "
f"--create {domain} {target_path}"
)
subprocess.call(
cmd,
shell=True,
)
subprocess.call(cmd, shell=True) # nosec B404

Check failure on line 50 in src/pas/plugins/oidc/locales/update.py

View workflow job for this annotation

GitHub Actions / Bandit comments

Test: subprocess_popen_with_shell_equals_true id: B602

subprocess call with shell=True identified, security issue. more info https://bandit.readthedocs.io/en/1.7.5/plugins/b602_subprocess_popen_with_shell_equals_true.html


def _sync(domain: str):
cmd = (
f"{i18ndude} sync --pot {locale_path}/{domain}.pot "
f"{locale_path}/*/LC_MESSAGES/{domain}.po"
)
subprocess.call(
cmd,
shell=True,
)
subprocess.call(cmd, shell=True) # nosec B404

Check failure on line 58 in src/pas/plugins/oidc/locales/update.py

View workflow job for this annotation

GitHub Actions / Bandit comments

Test: subprocess_popen_with_shell_equals_true id: B602

subprocess call with shell=True identified, security issue. more info https://bandit.readthedocs.io/en/1.7.5/plugins/b602_subprocess_popen_with_shell_equals_true.html


def update_locale():
Expand Down

0 comments on commit 4f4714e

Please sign in to comment.