[PR #6124] [management] chores: update dex version #29064

Closed
opened 2026-08-05 08:07:23 -04:00 by saavagebueno · 0 comments
Owner

Original Pull Request: https://github.com/netbirdio/netbird/pull/6124

State: closed
Merged: Yes


Describe your changes

With the MFA PR #5804 there was an update on the Dex package which was causing the server to fail while trying to load the idp password table

netbird-server     | 2026-05-12T09:21:56.077Z FATL management/internals/server/boot.go:115: failed to create API handler: failed to create instance manager: failed to list users: select password: sql: Scan error on column index 5, name "groups": expected []byte got string

The merged PR https://github.com/netbirdio/dex/pull/5 on our Dex fork fixes this issue by introducing an extra migration and some validation guardrails to make sure there aren't issues trying to parse this field


This pull request updates the Dex package to a version where this issue is no longer present

Stack

Checklist

  • Is it a bug fix
  • Is a typo/documentation fix
  • Is a feature enhancement
  • It is a refactor
  • Created tests that fail without the change (if possible)

By submitting this pull request, you confirm that you have read and agree to the terms of the Contributor License Agreement.

Documentation

Select exactly one:

  • I added/updated documentation for this change
  • Documentation is not needed for this change (explain why)

Docs PR URL (required if "docs added" is checked)

Paste the PR link from https://github.com/netbirdio/docs here:

https://github.com/netbirdio/docs/pull/__

Summary by CodeRabbit

  • Chores
    • Updated Go module replace directives for Dex packages to newer pseudo-version identifiers, refreshing dependency references.

Review Change Stack

**Original Pull Request:** https://github.com/netbirdio/netbird/pull/6124 **State:** closed **Merged:** Yes --- ## Describe your changes With the MFA PR #5804 there was an update on the Dex package which was causing the server to fail while trying to load the idp `password` table ``` netbird-server | 2026-05-12T09:21:56.077Z FATL management/internals/server/boot.go:115: failed to create API handler: failed to create instance manager: failed to list users: select password: sql: Scan error on column index 5, name "groups": expected []byte got string ``` The merged PR https://github.com/netbirdio/dex/pull/5 on our Dex fork fixes this issue by introducing an extra migration and some validation guardrails to make sure there aren't issues trying to parse this field --- This pull request updates the Dex package to a version where this issue is no longer present ## Issue ticket number and link ## Stack <!-- branch-stack --> ### Checklist - [x] Is it a bug fix - [ ] Is a typo/documentation fix - [ ] Is a feature enhancement - [ ] It is a refactor - [ ] Created tests that fail without the change (if possible) > By submitting this pull request, you confirm that you have read and agree to the terms of the [Contributor License Agreement](https://github.com/netbirdio/netbird/blob/main/CONTRIBUTOR_LICENSE_AGREEMENT.md). ## Documentation Select exactly one: - [ ] I added/updated documentation for this change - [x] Documentation is **not needed** for this change (explain why) ### Docs PR URL (required if "docs added" is checked) Paste the PR link from https://github.com/netbirdio/docs here: https://github.com/netbirdio/docs/pull/__ <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Chores** * Updated Go module replace directives for Dex packages to newer pseudo-version identifiers, refreshing dependency references. [![Review Change Stack](https://storage.googleapis.com/coderabbit_public_assets/review-stack-in-coderabbit-ui.svg)](https://app.coderabbit.ai/change-stack/netbirdio/netbird/pull/6124) <!-- end of auto-generated comment: release notes by coderabbit.ai -->
saavagebueno added the pull-request label 2026-08-05 08:07:23 -04:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: DYNR/netbird#29064