Skip to content

Conversation

@zamolxe
Copy link

@zamolxe zamolxe commented Aug 31, 2025

🎟️ Tracking

Proper fix for instances where IPv6 is disabled, related to #172, #394

📔 Objective

Add option to disable IPv6 where it's not enabled

📸 Screenshots

n/a

⏰ Reminders before review

  • Contributor guidelines followed
  • All formatters and local linters executed and passed
  • Written new unit and / or integration tests where applicable
  • Protected functional changes with optionality (feature flags)
  • Used internationalization (i18n) for all UI strings
  • CI builds passed
  • Communicated to DevOps any deployment requirements
  • Updated any necessary documentation (Confluence, contributing docs) or informed the documentation team

🦮 Reviewer guidelines

  • 👍 (:+1:) or similar for great changes
  • 📝 (:memo:) or ℹ️ (:information_source:) for notes or general info
  • ❓ (:question:) for questions
  • 🤔 (:thinking:) or 💭 (:thought_balloon:) for more open inquiry that's not quite a confirmed issue and could potentially benefit from discussion
  • 🎨 (:art:) for suggestions / improvements
  • ❌ (:x:) or ⚠️ (:warning:) for more significant problems or concerns needing attention
  • 🌱 (:seedling:) or ♻️ (:recycle:) for future improvements or indications of technical debt
  • ⛏ (:pick:) for minor or nitpick changes

@zamolxe zamolxe requested a review from a team as a code owner August 31, 2025 18:40
@zamolxe zamolxe requested a review from justindbaur August 31, 2025 18:40
@CLAassistant
Copy link

CLAassistant commented Aug 31, 2025

CLA assistant check
All committers have signed the CLA.

@bitwarden-bot
Copy link

Thank you for your contribution! We've added this to our internal tracking system for review.
ID: PM-25342
Link: https://bitwarden.atlassian.net/browse/PM-25342

Details on our contribution process can be found here: https://contributing.bitwarden.com/contributing/pull-requests/community-pr-process.

@bitwarden-bot bitwarden-bot changed the title Fix ipv6 [PM-25342] Fix ipv6 Aug 31, 2025
@zamolxe
Copy link
Author

zamolxe commented Sep 12, 2025

2 minors released since I've opened this PR and still no feedback. Don't want to push but at least it would be nice to get some feedback.

@djsmith85 djsmith85 requested review from a team and michalchecinski and removed request for a team and justindbaur December 10, 2025 19:01
Copy link

@djsmith85 djsmith85 left a comment

Choose a reason for hiding this comment

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

@zamolxe Thank you for your contribution 🎉

Due to a bitwarden-unified coming out of beta and also renamed to bitwarden-lite, there's some conflicts that would need to be addressed.

@djsmith85
Copy link

@zamolxe Could you please also look into signing the CLA. This is a requirement for your changes to be included.

@djsmith85 djsmith85 linked an issue Dec 10, 2025 that may be closed by this pull request
@zamolxe
Copy link
Author

zamolxe commented Dec 12, 2025

Due to a bitwarden-unified coming out of beta and also renamed to bitwarden-lite, there's some conflicts that would need to be addressed.

Will check it.

@zamolxe
Copy link
Author

zamolxe commented Dec 12, 2025

Opened a new one (#442) since it was to complex to fix the conflicts due to upstream changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

nginx enable IPv6

4 participants