Conversation
Owner
tis24dev
commented
Feb 27, 2026
tis24dev
commented
- Add Telegram pairing step to installer
- Support non-interactive upgrade auto-confirm
- Anticipate privilege context detection and reuse it in collectors
- Detect limited-privilege contexts beyond shifted userns
Introduce an optional Telegram pairing/verification step in the installer (CLI and TUI) for centralized bot mode. Adds a TUI wizard (internal/tui/wizard/telegram_setup_tui.go) with retry/skip behavior, identity detection, status feedback and persistence checks, plus extensive unit tests. Wire CLI to display Server ID and perform an interactive verification loop using notify.CheckTelegramRegistration. Update docs (CLI_REFERENCE.md, CONFIGURATION.md, INSTALL.md) to document the pairing flow and install log changes, and add logging hooks to record non-blocking failures and user choices.
Add support for auto-confirming the --upgrade flow by accepting a trailing 'y' (e.g. --upgrade y). Introduces Args.UpgradeAutoYes, extractUpgradeAutoYesArgs to preprocess os.Args, and updates Parse() to use the processed args. The upgrade command now respects UpgradeAutoYes and skips the interactive prompt (with a debug log). Documentation updated with examples and a unit test added to validate parsing behavior.
Move the unprivileged/rootless heuristic earlier in startup (right after logging Environment: ...), emit one INFO summary (Privilege context: ...) plus detailed DEBUG diagnostics, and inject the cached result into the orchestrator so collectors reuse it for privilege-sensitive command handling without re-reading /proc. Update environment + orchestrator tests for the new structured details and injection.
Extend the privilege-context detector to also use container signals (systemd container, container env, docker/podman markers, cgroup hints) and non-root EUID. Improve the startup INFO Privilege context: ... line while keeping full evidence in DEBUG. Update the privilege-sensitive SKIP hint and related docs/tests to match the broader “limited privileges” semantics.
Codecov Report❌ Patch coverage is 📢 Thoughts on this report? Let us know! |
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
1 participant
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.