Skip to content

Implement proper vhost#336

Merged
LeoColomb merged 1 commit intomainfrom
feat/vhost
May 26, 2025
Merged

Implement proper vhost#336
LeoColomb merged 1 commit intomainfrom
feat/vhost

Conversation

@LeoColomb
Copy link
Member

@LeoColomb LeoColomb commented May 25, 2025

Warning

Breaking change: the default exposed ports are now 8080 and 8443.

Ports 80 and 443 are outside userland Linux kernel boundaries.
If the container is run as a rootless process in a rather strict environment, it cannot bind these ports.
Kernel capabilities can be requested to gain access (for instance, the CAP_NET_BIND_SERVICE capability), but additional capabilities are considered security weaknesses.

Such strict environments become more and more common, both from container engines (Podman, for example) and from orchestrators (OpenShift is super strict about this).

Ports 8080 and 8443 are fairly common for such apps nowadays, and this change just makes everything simpler for everyone with virtually zero impact on usability.

@LeoColomb LeoColomb force-pushed the feat/vhost branch 4 times, most recently from 0ca07b3 to 1406ac9 Compare May 26, 2025 09:47
@LeoColomb LeoColomb merged commit 5975414 into main May 26, 2025
26 of 31 checks passed
@LeoColomb LeoColomb deleted the feat/vhost branch May 26, 2025 09:58
pooneyy added a commit to pooneyy/1Panel-Appstore that referenced this pull request Oct 28, 2025
- change port mapping from 80 to 8080 for both 1.10.2 and latest versions
Reference: YOURLS/containers/pull/336
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.

1 participant