Ivan Petrov

Results 40 comments of Ivan Petrov

We also have the following proto directories that could also be moved: - [`oak_containers/proto/`](https://github.com/project-oak/oak/tree/main/oak_containers/proto) - Could be moved into the recently added [`proto/containers/`](https://github.com/project-oak/oak/tree/main/proto/containers) - I also think we could split...

> @ipetr0v is the plan to eventually delete the session protos? I think we'll still use session protos for some of our use-cases. But these protos will no longer contain...

> Just want to check about this -- @ipetr0v @k-naliuka is it correct that we expect the client SDKs eventually to work against any gRPC streaming method taking RequestWrapper and...

Yes, the clients need to own their own code (though I still think it may be convenient for some clients to just rely on the default services/transport implementations). As for...

Ref https://github.com/project-oak/oak/issues/3442

Yes, it's done The library and the corresponding client code can be found here: - https://github.com/project-oak/oak/tree/main/oak_attestation_verification - https://github.com/project-oak/oak/tree/main/oak_client But right now we don't have C++ and Java bindings in open-source

Should we close this, since Github repo is now read-only?

As per Rust SDK we were planning to return only the decrypted message from the `decrypt` function (i.e. a vec). And since the `EncryptedMessage` already contains an associated data, we...

We might also want to look into the JNI Java to C++ string conversion and remove unnecessary copies (if possible): https://github.com/project-oak/oak/blob/5eba73f3f09a150af35cf8778f3a9917d45e0e83/cc/crypto/hpke/jni/jni_helper.cc#L23-L30

Is this error produced by an internal `clang-tidy` or by the one added in https://github.com/project-oak/oak/pull/4634 ? I'm trying to run `bazel build --config=clang-tidy //cc/...` locally and I don't get this...