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

Added an optional extra variable (a map) for machine identification #2

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Sadi-a
Copy link

@Sadi-a Sadi-a commented Mar 29, 2023

A MAC address is only composed of 48 bits of which the first 24 bits can be determined as they are the OUI, which really only leaves about 24 random bits. Furthermore, a MAC address can easily be spoofed, which means using only the MAC for machine identification may become a vulnerability. This commit concerns the bare-metal/flatcar-linux folder and adds an optional field in the controllers and workers variables as well as a variable in the worker/variables.tf file. It defaults on {}. For example, it can be used to add a field based on the machine's uuid, which is 128 bits long, and thus make bruteforcing much harder (48 bits --> 48 * 128 bits).

A MAC address is only composed of 48 bits of which the first 24 bits can be determined as they are the OUI, which really only leaves about 24 random bits.
Furthermore, a MAC address can easily be spoofed, which means using only the MAC for machine identification may become a vulnerability.
This commit concerns the bare-metal/flatcar-linux folder and adds an optional field in the controllers and workers variables as well as a variable in the worker/variables.tf file. It defaults on {}. For example, it can be used to add a field based on the machine's uuid, which is 128 bits long, and thus make bruteforcing much harder (48 bits --> 48 * 128 bits).
@Sadi-a Sadi-a self-assigned this Apr 17, 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

Successfully merging this pull request may close these issues.

1 participant