Skip to content

sources/ldap: fix incorrect error response for invalid sync_users_password#21016

Merged
BeryJu merged 1 commit intomainfrom
sources/ldap/fix-sync_users_password
Mar 20, 2026
Merged

sources/ldap: fix incorrect error response for invalid sync_users_password#21016
BeryJu merged 1 commit intomainfrom
sources/ldap/fix-sync_users_password

Conversation

@BeryJu
Copy link
Member

@BeryJu BeryJu commented Mar 19, 2026

an error in hiding for 4 or so years?

also the test for this wasn't actually checking the response hence this was not found

caused the UI to not be able to show the error on the correct field and just fallback to a generic error

…sword

Signed-off-by: Jens Langhammer <jens@goauthentik.io>
@BeryJu BeryJu requested a review from a team as a code owner March 19, 2026 16:19
@BeryJu BeryJu added area:backend backport/version-2026.2 Add this label to PRs to backport changes to version-2026.2 labels Mar 19, 2026
@netlify
Copy link

netlify bot commented Mar 19, 2026

Deploy Preview for authentik-docs ready!

Name Link
🔨 Latest commit e37be52
🔍 Latest deploy log https://app.netlify.com/projects/authentik-docs/deploys/69bc21f7ff0fe7000863cb27
😎 Deploy Preview https://deploy-preview-21016--authentik-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@codecov
Copy link

codecov bot commented Mar 19, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.75%. Comparing base (4dfdf9a) to head (e37be52).
⚠️ Report is 15 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #21016      +/-   ##
==========================================
- Coverage   92.78%   92.75%   -0.04%     
==========================================
  Files        1000     1000              
  Lines       56577    56580       +3     
  Branches      425      425              
==========================================
- Hits        52496    52481      -15     
- Misses       4081     4099      +18     
Flag Coverage Δ
conformance 37.39% <0.00%> (-0.01%) ⬇️
e2e 42.90% <0.00%> (+<0.01%) ⬆️
integration 22.17% <0.00%> (-0.05%) ⬇️
rust 0.23% <ø> (ø)
unit 91.69% <100.00%> (+<0.01%) ⬆️
unit-migrate 91.79% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@github-actions
Copy link
Contributor

authentik PR Installation instructions

Instructions for docker-compose

Add the following block to your .env file:

AUTHENTIK_IMAGE=ghcr.io/goauthentik/dev-server
AUTHENTIK_TAG=gh-e37be5252f3d2f0f55a1b471363826befec701a4
AUTHENTIK_OUTPOSTS__CONTAINER_IMAGE_BASE=ghcr.io/goauthentik/dev-%(type)s:gh-%(build_hash)s

Afterwards, run the upgrade commands from the latest release notes.

Instructions for Kubernetes

Add the following block to your values.yml file:

authentik:
    outposts:
        container_image_base: ghcr.io/goauthentik/dev-%(type)s:gh-%(build_hash)s
global:
    image:
        repository: ghcr.io/goauthentik/dev-server
        tag: gh-e37be5252f3d2f0f55a1b471363826befec701a4

Afterwards, run the upgrade commands from the latest release notes.

@BeryJu BeryJu merged commit 676189f into main Mar 20, 2026
110 checks passed
@BeryJu BeryJu deleted the sources/ldap/fix-sync_users_password branch March 20, 2026 13:01
authentik-automation bot pushed a commit that referenced this pull request Mar 20, 2026
…sword (#21016)

Signed-off-by: Jens Langhammer <jens@goauthentik.io>
@authentik-automation
Copy link
Contributor

🍒 Cherry-pick to version-2026.2 created: #21039

BeryJu added a commit that referenced this pull request Mar 20, 2026
…sword (cherry-pick #21016 to version-2026.2) (#21039)

sources/ldap: fix incorrect error response for invalid sync_users_password (#21016)

Signed-off-by: Jens Langhammer <jens@goauthentik.io>
Co-authored-by: Jens L. <jens@goauthentik.io>
kensternberg-authentik added a commit that referenced this pull request Mar 26, 2026
* main: (36 commits)
  website: fix typos (#20996)
  internal/outpost/ak: fix ws URL on outpost restart (#21041)
  sources/ldap: fix incorrect error response for invalid sync_users_password (#21016)
  website/docs: add missing dependencies for linux dev environment (#21020)
  core, web: update translations (#21021)
  web: bump flatted from 3.4.1 to 3.4.2 in /web (#21037)
  web: bump @sentry/browser from 10.44.0 to 10.45.0 in /web in the sentry group across 1 directory (#21022)
  website: bump flatted from 3.4.1 to 3.4.2 in /website (#21038)
  core: bump astral-sh/uv from 0.10.11 to 0.10.12 in /lifecycle/container (#21027)
  ci: bump actions-rust-lang/setup-rust-toolchain from 1.15.3 to 1.15.4 in /.github/actions/setup (#21030)
  ci: bump taiki-e/install-action from 2.68.26 to 2.69.2 in /.github/actions/setup (#21029)
  core: bump goauthentik/fips-debian from `7baeeaa` to `7726387` in /lifecycle/container (#21028)
  core: bump aws-cdk-lib from 2.243.0 to 2.244.0 (#21026)
  core: bump types-ldap3 from 2.9.13.20251121 to 2.9.13.20260319 (#21024)
  core: bump ruff from 0.15.6 to 0.15.7 (#21023)
  core: bump goauthentik/fips-python from `859ad57` to `bf45eb7` in /lifecycle/container (#21025)
  website/integrations: fix AWS SCIM with Identity Center (#21017)
  root: allow listening on multiple IPs (#20930)
  website: switch docs analytics to gtag (#20993)
  web: link file picker to docs (#20995)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:backend backport/version-2026.2 Add this label to PRs to backport changes to version-2026.2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants