[PR #596] [MERGED] Allow headscale to enable/disable logtail #1510

Closed
opened 2025-12-29 02:30:20 +01:00 by adam · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/juanfont/headscale/pull/596
Author: @kradalby
Created: 5/30/2022
Status: Merged
Merged: 5/30/2022
Merged by: @kradalby

Base: mainHead: disable-logcatcher


📝 Commits (5)

📊 Changes

6 files changed (+34 additions, -0 deletions)

View changed files

📝 CHANGELOG.md (+2 -0)
📝 api.go (+3 -0)
📝 app.go (+6 -0)
📝 cmd/headscale/cli/utils.go (+13 -0)
📝 cmd/headscale/headscale_test.go (+1 -0)
📝 config-example.yaml (+9 -0)

📄 Description

This PR allows users to disable logtail in their Tailscale clients, essentially allowing them to stop sending logs to Tailscale.

This resolves #552.

LogTail is now disabled by default.

  • read the CONTRIBUTING guidelines
  • raised a GitHub issue or discussed it on the projects chat beforehand
  • added unit tests
  • updated CHANGELOG.md

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/juanfont/headscale/pull/596 **Author:** [@kradalby](https://github.com/kradalby) **Created:** 5/30/2022 **Status:** ✅ Merged **Merged:** 5/30/2022 **Merged by:** [@kradalby](https://github.com/kradalby) **Base:** `main` ← **Head:** `disable-logcatcher` --- ### 📝 Commits (5) - [`a0c465c`](https://github.com/juanfont/headscale/commit/a0c465c2eb178b8ccbe07349c9b211a7d095f7a1) Wire up setting to enable/disable logtail - [`ff5f31b`](https://github.com/juanfont/headscale/commit/ff5f31b87eb17a48927f7221b9009a9dbc9ce96e) Disable logtail for clients - [`7f66d91`](https://github.com/juanfont/headscale/commit/7f66d9184bf7a29c531aa37c9c282662b2d3ff04) Add config test - [`86dfc91`](https://github.com/juanfont/headscale/commit/86dfc91dd50da407fdfe2ac9b00e6ae1b1029f73) update readme - [`df7d5fa`](https://github.com/juanfont/headscale/commit/df7d5fa2b9976dd7994c3347f10c11be76637e47) Fix lint ### 📊 Changes **6 files changed** (+34 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `CHANGELOG.md` (+2 -0) 📝 `api.go` (+3 -0) 📝 `app.go` (+6 -0) 📝 `cmd/headscale/cli/utils.go` (+13 -0) 📝 `cmd/headscale/headscale_test.go` (+1 -0) 📝 `config-example.yaml` (+9 -0) </details> ### 📄 Description This PR allows users to disable logtail in their Tailscale clients, essentially allowing them to stop sending logs to Tailscale. This resolves #552. LogTail is now disabled by default. <!-- Please tick if the following things apply. You… --> - [x] read the [CONTRIBUTING guidelines](README.md#contributing) - [x] raised a GitHub issue or discussed it on the projects chat beforehand - [x] added unit tests - [x] updated CHANGELOG.md <!-- If applicable, please reference the issue using `Fixes #XXX` and add tests to cover your new code. --> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
adam added the pull-request label 2025-12-29 02:30:20 +01:00
adam closed this issue 2025-12-29 02:30:21 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/headscale#1510