Skip to content

Display auto-assigned external port for TCP/UDP services#397

Merged
runleveldev merged 1 commit into
mainfrom
feature/display-transport-port
Jul 9, 2026
Merged

Display auto-assigned external port for TCP/UDP services#397
runleveldev merged 1 commit into
mainfrom
feature/display-transport-port

Conversation

@runleveldev

Copy link
Copy Markdown
Collaborator

Part of #360 (split out for a small, self-contained review).

What

TCP/UDP transport services get an auto-assigned external port. The value is already stored and serialized by the API, but it was never shown in the UI. This surfaces it (read-only) in each TCP/UDP service row of the container form. New, unsaved services show "Auto-assigned on save".

Scope

Frontend only — a single change to ContainerFormPage.tsx:

  • add externalPort to the service form schema,
  • populate it from transportService.externalPort when editing,
  • render a read-only "External port" line for tcp/udp services.

No backend, model, migration, or API changes.

Notes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the container “Services” form UI to surface the auto-assigned external port for TCP/UDP transport services (read-only), using data that is already returned by the API but previously not displayed.

Changes:

  • Extend the client-side service form schema to include an optional externalPort.
  • Populate externalPort from transportService.externalPort when initializing the edit form.
  • Render a read-only “External port” line for tcp/udp services, showing “Auto-assigned on save” for new/unsaved services.

The external port for TCP/UDP transport services is auto-assigned by the
backend and already serialized by the API, but it was never shown in the
UI. Surface it (read-only) in each TCP/UDP service row of the container
form; new, unsaved services show "Auto-assigned on save".
@runleveldev runleveldev force-pushed the feature/display-transport-port branch from 1e26a00 to 206b187 Compare July 9, 2026 17:18
@runleveldev runleveldev merged commit 28ca600 into main Jul 9, 2026
8 checks passed
@runleveldev runleveldev deleted the feature/display-transport-port branch July 9, 2026 19:30
@runleveldev runleveldev linked an issue Jul 9, 2026 that may be closed by this pull request
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.

[Bug]: No way to find non-ssh TCP port

3 participants