Skip to content

Inline classes from com.atlassian.crowd.directory.ldap package#441

Merged
pathob merged 2 commits intomainfrom
remove-crowd-ldap-dependency
Feb 24, 2026
Merged

Inline classes from com.atlassian.crowd.directory.ldap package#441
pathob merged 2 commits intomainfrom
remove-crowd-ldap-dependency

Conversation

@pathob
Copy link
Contributor

@pathob pathob commented Feb 24, 2026

These classes are no longer accessible in the latest Crowd version. Replaced with local constants and simple mapping logic.

Also add integration test for delegating directory CRUD

@pathob pathob force-pushed the remove-crowd-ldap-dependency branch 3 times, most recently from 10cfc88 to 5419805 Compare February 24, 2026 05:08
@pathob pathob changed the title Remove crowd-ldap dependency by inlining formerly used classes Inline classes from com.atlassian.crowd.directory.ldap package Feb 24, 2026
These classes are no longer accessible in the latest Crowd version.
Replaced with local constants and simple mapping logic.
@pathob pathob force-pushed the remove-crowd-ldap-dependency branch from 5419805 to 5929466 Compare February 24, 2026 05:38
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@pathob pathob merged commit 81240af into main Feb 24, 2026
8 of 9 checks passed
@pathob pathob deleted the remove-crowd-ldap-dependency branch February 24, 2026 06:10
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