Skip to content

Fix IPv6 support#398

Merged
frenck merged 3 commits intohassio-addons:mainfrom
8191:8191-ipv6
Feb 14, 2026
Merged

Fix IPv6 support#398
frenck merged 3 commits intohassio-addons:mainfrom
8191:8191-ipv6

Conversation

@8191
Copy link
Contributor

@8191 8191 commented Jan 31, 2026

Proposed Changes

Let NGINX listen on both, IPv4 and IPv6 interfaces within the container.

Docker config already forwards IPv6 requests to the container, so no change there.

Related Issues

#395

Summary by CodeRabbit

  • New Features

    • Added full IPv6 support to server configurations across all available endpoints, enabling comprehensive dual-stack networking connectivity. The service now accepts connections from both IPv4 and IPv6 addresses, significantly improving overall compatibility and accessibility for users across modern network environments.
  • Chores

    • Standardized configuration file formatting and structure.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Jan 31, 2026

Walkthrough

Two nginx server configuration files are updated to support IPv6 by adding IPv6 listen directives on port 7277 alongside existing IPv4 listeners, and formatting is corrected by adding trailing newlines.

Changes

Cohort / File(s) Summary
IPv6 nginx Configuration
vaultwarden/rootfs/etc/nginx/servers/direct-ssl.disabled, vaultwarden/rootfs/etc/nginx/servers/direct.disabled
Added IPv6 listen directive (listen [::]:7277 default_server ssl; or listen [::]:7277 default_server;) to enable IPv6 support alongside IPv4. Added trailing newlines for proper file formatting.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Hoppity-hop, the config's complete,
IPv6 addresses now meet,
On port 7277, both old and new,
Dual-stack listening, through and through! 🌐

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Fix IPv6 support' directly and accurately summarizes the main change: adding IPv6 listen directives to nginx server configurations in two files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@8191
Copy link
Contributor Author

8191 commented Feb 14, 2026

@frenck Any chance to merge this within the next updates? I manually need to apply this patch after each update/restart. ☹️

@frenck frenck added the bugfix Inconsistencies or issues which will cause a problem for users or implementors. label Feb 14, 2026
@frenck frenck merged commit 7db6599 into hassio-addons:main Feb 14, 2026
13 of 14 checks passed
@frenck
Copy link
Member

frenck commented Feb 14, 2026

Thanks, @8191 👍

../Frenck

                       

Blogging my personal ramblings at frenck.dev

@github-actions github-actions bot locked and limited conversation to collaborators Feb 16, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bugfix Inconsistencies or issues which will cause a problem for users or implementors.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants