[PR #4480] [MERGED] feat: ability for aio in sub-path access configurations to specify alternate ports #4823

Closed
opened 2026-03-17 02:19:30 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/hoppscotch/hoppscotch/pull/4480
Author: @AndrewBastin
Created: 10/25/2024
Status: Merged
Merged: 10/28/2024
Merged by: @AndrewBastin

Base: nextHead: feat/aio-alternate-port


📝 Commits (1)

  • 88ddf5f feat: add ability for aio containers to specify alternate binding ports

📊 Changes

3 files changed (+3 additions, -17 deletions)

View changed files

📝 aio-multiport-setup.Caddyfile (+0 -4)
📝 aio-subpath-access.Caddyfile (+1 -13)
📝 prod.Dockerfile (+2 -0)

📄 Description

This PR allows for users to specify alternate ports for AIO container configurations having sub-path access (single port) configurations. The container normally binds port 80 when using single port access, but port 80 is a privileged port and certain systems (esp. rootless systems like Podman or hardened systems like OpenShift) do not provide root level privileges to allow for binding to port 80.

What's changed

  • Added new optional environment variable which is read by AIO container HOPP_AIO_ALTERNATE_PORT which is used by Caddy to define which port is to be bound to. No behaviour change for non-AIO containers.
  • If the variable is not specified, Caddy will use the default port 80.

Notes

@SanskritiHarmukh do remind me to update the docs to add a section for this.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/hoppscotch/hoppscotch/pull/4480 **Author:** [@AndrewBastin](https://github.com/AndrewBastin) **Created:** 10/25/2024 **Status:** ✅ Merged **Merged:** 10/28/2024 **Merged by:** [@AndrewBastin](https://github.com/AndrewBastin) **Base:** `next` ← **Head:** `feat/aio-alternate-port` --- ### 📝 Commits (1) - [`88ddf5f`](https://github.com/hoppscotch/hoppscotch/commit/88ddf5febf4f4d9a82914d436d9280eadd6e80ac) feat: add ability for aio containers to specify alternate binding ports ### 📊 Changes **3 files changed** (+3 additions, -17 deletions) <details> <summary>View changed files</summary> 📝 `aio-multiport-setup.Caddyfile` (+0 -4) 📝 `aio-subpath-access.Caddyfile` (+1 -13) 📝 `prod.Dockerfile` (+2 -0) </details> ### 📄 Description This PR allows for users to specify alternate ports for AIO container configurations having sub-path access (single port) configurations. The container normally binds port 80 when using single port access, but port 80 is a privileged port and certain systems (esp. rootless systems like Podman or hardened systems like OpenShift) do not provide root level privileges to allow for binding to port 80. ### What's changed - Added new optional environment variable which is read by AIO container `HOPP_AIO_ALTERNATE_PORT` which is used by Caddy to define which port is to be bound to. No behaviour change for non-AIO containers. - If the variable is not specified, Caddy will use the default port 80. ### Notes @SanskritiHarmukh do remind me to update the docs to add a section for this. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-17 02:19:30 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/hoppscotch#4823
No description provided.