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

Notify runtimes of its key manager policy updates #2919

Closed
kostko opened this issue May 18, 2020 · 0 comments · Fixed by #2947
Closed

Notify runtimes of its key manager policy updates #2919

kostko opened this issue May 18, 2020 · 0 comments · Fixed by #2947
Assignees
Labels
c:key management Category: key management c:runtime Category: runtime

Comments

@kostko
Copy link
Member

kostko commented May 18, 2020

Currently runtimes are unaware of any key manager policy updates. For example, the runtime (simple-keyvalue) used in E2E tests initializes its key manager policy on startup and passes it to the key manager client.

If the key manager policy changes while the runtime is running (e.g., due to the key manager being upgraded, see #2517), the runtime is not notified and keeps using the old policy. This will cause EnclaveRPC sessions to fail when enclave identity changes.

One possible way of resolving this is extending the Runtime Host Protocol (RHP) with a method for notifying the runtime of a new key manager policy:

  • Currently, the runtime can query the policy at any point by sending a HostKeyManagerPolicyRequest message over RHP.
  • We could turn this around (removing the above message) and introduce a RuntimeKeyManagerPolicyUpdateRequest that the host uses to notify the runtime of a new policy.

Watching policy updates and propagating them to the runtime should most likely be part of the common runtime host handler (see go/worker/common/committee/runtime_host.go).

@kostko kostko added c:key management Category: key management c:runtime Category: runtime labels May 18, 2020
@ptrus ptrus self-assigned this May 18, 2020
@ptrus ptrus mentioned this issue May 18, 2020
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c:key management Category: key management c:runtime Category: runtime
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants