Skip to content

Add clickable server URL to dev mode port info output - #533

Merged
venmanyarun merged 1 commit into
OpenLiberty:1.xfrom
venmanyarun:print_welcome_page_sync
Aug 14, 2026
Merged

Add clickable server URL to dev mode port info output#533
venmanyarun merged 1 commit into
OpenLiberty:1.xfrom
venmanyarun:print_welcome_page_sync

Conversation

@venmanyarun

@venmanyarun venmanyarun commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

When dev mode starts or the user presses 'p', print a "Liberty welcome page:" URL alongside each HTTP/HTTPS port line so users can navigate directly to the running server.

  • Non-container: prints http(s)://:/ after each port line
  • Container: prints http(s)://localhost:/ when a port mapping exists
  • Expose printPortInfo as package-private and add protected setters for port fields to support unit testing without Docker
  • Add DevUtilPrintPortInfoTest covering server and container scenarios

Fixes #528

main PR link :- #529

When dev mode starts or the user presses 'p', print a "Liberty welcome
page:" URL alongside each HTTP/HTTPS port line so users can navigate
directly to the running server.

- Non-container: prints http(s)://<hostName>:<port>/ after each port line
- Container: prints http(s)://localhost:<mappedPort>/ when a port mapping exists
- Expose printPortInfo as package-private and add protected setters for
  port fields to support unit testing without Docker
- Add DevUtilPrintPortInfoTest covering server and container scenarios

Fixes OpenLiberty#528
@venmanyarun
venmanyarun requested a review from sajeerzeji August 14, 2026 06:19

@sajeerzeji sajeerzeji 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.

LGTM

@venmanyarun
venmanyarun requested a review from cherylking August 14, 2026 06:44
@venmanyarun
venmanyarun merged commit e78f20a into OpenLiberty:1.x Aug 14, 2026
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.

3 participants