confidential-containers / simple-kbs

Key Broker Server for SEV(-ES)
Apache License 2.0
12 stars 13 forks source link

Dunnderr missing keys logging #48

Closed dunnderr closed 1 year ago

dunnderr commented 1 year ago

This pull request addresses issue #33 to provide better error handing for missing keysets. A warning will be logged when no keyset ids are returned from db::get_keyset_ids.

dubek commented 1 year ago

Please add a unit-test to verify this scenario (test that we return an empty Vec<String> in such a case).

dunnderr commented 1 year ago

Hi Dov, I will do this and expand the check to cover returning an empty string and to catch parse errors.