Skip to content

chore: scrub internal network identifiers - #118

Open
jmagar wants to merge 4 commits into
mainfrom
chore/scrub-internal-identifiers-20260805
Open

chore: scrub internal network identifiers#118
jmagar wants to merge 4 commits into
mainfrom
chore/scrub-internal-identifiers-20260805

Conversation

@jmagar

@jmagar jmagar commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

Replaces internal network identifiers in tracked files with neutral documentation placeholders: private LAN addresses -> 192.0.2.0/24 values, overlay-network addresses -> 198.51.100.0/24 values, the internal tailnet domain -> example.ts.net, internal service domains -> *.example.internal, and host aliases -> generic role-based names (devhost, nashost, edgehost, backuphost, winhost, laptophost, deckhost, homelan, gatewayhost). No functional changes intended; see the diff for the full set of edits.

jmagar added 2 commits August 5, 2026 00:25
Replaces homelab-internal network identifiers in tracked text files with
neutral placeholders. aurora.tootie.tv (the public registry URL) and its
reverse-DNS Android package/path form (tv.tootie.aurora / tv/tootie/aurora)
are left untouched since they must keep resolving/compiling.

Replacement mapping:

LAN IPs:
  10.1.0.6            -> 192.0.2.6

Tailscale IPs (100.64.0.0/10):
  100.88.16.79         -> 198.51.100.1
  100.120.242.29       -> 198.51.100.2

Tailnet domain:
  <host>.manatee-triceratops.ts.net -> <mapped-host>.example.ts.net
  bare "manatee-triceratops"        -> "example-tailnet"

Other *.tootie.tv subdomains (aurora.tootie.tv excluded):
  lab.tootie.tv        -> lab.example.internal
  bare "tootie.tv" zone -> "example.internal"

Device/host names (word-boundary, case-insensitive):
  dookie    -> devhost
  tootie    -> nashost
  squirts   -> edgehost
  shart     -> backuphost
  steamy(y) -> winhost
  vivobook  -> laptophost
  steamdeck -> deckhost
  willynet  -> homelan
  mothership -> gatewayhost

Sanity check: tsc --noEmit against the repo's existing node_modules passes
identically before and after this change (two pre-existing, unrelated
"Cannot find module" errors in apps/device-catalog/vite.config.ts reproduce
on the base commit via git stash, so they are not caused by this change).
Replace internal IP addresses, the internal tailnet domain, and host
aliases in tracked files with neutral documentation placeholders
(192.0.2.0/24, 198.51.100.0/24, example.ts.net, *.example.internal,
role-based host aliases).
@jmagar
jmagar force-pushed the chore/scrub-internal-identifiers-20260805 branch from fb8bddb to 3bdea18 Compare August 5, 2026 04:59
jmagar added 2 commits August 5, 2026 15:29
The internal-identifier scrub replaced the bare Cloudflare zone name
with a placeholder, but left the aurora subdomain reference untouched
a few words later in the same doc (correctly, per the allowed public
app-domain exception). That made the doc self-contradictory. Restore
the bare zone-name references in the ops runbook and the theme mockup
for consistency with the rest of the file/PR.
@jmagar

jmagar commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator Author

Addressed the review finding about the internally-inconsistent scrub in ops/synthetics-cloudflare.md: the bare Cloudflare zone name had been replaced with a placeholder while the app subdomain a few words later was correctly left untouched (same public domain, allowed exception), making the doc read as self-contradictory.

Restored the bare zone-name references in ops/synthetics-cloudflare.md and docs/mockups/theme-previews.html for consistency with the rest of the file/PR.

Note: one more occurrence of the same phrase in docs/sessions/2026-07-17-live-demos-scroll-docker-synthetics.md (the "Next Steps" section) was left as-is — it should read the same way as the other two for full consistency, but a local tooling restriction in this session prevented that specific edit from landing. Flagging it here so it can be picked up in a follow-up.

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