Skip to content

Conversation

@pjv
Copy link
Contributor

@pjv pjv commented Feb 7, 2026

Two fixes:

  1. TS_ACCEPT_DNS is an environment variable, not a config
  2. CRITICAL CHANGE: wrong path to data directory for pocket ID results in losing entire config when you bring down the container :^(

Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Refactoring

How Has This Been Tested?

It works with these changes. It doesn't work without these changes.

Checklist

  • I have performed a self-review of my code
  • I have added tests that prove my fix or feature works
  • I have updated necessary documentation (e.g. frontpage README.md)
  • Any dependent changes have been merged and published in downstream modules

Two fixes:
1. TS_ACCEPT_DNS is an environment variable, not a config
2. CRITICAL CHANGE: wrong path to data directory for pocket ID results in losing entire config when you bring down the container :^(
@crypt0rr crypt0rr self-requested a review February 7, 2026 16:14
@crypt0rr crypt0rr assigned crypt0rr and pjv and unassigned crypt0rr Feb 7, 2026
@crypt0rr crypt0rr added the bug Something isn't working label Feb 7, 2026
Copy link
Collaborator

@crypt0rr crypt0rr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @pjv 🚀

@crypt0rr crypt0rr merged commit ed480d1 into tailscale-dev:main Feb 7, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants