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

MachineID is confidential and should not be exposed on the network #86

Open
cmbaron opened this issue Apr 17, 2023 · 1 comment
Open
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@cmbaron
Copy link

cmbaron commented Apr 17, 2023

The Hostname Plugin allows using MachineID in a template to generate hostname.

Per the manual page and the machine id library used by this project, machine-id is confidential and shouldn't be disclosed in whole or part.

How do we feel about adding "SecuredMachineID" as an available template variable in the hostname plugin to generate a machine-id derived ID using machine.id.ProtectedID("yipCloudInitHostnameConfiguration"), then later deprecating "MachineID" with a warning when its used?

@mudler
Copy link
Owner

mudler commented Apr 17, 2023

Hey @cmbaron,

very good point, this should have been hashed directly from the start - what you propose sounds good actually, I'd be even tempted to say to hash directly the MachineID so it's safe to use by default and we avoid misuse, altough if someone already relied on it that would be bad - so it's best to split those two, and make a deprecation warning as you suggest.

However, there might be use-cases where it is used just internally, and not exposed at all, so maybe a warning would be enough to make sure that the user understand implications of using it.

@mudler mudler moved this to Under review 🔍 in 🧙Issue tracking board Apr 17, 2023
@mudler mudler moved this from Under review 🔍 to Todo 🖊 in 🧙Issue tracking board Apr 17, 2023
@mudler mudler added enhancement New feature or request help wanted Extra attention is needed good first issue Good for newcomers labels Jul 4, 2023
@mudler mudler moved this from Todo 🖊 to Under review 🔍 in 🧙Issue tracking board Jul 4, 2023
@mudler mudler moved this from Under review 🔍 to Todo 🖊 in 🧙Issue tracking board Jul 4, 2023
@mudler mudler moved this from Todo 🖊 to Incoming in 🧙Issue tracking board Sep 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants