Conversation
Adds two Markdown-table registries under docs/home-infrastructure: - registry-web-accounts.md: services and devices reachable through a browser, grouped into providers, utilities, network, servers and smart home, prefilled with the standard local admin addresses. - registry-no-web-ui.md: everything without a web interface — SSH-only hosts, app-only devices, SMS and panel control, and offline secrets that cannot be recovered by email. No credentials are stored in the repository. The registries reference password-manager entries through a pm://<vault>/<entry> pointer whose entry name matches the row ID. README documents the column legend, the fill-in order and the access hygiene rules; .gitignore now excludes the local plaintext companion file. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JmyaT7avw51m44oq4yEboS
|
You have reached your Codex usage limits for security reviews. Please try again later. |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a6bfe35b0b
ℹ️ 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".
- Require personalized copies to live under an ignored filename. The checked-in files are templates and say so in a banner; README spells out that filling them in place and committing would publish logins, account numbers, service endpoints and a device inventory, none of which is made safe by omitting passwords alone. - Describe the password-manager entry naming as it is actually used: the entry name starts with the lowercased row ID and carries a short readable suffix, so NET-01 maps to net-01-router rather than net-01. - Give each product in a multi-product row its own default endpoint instead of one port that only fits the first product: Proxmox Backup Server answers on 8007 while 8200 is Duplicati, and the same split applies to the NAS, Wi-Fi controller, DNS filter, media, photo, monitoring and download rows. - Point Frigate at 8971, its authenticated interface. Port 5000 serves UI, API and video with no authentication at all, so listing it beside a login implied a protection that is not there. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JmyaT7avw51m44oq4yEboS
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Что добавлено
Каталог
docs/home-infrastructure/с двумя реестрами домашней инфраструктуры в виде выровненных Markdown-таблиц.registry-web-accounts.md— всё, куда можно зайти браузером: провайдеры и домены, ЖКХ и госуслуги, сеть, серверы и self-hosted, умный дом и видеонаблюдение. 54 строки со стандартными локальными адресами админок (192.168.1.1,:8006Proxmox,:8123Home Assistant и т. д.).registry-no-web-ui.md— всё, у чего веб-интерфейса нет: доступ только по SSH и локальной консоли, устройства с одним лишь мобильным приложением, управление по SMS и кнопкам на корпусе, и офлайн-секреты, которые не восстанавливаются письмом на почту.README.md— легенда колонок, схема ID, порядок заполнения и правила гигиены доступа.Это шаблон, а не заполненный реестр
Файлы остаются с плейсхолдерами. Заполнять их на месте и коммитить нельзя, и дело не только в паролях: логины, номера договоров и лицевых счетов, точные адреса и порты сервисов и сам перечень техники — роутеры, камеры, замки, банк, госуслуги — это карта дома, полезная и без единого пароля.
Порядок работы описан в README: шаблон копируется в файл с суффиксом
.local.md, заполняется только копия..gitignoreдополнен масками*.local.md,*.secrets.mdиhome-infra-secrets*, чтобы такие копии не попали в индекс по случайномуgit add -A; в README отдельно оговорено, что это подстраховка, а не защита.Обращение с секретами
В репозиторий не попадает ни одного пароля, ключа, PIN-кода или резервного кода 2FA. Вместо значения в колонке «Секрет» стоит указатель на запись в менеджере паролей вида
pm://хранилище/запись.Имя записи строится из ID строки в нижнем регистре и короткого читаемого имени: строка
NET-01→ записьnet-01-router, строкаLK-01→ записьlk-01-isp. Префикс с ID связывает строку и запись однозначно, хвост нужен, чтобы список в менеджере паролей читался без реестра.Строки
KEY-01(мастер-пароль хранилища) иKEY-02(резервные коды 2FA) — единственные без указателяpm://: они открывают само хранилище и по определению не могут в нём лежать.Адреса админок
Порты в колонке «Адрес входа» — заводские значения по умолчанию, у каждого продукта свой: строка «Бэкапы» несёт и
:8200Duplicati, и:8007Proxmox Backup Server, строка NAS —:5001DSM,:8080QNAP и:443TrueNAS, и так далее. Для Frigate указан:8971— интерфейс с аутентификацией; под таблицей оговорено, что порт:5000отдаёт UI, API и видео вообще без проверки пароля.Формат
Таблицы выровнены по ширине колонок — ровно так, как их приводит команда
Align Tableсамого плагина, поэтому файлы заодно служат живым примером его вывода на широких таблицах с кириллицей.Проверка
Все табличные блоки проверены на прямоугольность: в каждом блоке строки имеют одинаковую длину и одинаковое число разделителей. Внутренние ссылки между файлами разрешаются, маска
.local.mdпроверена черезgit check-ignore. Изменения затрагивают только документацию — код плагина, ресурсы и сборка не тронуты.