Skip to content

Conversation

zardus
Copy link
Member

@zardus zardus commented Sep 5, 2025

The existing simple image is too weird. Python in /usr/local/bin? That's lame. Let's use use ubuntu?

@ConnorNelson
Copy link
Member

I wanted just python and exec-suid in this image, and I wanted the image to be tiny.

Probably this is decently bigger, and also suddenly it comes with two python packages. Should we install anything else too? Or create another Dockerfile? :)

Nix nix nix.

On a more serious note (or at least on a less poke poke poke note), this repo’s contents/purpose now also belongs in pwncollege/challenges.

@zardus
Copy link
Member Author

zardus commented Sep 5, 2025

I removed the two packages. I do think that having python randomly in /usr/local/bin has challenge impact, and while I agree that eventually (very soon!) this repo should no longer be necessary, I think we should have an image here that has python in /usr/bin. I think we'll survive the extra 30mb or so of the image :-)

@ConnorNelson
Copy link
Member

Yeah, beyond size though, we would need to PR anything currently using this image to change the path to the python interpreter from /usr/local/bin/python to /usr/bin/python, e.g.:

https://github.com/pwncollege/example-dojo/blob/10b0b68c57b87c7b963b50e01f279797e10d0c4c/hello/apple/apple#L1

@ConnorNelson
Copy link
Member

To be fair, after a very quick glance, that might be the only spot.

@zardus
Copy link
Member Author

zardus commented Sep 5, 2025

To be safe, I added a symlink for backwards compatibility!

@snowy-fluff
Copy link

I'm curious is there a reason behind not using python in /usr/local/bin/python? Without it wouldn't pip install packages directly to /usr/bin/python, aka system python?

@zardus zardus merged commit 504b88b into main Oct 14, 2025
@zardus
Copy link
Member Author

zardus commented Oct 14, 2025

It's time to YOLO!!!!!

@ConnorNelson
Copy link
Member

This just broke CI @zardus

@ConnorNelson
Copy link
Member

It also broke at least 2 people's private dojos.

zardus added a commit that referenced this pull request Oct 15, 2025
@zardus
Copy link
Member Author

zardus commented Oct 15, 2025

Rolled back for now :-)

zardus added a commit that referenced this pull request Oct 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants