Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

the Purpose enum is poorly named #150

Open
flihp opened this issue May 5, 2023 · 0 comments
Open

the Purpose enum is poorly named #150

flihp opened this issue May 5, 2023 · 0 comments

Comments

@flihp
Copy link
Collaborator

flihp commented May 5, 2023

This enum was a quick & dirty way for a KeySpec to pass a configuration value to the ca initialize command. The mechanism is fine, but the name is all wrong. I'm tempted to split all of the CA config off into a new type CaSpec, but it'll need to be linked to the HSM probalby through the label. That fragility can be avoided by just keeping it all in once structure. Probably not worth the effort either.

At the very least, Purpose should be given a better name. Something like CaSection?

@flihp flihp changed the title the Purpose enum has outlived its usefulness the Purpose enum is poorly named May 5, 2023
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

No branches or pull requests

1 participant