public/steve
6
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Make STEVE app upstream configurable #5

Closed
Ghost wants to merge 0 commits from refs/pull/5/head into main
pull from: refs/pull/5/head
merge into: public:main
Conversation 1 Commits - Files changed -
Ghost commented 2026-06-15 10:15:27 +00:00 (Migrated from codeberg.org)
Copy link

Caution platform counterpart: https://codeberg.org/caution/platform/pulls/271

steve has the app upstream hardcoded to 127.0.0.1:8083.

That means when e2e: true is enabled, traffic goes:

Caddy -> STEVE :49500 -> app :8083

even if the app’s Procfile says something else like:

ports: 3000
http_port: 3000
e2e: true

So any e2e app that doesn’t listen on 8083 just fails behind steve, usually as some vague upstream/proxy error. This is confusing because the docs/config imply the app can use any unreserved port and http_port should define the HTTP app port.

Steve should take its app upstream from config/env, e.g. STEVE_APP_UPSTREAM, and the enclave startup should set that from the app’s http_port. 8083 can stay as a fallback for compatibility, but it shouldn’t be the only working app port.

> Caution platform counterpart: https://codeberg.org/caution/platform/pulls/271 steve has the app upstream hardcoded to `127.0.0.1:8083`. That means when `e2e: true` is enabled, traffic goes: ```text Caddy -> STEVE :49500 -> app :8083 ``` even if the app’s `Procfile` says something else like: ```yaml ports: 3000 http_port: 3000 e2e: true ``` So any e2e app that doesn’t listen on `8083` just fails behind steve, usually as some vague upstream/proxy error. This is confusing because the docs/config imply the app can use any unreserved port and `http_port` should define the HTTP app port. Steve should take its app upstream from config/env, e.g. `STEVE_APP_UPSTREAM`, and the enclave startup should set that from the app’s `http_port`. `8083` can stay as a fallback for compatibility, but it shouldn’t be the only working app port.
Ghost commented 2026-07-22 16:11:40 +00:00 (Migrated from codeberg.org)
Copy link

already merged

already merged

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
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
public/steve!5
No description provided.