[PR #1333] [CLOSED] Add unix_socket directive to the "likely wanted changes" area #2044

Closed
opened 2025-12-29 03:19:31 +01:00 by adam · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/juanfont/headscale/pull/1333
Author: @qub
Created: 4/9/2023
Status: Closed

Base: mainHead: patch-1


📝 Commits (1)

  • 70f8443 Add unix_socket directive to the "likely wanted changes" area

📊 Changes

1 file changed (+3 additions, -1 deletions)

View changed files

📝 docs/running-headscale-container.md (+3 -1)

📄 Description

The default config places the control socket in /var/lib/headscale/headscale.sock, causing an error. As noted for the other files this area is not writable in the docker image. I also fixed a double space while I was at it.

I'm not sure if protocol is to also create an issue and link it to this, but please let me know and I'll do that if it's needed.

  • read the CONTRIBUTING guidelines
  • raised a GitHub issue or discussed it on the projects chat beforehand
  • added unit tests
  • added integration tests
  • updated documentation if needed
  • 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/1333 **Author:** [@qub](https://github.com/qub) **Created:** 4/9/2023 **Status:** ❌ Closed **Base:** `main` ← **Head:** `patch-1` --- ### 📝 Commits (1) - [`70f8443`](https://github.com/juanfont/headscale/commit/70f8443d40a0dc25fd87cb443dde01ef8e48cfe8) Add unix_socket directive to the "likely wanted changes" area ### 📊 Changes **1 file changed** (+3 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `docs/running-headscale-container.md` (+3 -1) </details> ### 📄 Description The default config places the control socket in /var/lib/headscale/headscale.sock, causing an error. As noted for the other files this area is not writable in the docker image. I also fixed a double space while I was at it. I'm not sure if protocol is to also create an issue and link it to this, but please let me know and I'll do that if it's needed. <!-- Please tick if the following things apply. You… --> - [ ] read the [CONTRIBUTING guidelines](README.md#contributing) - [ ] raised a GitHub issue or discussed it on the projects chat beforehand - [ ] added unit tests - [ ] added integration tests - [x] updated documentation if needed - [ ] 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 03:19:31 +01:00
adam closed this issue 2025-12-29 03:19:31 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/headscale#2044