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
Use token enclave public key when in privacy multi-tenancy mode #272
Use token enclave public key when in privacy multi-tenancy mode #272
Conversation
Signed-off-by: Jason Frame <jasonwframe@gmail.com>
Signed-off-by: Jason Frame <jasonwframe@gmail.com>
…oller Signed-off-by: Jason Frame <jasonwframe@gmail.com>
Signed-off-by: Jason Frame <jasonwframe@gmail.com>
Signed-off-by: Jason Frame <jasonwframe@gmail.com>
Signed-off-by: Jason Frame <jasonwframe@gmail.com>
Signed-off-by: Jason Frame <jasonwframe@gmail.com>
|
||
public class PrivFindPrivacyGroupTest { | ||
private static final String ENCLAVE_PUBLIC_KEY = "A1aVtMxLCUHmBVHXoZzzBgPbW/wj5axDpW9X8l91SGo="; | ||
private static final List<String> ADDRESSES = |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we have List.of available as well in core Java.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me
Signed-off-by: Jason Frame <jasonwframe@gmail.com>
…he enclave public key resolution out of the privacy controller Signed-off-by: Jason Frame <jasonwframe@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ignoring the red-cross, I think this looks good.
…rledger#272) Signed-off-by: Jason Frame <jasonwframe@gmail.com>
…rledger#272) Signed-off-by: Jason Frame <jasonwframe@gmail.com> Signed-off-by: edwardmack <ed@edwardmack.com>
PR description
This provides the multi-tenancy enclave public key from the request when the user is using privacy multi-tenancy. Also changes the API of the PrivacyController to now have an optional enclave public key. This will allow the privacy controller in the future to be able to verify enclave operation is allowed for the supplied enclave public key.
Fixed Issue(s)