Skip to content

sources/ldap: make server info optional#18648

Merged
BeryJu merged 1 commit intomainfrom
sources/ldap/info-optional
Dec 7, 2025
Merged

sources/ldap: make server info optional#18648
BeryJu merged 1 commit intomainfrom
sources/ldap/info-optional

Conversation

@BeryJu
Copy link
Member

@BeryJu BeryJu commented Dec 6, 2025

closes #16837

Signed-off-by: Jens Langhammer <jens@goauthentik.io>
@BeryJu BeryJu requested a review from a team as a code owner December 6, 2025 17:27
@netlify
Copy link

netlify bot commented Dec 6, 2025

Deploy Preview for authentik-docs ready!

Name Link
🔨 Latest commit 5738026
🔍 Latest deploy log https://app.netlify.com/projects/authentik-docs/deploys/6934679110c1c60008350515
😎 Deploy Preview https://deploy-preview-18648--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.

@netlify
Copy link

netlify bot commented Dec 6, 2025

Deploy Preview for authentik-integrations canceled.

Name Link
🔨 Latest commit 5738026
🔍 Latest deploy log https://app.netlify.com/projects/authentik-integrations/deploys/6934679162ef2400084db8c2

@netlify
Copy link

netlify bot commented Dec 6, 2025

Deploy Preview for authentik-storybook ready!

Name Link
🔨 Latest commit 5738026
🔍 Latest deploy log https://app.netlify.com/projects/authentik-storybook/deploys/69346791041d5f0008e8489a
😎 Deploy Preview https://deploy-preview-18648--authentik-storybook.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 Dec 6, 2025

Codecov Report

❌ Patch coverage is 16.66667% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 93.17%. Comparing base (859a753) to head (5738026).
⚠️ Report is 21 commits behind head on main.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
authentik/sources/ldap/models.py 16.66% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #18648      +/-   ##
==========================================
- Coverage   93.22%   93.17%   -0.06%     
==========================================
  Files         933      933              
  Lines       51227    51233       +6     
==========================================
- Hits        47757    47735      -22     
- Misses       3470     3498      +28     
Flag Coverage Δ
e2e 44.78% <16.66%> (+0.03%) ⬆️
integration 22.76% <16.66%> (-0.06%) ⬇️
unit 91.43% <16.66%> (-0.01%) ⬇️
unit-migrate 91.48% <16.66%> (-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

github-actions bot commented Dec 7, 2025

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-57380266de4b4d40a85af0a83ee8e968a671c4d0
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-57380266de4b4d40a85af0a83ee8e968a671c4d0

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

@BeryJu BeryJu merged commit a0fe677 into main Dec 7, 2025
135 of 138 checks passed
@BeryJu BeryJu deleted the sources/ldap/info-optional branch December 7, 2025 15:57
@BeryJu BeryJu added the backport/version-2025.10 Add this label to PRs to backport changes to version-2025.10 label Dec 7, 2025
authentik-automation bot pushed a commit that referenced this pull request Dec 7, 2025
Signed-off-by: Jens Langhammer <jens@goauthentik.io>
@authentik-automation
Copy link
Contributor

🍒 Cherry-pick to version-2025.10 created: #18654

BeryJu added a commit that referenced this pull request Dec 7, 2025
…n-2025.10) (#18654)

sources/ldap: make server info optional (#18648)

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 Dec 11, 2025
* main: (23 commits)
  *: Auto compress images (#18673)
  website/integrations: update kimai doc (#18629)
  root: skip current tab when refreshing others (#18674)
  core: add digraph group hierarchy (#17050)
  core: bump astral-sh/uv from 0.9.15 to 0.9.16 (#18668)
  core: bump goauthentik.io/api/v3 from 3.2025120.16 to 3.2025120.18 (#18661)
  web: bump type-fest from 5.3.0 to 5.3.1 in /web (#18663)
  ci: bump peter-evans/create-pull-request from 7.0.9 to 7.0.11 (#18666)
  web: bump vite from 7.2.6 to 7.2.7 in /web (#18662)
  core: bump goauthentik/fips-debian from `a80dbbd` to `10c8086` (#18665)
  ci: bump actions/create-github-app-token from 2.2.0 to 2.2.1 (#18664)
  ci: bump astral-sh/setup-uv from 7.1.4 to 7.1.5 in /.github/actions/setup (#18667)
  website/docs: background tasks: add more detail about "next run" (#18660)
  website/docs: install-config: fix dump_config command (#18659)
  website/integrations: wordpress: fix redirect uri (#18658)
  stages/mtls: always include cert in flow plan (#18657)
  endpoints: fix UI bugs, add user binding, etc (#18609)
  sources/ldap: make server info optional (#18648)
  web/admin: fix event volume chart not updating with query (#18649)
  web: Bump types, fix ESLint errors (#17546)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport/version-2025.10 Add this label to PRs to backport changes to version-2025.10

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Please make vendor_name and vendor_version optional in check_connection for LDAP integrations

1 participant