fleetdm / fleet

Open-source platform for IT, security, and infrastructure teams. (Linux, macOS, Chrome, Windows, cloud, data center)
https://fleetdm.com
Other
3.11k stars 430 forks source link

Hide enroll secret by default #4805

Open GuillaumeRoss opened 2 years ago

GuillaumeRoss commented 2 years ago

Goal

As a user taking a screenshot of the Hosts > Add Hosts modal, I want Fleet to save me from accidentally sharing my enroll secret publicly.

Figma

TODO

noahtalerman commented 2 years ago

@GuillaumeRoss we decided to deprioritize this improvement.

Reasoning:

In the current experience, the user can see the entire command (realize that it's a bash command they have to run in the terminal) and copy the exact command they see.

One solution is to cover only the enroll secret in the fleetctl package command. This makes the experience of copying the command awkward. As a user I might think the masked characters (•) are included in what I copy. Do I have to replace them?

There are other ways to solve this^. However, we've decided to not prioritize thinking about how we can accomplish this.

Please let me know if you have any concerns.

noahtalerman commented 2 years ago

@RachelElysia @fx5 FYI this issue tracks the request you brought to product office hours on 2022-09-29.

noahtalerman commented 2 years ago

@RachelElysia @fx5 heads up, we're going to deprioritize this. The reasoning is the same as in the above comment plus one additional point:

noahtalerman commented 2 years ago

cc @mikermcneil. I'm notifying you because this has been deprioritized.