search

CCC-Attestation / interoperable-ra-tls

Design documents and interoperability tests for Interoperable RA-TLS projects
Apache License 2.0
10 stars 3 forks source link

Add interoperability test between Gramine and rats-tls #10

Closed imlk0 closed 1 year ago

imlk0 commented 1 year ago

Closes #8

imlk0 commented 1 year ago

@shnwc @dimakuv @jiazhang0 Please feel free to provide any feedback or comments on this PR. Thanks!

imlk0 commented 1 year ago

The cert files have been updated with new CBOR tag values.

dcmiddle commented 1 year ago

thanks @KB5201314 ! fyi @dimakuv @shnwc

shnwc commented 1 year ago

Great job @KB5201314 and @dimakuv !

dimakuv commented 1 year ago

I checked:

  1. Both certificates are correct (examined via openssl x509 -in gramine-cert.pem -text -noout -certopt ext_parse).
  2. The description is correct:
    • Gramine description is definitely correct;
    • Occlum/RATS-TLS description looks fine, though I didn't test the steps.

I would amend the descriptions a bit, but the general flow is good. Anyway, the PR was already merged, so I won't nitpick.

shnwc commented 1 year ago

@dimakuv feel free to create a new PR to amend the description.