Skip to content

Fix #231 - force username from ldap if possible#232

Merged
Gaetanbrl merged 1 commit intogeorchestra:mainfrom
jdev-org:fix-231
Nov 20, 2025
Merged

Fix #231 - force username from ldap if possible#232
Gaetanbrl merged 1 commit intogeorchestra:mainfrom
jdev-org:fix-231

Conversation

@Gaetanbrl
Copy link
Copy Markdown
Contributor

@Gaetanbrl Gaetanbrl commented Oct 29, 2025

ref issue #231

This PR will replace unknown external auth (oidc) username by ldap username if user exists.

This is usefull to retrieve correct user to insert into new org (if org doesn't exists).

This avoid as 500 error - user id does not exists (see issue to get more details.)

@Gaetanbrl Gaetanbrl marked this pull request as draft October 29, 2025 14:47
@Gaetanbrl Gaetanbrl marked this pull request as ready for review October 29, 2025 14:55
@Gaetanbrl Gaetanbrl self-assigned this Oct 29, 2025
@landryb landryb linked an issue Oct 29, 2025 that may be closed by this pull request
@pierrejego pierrejego requested a review from f-necas November 14, 2025 08:39
@pierrejego
Copy link
Copy Markdown
Member

@f-necas can I merge ?

@Gaetanbrl
Copy link
Copy Markdown
Contributor Author

I merge according to @f-necas response.

@Gaetanbrl Gaetanbrl merged commit f00d130 into georchestra:main Nov 20, 2025
3 checks passed
@github-actions
Copy link
Copy Markdown

💚 All backports created successfully

Status Branch Result
2.0.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

github-actions bot added a commit that referenced this pull request Dec 22, 2025
[2.0.x] Merge pull request #232 from jdev-org/fix-231
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Proconnect - 500 error if user already exists

2 participants