Skip to content

dbeaver/pro#9627 Refactor LDAP module - #4634

Open
yagudin10 wants to merge 2 commits into
develfrom
dbeaver/pro#9627-refactor-cloudbeaver-modules
Open

dbeaver/pro#9627 Refactor LDAP module#4634
yagudin10 wants to merge 2 commits into
develfrom
dbeaver/pro#9627-refactor-cloudbeaver-modules

Conversation

@yagudin10

Copy link
Copy Markdown
Member

Closes dbeaver/pro#9627

Summary

  • rename the LDAP bundle and Java packages to io.cloudbeaver.service.auth.ldap
  • update reactor and feature references
  • use the shared dbeaver-common Maven Wrapper in backend build scripts

Validation

  • shell syntax validation
  • XML validation
  • git diff --check

@codacy-production

Copy link
Copy Markdown

Not up to standards ⛔

🔴 Issues 1 medium

Alerts:
⚠ 1 issue (≤ 0 issues of at least minor severity)

Results:
1 new issue

Category Results
BestPractice 1 medium

View in Codacy

🟢 Metrics 140 complexity · 3 duplication

Metric Results
Complexity 140
Duplication 3

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The updated backend build script has incorrect failure handling (can exit successfully on build failure), and the newly added LDAP icon resources are not consistently packaged/referenced.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Refactors the CloudBeaver LDAP authentication bundle by renaming the OSGi bundle/artifact and Java packages to io.cloudbeaver.service.auth.ldap, updating server feature/reactor references, and switching backend build scripts to use the shared dbeaver-common Maven Wrapper.

Changes:

  • Renamed LDAP bundle coordinates (symbolic name/artifactId) and moved Java packages under io.cloudbeaver.service.auth.ldap.
  • Updated product/feature and reactor module references to the new LDAP bundle id.
  • Updated backend build scripts to invoke dbeaver-common/mvnw instead of relying on a system mvn.
File summaries
File Description
server/product/aggregate/build-full.cmd Switches Windows aggregate build to shared Maven Wrapper.
server/features/io.cloudbeaver.server.feature/feature.xml Updates included LDAP plugin id to renamed bundle.
server/bundles/pom.xml Updates reactor module name for renamed LDAP bundle.
server/bundles/io.cloudbeaver.service.auth.ldap/src/io/cloudbeaver/service/auth/ldap/ssl/LdapSslSocketFactory.java Updates package name + header year.
server/bundles/io.cloudbeaver.service.auth.ldap/src/io/cloudbeaver/service/auth/ldap/ssl/LdapSslSetting.java Updates package name + header year.
server/bundles/io.cloudbeaver.service.auth.ldap/src/io/cloudbeaver/service/auth/ldap/LdapUtils.java Updates package name.
server/bundles/io.cloudbeaver.service.auth.ldap/src/io/cloudbeaver/service/auth/ldap/LdapSettings.java Updates package/imports + header year.
server/bundles/io.cloudbeaver.service.auth.ldap/src/io/cloudbeaver/service/auth/ldap/LdapSession.java Updates package name + header year.
server/bundles/io.cloudbeaver.service.auth.ldap/src/io/cloudbeaver/service/auth/ldap/LdapConstants.java Updates package name + header year.
server/bundles/io.cloudbeaver.service.auth.ldap/src/io/cloudbeaver/service/auth/ldap/LdapAuthProvider.java Updates package/imports to renamed ldap namespace.
server/bundles/io.cloudbeaver.service.auth.ldap/pom.xml Renames Maven artifactId to match new bundle id.
server/bundles/io.cloudbeaver.service.auth.ldap/plugin.xml Updates extension class reference to renamed Java package.
server/bundles/io.cloudbeaver.service.auth.ldap/OSGI-INF/l10n/bundle.properties Adds English localization for LDAP auth properties.
server/bundles/io.cloudbeaver.service.auth.ldap/OSGI-INF/l10n/bundle_ru.properties Adds Russian localization for LDAP auth properties.
server/bundles/io.cloudbeaver.service.auth.ldap/META-INF/MANIFEST.MF Renames bundle symbolic name, exports, and automatic module name.
server/bundles/io.cloudbeaver.service.auth.ldap/icons/ldap.svg Adds an LDAP icon asset.
server/bundles/io.cloudbeaver.service.auth.ldap/build.properties Introduces PDE build includes list for the renamed bundle.
deploy/build.bat Switches Windows deployment build to shared Maven Wrapper.
deploy/build-backend.sh Switches backend deployment build to shared Maven Wrapper.
Review details

Suppressed comments (1)

server/bundles/io.cloudbeaver.service.auth.ldap/plugin.xml:10

  • icons/ldap.svg was added to this bundle, but the auth provider still references the icon from org.jkiss.dbeaver.model. If the intent is to use the bundled icon (and avoid relying on an external bundle resource path), update the icon attribute accordingly.
  • Files reviewed: 15/19 changed files
  • Comments generated: 2
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread deploy/build-backend.sh Outdated
Comment thread server/product/aggregate/build-full.cmd Outdated

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The LDAP bundle’s build.properties currently references a non-existent schema/ directory and omits OSGI-INF/, which can exclude localization resources from the built plugin.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Review details
  • Files reviewed: 15/19 changed files
  • Comments generated: 1
  • Review effort level: Lite

Comment thread server/product/aggregate/build-full.cmd
@yagudin10
yagudin10 force-pushed the dbeaver/pro#9627-refactor-cloudbeaver-modules branch from ded1725 to 8866b8e Compare September 4, 2026 13:31
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.

2 participants