Skip to content

[BACKLOG-49619][BACKLOG-48608] feat: add broker-based oauth client support#10521

Draft
AAravind27 wants to merge 1 commit into
pentaho:masterfrom
AAravind27:oauth-plugin
Draft

[BACKLOG-49619][BACKLOG-48608] feat: add broker-based oauth client support#10521
AAravind27 wants to merge 1 commit into
pentaho:masterfrom
AAravind27:oauth-plugin

Conversation

@AAravind27

@AAravind27 AAravind27 commented May 18, 2026

Copy link
Copy Markdown
Contributor

Depends on: https://github.com/pentaho/pentaho-plugin-oauth-ee/pull/2

This pull request introduces significant enhancements to support advanced OAuth 2.0 authentication flows and adds a robust, user-friendly CLI configuration management system. The main changes include new OAuth-related parameters, a new configuration manager, and a CLI command handler for managing configuration settings.

Authentication and OAuth 2.0 Enhancements:

  • Added new parameters to IParams and Params (browserAuth, deviceCode, preferredIdp, serviceAccount) to support OAuth 2.0 browser-based, device code, and service account authentication flows. These parameters are exposed via getters/setters and the builder pattern, allowing flexible configuration of authentication methods. [1] [2] [3] [4] [5]
  • Added a dependency on pentaho-oauth-client in pom.xml to enable OAuth 2.0 client functionality.

CLI Configuration Management:

  • Introduced a new CliConfig class responsible for secure storage and retrieval of CLI configuration settings (e.g., authentication timeouts, token store backend, preferred IdP). Supports loading, saving, updating, and resetting configuration properties, with validation and sensible defaults.
  • Implemented CliConfigCommandHandler, enabling users to manage CLI configuration via commands similar to npm config or git config (e.g., -config:set, -config:get, -config:list, -config:reset). This handler provides clear usage instructions and validation for supported configuration keys.

@buildguy

Copy link
Copy Markdown
Collaborator

❌ Build failed in 1m 27s

Build command:

mvn clean verify -B -e -Daudit -Djs.no.sandbox -pl \
engine,plugins/pur/core,plugins/repositories/core,ui

❗ No tests found!

Errors:

Filtered log (click to expand)

script returned exit code 1

ℹ️ This is an automatic message

@AAravind27 AAravind27 changed the title [BACKLOG-49619] feat: add broker-based oauth client support [BACKLOG-49619][BACKLOG-48606] feat: add broker-based oauth client support May 18, 2026
@AAravind27 AAravind27 changed the title [BACKLOG-49619][BACKLOG-48606] feat: add broker-based oauth client support [BACKLOG-49619][BACKLOG-48608] feat: add broker-based oauth client support May 18, 2026
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