search

linux-system-roles / nbde_client

Ansible role for configuring Network Bound Disk Encryption clients (e.g. clevis)
https://linux-system-roles.github.io/nbde_client/
MIT License
14 stars 24 forks source link

fix temp pass key test #14

Closed richm closed 4 years ago

richm commented 4 years ago

The test needs to copy the original temporary key file to the managed host to see if it can still be used to unlock. Check that the error returned from verify is the error that says the key did not work.

pcahyna commented 4 years ago

lgtm. Is there a bug which was not detected by the previous version of the test and gets detected by the updated version, or is the change just in order to be sure?

sergio-correia commented 4 years ago

lgtm. Thanks for finding this.

richm commented 4 years ago

lgtm. Is there a bug which was not detected by the previous version of the test and gets detected by the updated version, or is the change just in order to be sure?

No, there was not a bug being masked or missed. The code was/is working fine.