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

KES User Interface #319

Closed
wants to merge 80 commits into from
Closed

KES User Interface #319

wants to merge 80 commits into from

Conversation

reivaj05
Copy link
Contributor

@reivaj05 reivaj05 commented Jan 19, 2023

To Test this PR

  1. Run KES Server
  2. make assets
  3. make kes
  4. run ./kes ui

@reivaj05 reivaj05 mentioned this pull request Jan 19, 2023
@reivaj05 reivaj05 marked this pull request as draft January 19, 2023 21:44
aead and others added 6 commits March 28, 2023 15:43
This commit adds the ability to customize the
KES server API.

Now the KES server config file contains an `api`
section:
```
api:
  <api-path>:
    skip_auth: bool
    timeout:   time.Duration
```

The KES server uses sane defaults for all APIs.
Further, all APIs except `/version` and
`/v1/identity/self/describe` require authentication
by default.

In general, authentication should not be disabled.
Under certain circumstances, disabling authentication
may be an option to simplify the integration of e.g.
monitoring.

At the moment, authentication can only be turned
of for the following APIs:
 - `/v1/metrics`
 - `/v1/status`
 - `/v1/api`

Now, all APIs returned by the `/v1/api` API also
report whether authentication is enabled or disabled.

Signed-off-by: Andreas Auernhammer <[email protected]>
This commit introduces two new packages:
 - `kv`
 - `edge`

The `edge` package implements decoding KES
edge server configuration into Go types.

The `kv` package implements generic abstractions
for key stores used by the KES edge server.

Further, the `edge` package contains additional tests
for decoding KES configuration.

Signed-off-by: Andreas Auernhammer <[email protected]>
@allanrogerr
Copy link

@reivaj05 Is this still required?

@harshavardhana
Copy link
Member

I am closing this PR as stale and outdated.

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.

7 participants