-
Notifications
You must be signed in to change notification settings - Fork 91
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
AA: use URL safe base64 encoding for TeePubKey #177
AA: use URL safe base64 encoding for TeePubKey #177
Conversation
Thanks @katexochen . I guess this PR will have to be modified with the new kbs-types release. Otherwise LGTM. |
@sameo I have that change prepared, but I'm still waiting for the crate to be published: https://crates.io/crates/kbs-types/versions |
It seems that we have a unit test failure @katexochen |
@sameo The failing test isn't related to my change, right? This seems to be the failure mentioned in https://app.slack.com/client/T08PSQ7BQ/C050S70D9SR/thread/C050S70D9SR-1688691629.377139 |
Hi @katexochen , could you please rebase the code? I think the error here has been fixed. |
Signed-off-by: Paul Meyer <49727155+katexochen@users.noreply.github.com>
Signed-off-by: Paul Meyer <49727155+katexochen@users.noreply.github.com>
Port of confidential-containers/attestation-agent#211
Fixes #184