Skip to content

providers/SCIM: fix object exists error for users, attempt to look up user ID in remote system#13437

Merged
BeryJu merged 4 commits intomainfrom
providers/scim/fix-object-exists-error
Mar 7, 2025
Merged

providers/SCIM: fix object exists error for users, attempt to look up user ID in remote system#13437
BeryJu merged 4 commits intomainfrom
providers/scim/fix-object-exists-error

Conversation

@BeryJu
Copy link
Member

@BeryJu BeryJu commented Mar 7, 2025

Details

REPLACE ME


Checklist

  • Local tests pass (ak test authentik/)
  • The code has been formatted (make lint-fix)

If an API change has been made

  • The API schema has been updated (make gen-build)

If changes to the frontend have been made

  • The code has been formatted (make web)

If applicable

  • The documentation has been updated
  • The documentation has been formatted (make website)

BeryJu added 3 commits March 7, 2025 22:37
…pported by remote system

Signed-off-by: Jens Langhammer <jens@goauthentik.io>
Signed-off-by: Jens Langhammer <jens@goauthentik.io>
Signed-off-by: Jens Langhammer <jens@goauthentik.io>
@BeryJu BeryJu requested review from a team as code owners March 7, 2025 22:39
@netlify
Copy link

netlify bot commented Mar 7, 2025

Deploy Preview for authentik-docs canceled.

Name Link
🔨 Latest commit e2ac813
🔍 Latest deploy log https://app.netlify.com/sites/authentik-docs/deploys/67cb7bf8320ef90008ddbf0e

@netlify
Copy link

netlify bot commented Mar 7, 2025

Deploy Preview for authentik-storybook ready!

Name Link
🔨 Latest commit e2ac813
🔍 Latest deploy log https://app.netlify.com/sites/authentik-storybook/deploys/67cb7bf86042370007cd4718
😎 Deploy Preview https://deploy-preview-13437--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 site configuration.

@codecov
Copy link

codecov bot commented Mar 7, 2025

Codecov Report

Attention: Patch coverage is 55.55556% with 8 lines in your changes missing coverage. Please review.

Project coverage is 92.68%. Comparing base (354634c) to head (e2ac813).
Report is 9 commits behind head on main.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
authentik/providers/scim/clients/users.py 55.55% 8 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #13437      +/-   ##
==========================================
- Coverage   92.75%   92.68%   -0.07%     
==========================================
  Files         793      793              
  Lines       40364    40376      +12     
==========================================
- Hits        37438    37421      -17     
- Misses       2926     2955      +29     
Flag Coverage Δ
e2e 47.87% <0.00%> (-0.10%) ⬇️
integration 24.06% <0.00%> (-0.01%) ⬇️
unit 90.46% <55.55%> (-0.02%) ⬇️

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.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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

github-actions bot commented Mar 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-e2ac81323422689bed0c34204c7fd63872fefeba
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-e2ac81323422689bed0c34204c7fd63872fefeba

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

@BeryJu BeryJu merged commit 1bc99e4 into main Mar 7, 2025
61 of 62 checks passed
@BeryJu BeryJu deleted the providers/scim/fix-object-exists-error branch March 7, 2025 23:07
kensternberg-authentik added a commit that referenced this pull request Mar 11, 2025
* main: (77 commits)
  web: Ignore Storybook when running codespell. (#13454)
  core: bump ruff from 0.9.9 to 0.9.10 (#13448)
  core: bump webauthn from 2.5.1 to 2.5.2 (#13449)
  website/docs: backup and restore: remove extra period (#13440)
  website: bump prismjs from 1.29.0 to 1.30.0 in /website (#13456)
  web: bump prismjs from 1.29.0 to 1.30.0 in /web (#13455)
  web: admin interface: faster card load (#13331)
  web/admin: fix display bug for assigned users in application bindings in the wizard (#13435)
  website: bump the build group across 1 directory with 9 updates (#13442)
  core: bump django from 5.0.12 to 5.0.13 (#13425)
  providers/SCIM: fix object exists error for users, attempt to look up user ID in remote system (#13437)
  website/docs: sys mgmt: document authentik backups/restoration (#12943)
  website: fix build in docker (#13430)
  website/integrations: zipline: add (#13257)
  translate: Updates for file web/xliff/en.xlf in fr (#13431)
  lifecycle/aws: bump aws-cdk from 2.1002.0 to 2.1003.0 in /lifecycle/aws (#13426)
  translate: Updates for file web/xliff/en.xlf in zh_CN (#13428)
  translate: Updates for file web/xliff/en.xlf in zh-Hans (#13429)
  core, web: update translations (#13423)
  website: add a better edit this page element (#13391)
  ...
@BeryJu
Copy link
Member Author

BeryJu commented Mar 13, 2025

/cherry-pick version-2025.2

@gcp-cherry-pick-bot
Copy link
Contributor

Cherry-pick failed with Merge error 1bc99e48e02584fb727959d3826e969762f7da92 into temp-cherry-pick-5e5f05-version-2025.2

BeryJu added a commit that referenced this pull request Mar 13, 2025
… user ID in remote system (#13437)

* providers/scim: handle ObjectExistsSyncException when filtering is supported by remote system

Signed-off-by: Jens Langhammer <jens@goauthentik.io>

* unrelated: correctly check for backchannel application in SCIM view page

Signed-off-by: Jens Langhammer <jens@goauthentik.io>

* unrelated: fix missing ignore paths in codespell

Signed-off-by: Jens Langhammer <jens@goauthentik.io>

* format

Signed-off-by: Jens Langhammer <jens@goauthentik.io>

---------

Signed-off-by: Jens Langhammer <jens@goauthentik.io>
# Conflicts:
#	pyproject.toml
kensternberg-authentik added a commit that referenced this pull request Apr 25, 2025
* main:
  web: admin interface: faster card load (#13331)
  web/admin: fix display bug for assigned users in application bindings in the wizard (#13435)
  website: bump the build group across 1 directory with 9 updates (#13442)
  core: bump django from 5.0.12 to 5.0.13 (#13425)
  providers/SCIM: fix object exists error for users, attempt to look up user ID in remote system (#13437)
  website/docs: sys mgmt: document authentik backups/restoration (#12943)
  website: fix build in docker (#13430)
  website/integrations: zipline: add (#13257)
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.

1 participant