Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cmd: rework server startup message #255

Merged
merged 2 commits into from Jul 13, 2022
Merged

cmd: rework server startup message #255

merged 2 commits into from Jul 13, 2022

Conversation

aead
Copy link
Member

@aead aead commented Jul 13, 2022

This commit separates the KES server/gateway
code and reworks the startup message.

Now, the lipgloss tui framework is used instead
of the previous github.com/fatih/color package.

A subsequent commit will remove the
github.com/fatih/color completely by refactoring
the kes migrate command.

$ kes server --config config.yml --auth off
Copyright  MinIO, Inc.  https://min.io/
License    GNU AGPLv3   https://www.gnu.org/licenses/agpl-3.0.html
Version    v0.0.0-dev   darwin/arm64

KMS        Filesystem: /tmp/keys
Endpoints  https://127.0.0.1:7373/
           https://192.168.188.52:7373/

Admin      3ecfcdf38fcbe141ae26a1030f81e96b753365a46760ae6b578698a97c59fd22
mTLS       skip verify  Client certificates are not verified
Mem Lock   off          Not supported on darwin/arm64

@aead aead requested a review from vadmeste July 13, 2022 11:24
This commit separates the KES server/gateway
code and reworks the startup message.

Now, the lipgloss tui framework is used instead
of the previous `github.com/fatih/color` package.

A subsequent commit will remove the
`github.com/fatih/color` completely by refactoring
the `kes migrate` command.

```
$ kes server --config config.yml --auth off
Copyright  MinIO, Inc.  https://min.io
License    GNU AGPLv3   https://www.gnu.org/licenses/agpl-3.0.html
Version    v0.0.0-dev   darwin/arm64

KMS        Filesystem: /tmp/keys
Endpoints  https://127.0.0.1:7373
           https://192.168.188.52:7373

Admin      3ecfcdf38fcbe141ae26a1030f81e96b753365a46760ae6b578698a97c59fd22
mTLS       skip verify  Client certificates are not verified
Mem Lock   off          Not supported on darwin/arm64
```

Signed-off-by: Andreas Auernhammer <hi@aead.dev>
@aead
Copy link
Member Author

aead commented Jul 13, 2022

Screenshot 2022-07-13 at 13 27 23

@harshavardhana harshavardhana merged commit 2b3a66b into master Jul 13, 2022
@harshavardhana harshavardhana deleted the startup-tui branch July 13, 2022 21:00
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.

None yet

2 participants