yuvipanda / jupyterhub-ssh

SSH Access to JupyterHubs
BSD 3-Clause "New" or "Revised" License
93 stars 29 forks source link

sftp: fix shebang using user idependent path #60

Closed consideRatio closed 3 years ago

consideRatio commented 3 years ago

59 updated a shebang, but didn't account for #!/usr/bin/env python3 would fail if the user invoking the script was some user that didn't have python3 on path.

This PR fixes that mistake by making the script reference /usr/bin/python3 directly no matter what.

Related reading on why to use /usr/bin/env python3 over /usr/bin/python3: https://unix.stackexchange.com/questions/29608/why-is-it-better-to-use-usr-bin-env-name-instead-of-path-to-name-as-my