Skip to content

Switch helloworld client to fat JAR with --ssl option - #513

Merged
hpehl merged 5 commits into
wildfly-extras:mainfrom
hpehl:helloworld-fat-jar
Oct 8, 2026
Merged

hpehl merged 5 commits into
wildfly-extras:mainfrom
hpehl:helloworld-fat-jar

Conversation

@hpehl

@hpehl hpehl commented Oct 7, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Replace exec-maven-plugin with maven-shade-plugin to produce a self-contained helloworld-client.jar
  • Refactor argument parsing to use --ssl=<mode> option with name as positional arg, matching the chat client pattern from [#490] Replace JavaFX chat client with TamboUI terminal UI #511
  • Extract SSL channel creation logic into SslMode enum
  • Update README and CLAUDE.md with new usage instructions

Test plan

  • Build with mvn package -pl examples/helloworld/client -am
  • Run java -jar target/helloworld-client.jar Bob against a running server
  • Verify --ssl=oneway and --ssl=twoway modes work
  • Verify --help prints usage

Replace exec-maven-plugin with maven-shade-plugin to produce a
self-contained helloworld-client.jar. Refactor argument parsing to
use --ssl=<mode> option with name as positional arg, matching the
chat client pattern. Extract SSL channel logic into SslMode enum.
This produces a fat JAR that can be launched directly:

```shell
# clear text connection

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think we should still keep the grpcurl commands as an alternative to the client

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

grpcurl commands are back in the REAMDE.md

@jmesnil jmesnil linked an issue Oct 8, 2026 that may be closed by this pull request
@hpehl
hpehl merged commit 9cada6a into wildfly-extras:main Oct 8, 2026
3 checks passed
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.

Update the examples to streamline their uses

2 participants