Skip to content

Conversation

@avktech78
Copy link

No description provided.

@anatol
Copy link
Owner

anatol commented Sep 18, 2025

Thank you for the change! TPM2 support is very much welcome.

One thing that I would really love to see is having a good integration test for this functionality. Could you please describe the sequence of steps to reproduce the original issue, so I can codify it as a test case.

@avktech78
Copy link
Author

The thing is, I've never programmed in Go, but I really missed having this feature. At first, I used a separate old version of systemd-cryptenroll. But at some point, I decided to try updating your code to support tpm2_srk.
So I used ChatGPT for that. At the time of publication, I still remembered the details, but now I can hardly remember anything.
I'm sorry, I understand that it's important for a developer to have a well-tested product, but all I can say is that this patch works on my computer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants