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

fix(python): Support binding to ports less than 1024 #41381

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

EyeCantCU
Copy link
Member

Introduces subpackage with capabilities set to bind to privileged ports

Currently needed by pgAdmin

Dentrax
Dentrax previously approved these changes Feb 5, 2025
@octo-sts octo-sts bot added the bincapz/pass bincapz/pass Bincapz (aka. malcontent) scan didn't detect any CRITICALs on the scanned packages. label Feb 5, 2025
@kranurag7 kranurag7 force-pushed the python-privileged-bind branch from c162f7c to 51061d0 Compare February 11, 2025 18:17
python-3.12.yaml Outdated
- libcap-utils
pipeline:
- runs: |
getcap "/usr/bin/${{vars.python}}" | grep "cap_net_bind_service=eip"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Love this @EyeCantCU

Introduces subpackage with capabilities set to bind to privileged ports

Currently needed by pgAdmin

Signed-off-by: RJ Sampson <rj.sampson@chainguard.dev>
@EyeCantCU EyeCantCU force-pushed the python-privileged-bind branch from 8b217e7 to 0c6917e Compare February 11, 2025 23:28
Copy link
Member

@kranurag7 kranurag7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you RJ, do we want to the changes around var-transforms of using "" inside python-3.13 as well 💭

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bincapz/pass bincapz/pass Bincapz (aka. malcontent) scan didn't detect any CRITICALs on the scanned packages.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants