Skip to content

fix(netbird): use structured connection config for seed specs#34

Merged
mikkeldamsgaard merged 1 commit intomainfrom
fix/32-url-encode-db-password
Mar 14, 2026
Merged

fix(netbird): use structured connection config for seed specs#34
mikkeldamsgaard merged 1 commit intomainfrom
fix/32-url-encode-db-password

Conversation

@mikkeldamsgaard
Copy link
Copy Markdown
Contributor

Summary

  • Switch seed spec and PAT seed database connections from URL strings to Initium v2's structured config (host, port, user, password, name, options), eliminating URL-encoding issues when passwords contain special characters (@, %, :, etc.)
  • Bump Initium from 1.2.0 to 2.0.0
  • E2E tests now use password test%40pass to guard against regressions

Fixes #32

How to verify

# Unit tests
make test

# E2E (postgres backend with special-char password)
make e2e-postgres

# Rendered seed spec should show structured fields, not URL strings
helm template test charts/netbird \
  -f charts/netbird/ci/e2e-values-postgres.yaml \
  --set pat.enabled=true \
  --set pat.secret.secretName=netbird-pat \
  | grep -A12 'seed.yaml: |'

🤖 Generated with Claude Code

Switch seed spec and PAT seed database connections from URL strings
to Initium v2's structured config (host, port, user, password, name,
options). This eliminates URL-encoding issues when the database
password contains special characters like @, %, or :.

- Bump Initium from 1.2.0 to 2.0.0
- Remove patDatabaseUrl helper (inlined as structured fields)
- E2E tests now use password "test%40pass" to guard against regressions
- Update unit test assertions and snapshots

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@mikkeldamsgaard mikkeldamsgaard merged commit b002551 into main Mar 14, 2026
7 checks passed
@mikkeldamsgaard mikkeldamsgaard deleted the fix/32-url-encode-db-password branch March 14, 2026 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

netbird: db-seed init container fails when database password contains URL-special characters

1 participant