Skip to content

Commit

Permalink
make fixes suggested in PR #2472
Browse files Browse the repository at this point in the history
  • Loading branch information
cloud303-tfurlong committed Oct 2, 2023
1 parent e01dafe commit 7bd2d33
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -363,7 +363,7 @@ def _is_session_valid(user, session_id):
logger.info("Verifying NICE DCV session validity..")

# Query by uid rather than username to avoid truncation by ps command
uid = subprocess.check_output(["id", "-u", user]).decode("utf-8").strip()
uid = subprocess.check_output(["/usr/bin/id", "-u", user]).decode("utf-8").strip() # nosec B603

Check warning on line 366 in cookbooks/aws-parallelcluster-platform/files/dcv/pcluster_dcv_authenticator.py

View check run for this annotation

Codecov / codecov/patch

cookbooks/aws-parallelcluster-platform/files/dcv/pcluster_dcv_authenticator.py#L366

Added line #L366 was not covered by tests

# Remove the first and the last because they are the heading and empty, respectively
# All commands and arguments in this subprocess call are built as literals
Expand Down

0 comments on commit 7bd2d33

Please sign in to comment.