Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pre-check/Birdcage permission error #1068

Open
furi0us333 opened this issue May 4, 2023 · 3 comments
Open

Pre-check/Birdcage permission error #1068

furi0us333 opened this issue May 4, 2023 · 3 comments
Labels
bug Something isn't working medium priority Should be handled as soon as possible

Comments

@furi0us333
Copy link
Contributor

Overview

I am getting the following error when trying to use phylum with pip: PermissionError: [Errno 13] Permission denied: '/etc/debian_version'

How To Reproduce

Steps to reproduce this behavior:

  1. Given the system information below, run phylum pip install "pyyaml==5.3.1"
  2. See error

Expected Behavior

Phylum pre-check/birdcage run without the exception.

Additional Context

System details:

$ python3 -m pip list
Package       Version
------------- -------
Pillow        9.5.0
pip           23.1.2
pkg_resources 0.0.0
setuptools    44.0.0

$ python3 -c 'import sys; print(sys.base_exec_prefix)'
/usr

$ uname -a
Linux DESKTOP-RQ0PRHE 5.15.90.1-microsoft-standard-WSL2 #1 SMP Fri Jan 27 02:56:13 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

$ phylum version
✅ phylum (Version v5.0.1)
@furi0us333 furi0us333 added bug Something isn't working needs triage Needs to be reviewed or assigned labels May 4, 2023
@kylewillmon kylewillmon added medium priority Should be handled as soon as possible and removed needs triage Needs to be reviewed or assigned labels May 9, 2023
@kylewillmon
Copy link
Contributor

kylewillmon commented Jun 21, 2023

Unfortunately, it appears that pip install "pyyaml==5.3.1" on WSL goes through the process of compiling pyyaml from source...

To allow compiling C code, we would need to significantly weaken our sandbox.

@louislang
Copy link
Contributor

this seems like a pretty common thing people are likely to encounter. We should document this and perhaps consider adding some sort of escape hatch, e.g., Sandbox prevented package from compiling. Compile outside of the sandbox? [y/N]:

@cd-work
Copy link
Contributor

cd-work commented Jun 23, 2023

I feel like that would just completely circumvent the whole purpose of sandboxing it in the first place?

@kylewillmon kylewillmon removed their assignment Aug 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working medium priority Should be handled as soon as possible
Projects
None yet
Development

No branches or pull requests

4 participants