Skip to content

Commit

Permalink
Add venv path to firejail whitelist
Browse files Browse the repository at this point in the history
Since venvs were moved out of assignment folders in 0575701, they now need to be explicitly whitelisted. This does not remove their read-only status.
  • Loading branch information
krishnans2006 committed Mar 29, 2024
1 parent ac6926d commit 22965a3
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions tin/apps/submissions/tasks.py
Original file line number Diff line number Diff line change
Expand Up @@ -133,6 +133,7 @@ def run_submission(submission_id):
whitelist = [os.path.dirname(grader_path)]
read_only = [grader_path, submission_path, os.path.dirname(submission_wrapper_path)]
if submission.assignment.venv_fully_created:
whitelist.append(submission.assignment.venv.path)
read_only.append(submission.assignment.venv.path)

args = sandboxing.get_assignment_sandbox_args(
Expand Down

0 comments on commit 22965a3

Please sign in to comment.