Harden public documentation and access-control boundaries - #1
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5e8e2e7d9f
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| "".join(("BBQ-", "BM6HJ64")): "real institution-managed desktop hostname", | ||
| "".join(("BBQEDU-", "PF3NRBA0")): "real institution-managed notebook hostname", |
There was a problem hiding this comment.
Remove the real hostnames from the validator
In a public checkout, each join reconstructs the exact institution-managed hostname that this commit claims to remove, and the descriptions explicitly confirm that the values are real. Splitting the strings only hides them from simple searches; anyone reading or executing the validator can recover the device identifiers. Replace these values with a non-reversible representation such as hashes, or remove this device-specific blacklist, so the public-safety change does not republish the sensitive identifiers.
Useful? React with 👍 / 👎.
What changed
grantspolicy example.gitignorefor private policy exports, enrollment artifacts, raw output and admin-console exportsWhy
The previous baseline was useful but had two material gaps:
This PR keeps the project small while making its security and portfolio claims more precise.
Deliberate scope
Validation
The
Public safetyworkflow uses Python 3.12 to:tag:managed-clienttotag:remote-targetontcp:22