Skip to content

Conversation

dcleao
Copy link
Contributor

@dcleao dcleao commented Sep 15, 2025

  • Added IResourceSpecificAuthorizationRequest to be used for "specific" resource requests
  • Added IResourceAuthorizationRequest#asSpecific()
  • Added ResourceSpecificAuthorizationRequest, ResourceSpecificAuthorizationRule, ResourceSpecificAuthorizationDecision
  • Renamed some "general" classes to make them more consistent with the new "specific" names: ResourceGeneralAuthorizationRule, ResourceGeneralAuthorizationDecision

https://github.com/pentaho/pentaho-platform-ee/pull/1889

…C-122]

- Added `IResourceSpecificAuthorizationRequest` to be used for "specific" resource requests
- Added `IResourceAuthorizationRequest#asSpecific()`
- Added `ResourceSpecificAuthorizationRequest`, `ResourceSpecificAuthorizationRule`, `ResourceSpecificAuthorizationDecision`
- Renamed some "general" classes to make them more consistent with the new "specific" names: `ResourceGeneralAuthorizationRule`, `ResourceGeneralAuthorizationDecision`
Copy link

Failed

  • 31.70% Coverage on New Code (is less than 80.00%)

Analysis Details

0 Issues

  • Bug 0 Bugs
  • Vulnerability 0 Vulnerabilities
  • Code Smell 0 Code Smells

Coverage and Duplications

  • Coverage 31.70% Coverage (40.20% Estimated after merge)
  • Duplications 0.00% Duplicated Code (8.00% Estimated after merge)

Project ID: pentaho:pentaho-platform-ce-parent

View in SonarQube

@buildguy
Copy link
Collaborator

👍 Frogbot scanned this pull request and did not find any new security issues.

Note:

Frogbot also supports Contextual Analysis, Secret Detection, IaC and SAST Vulnerabilities Scanning. This features are included as part of the JFrog Advanced Security package, which isn't enabled on your system.


@buildguy
Copy link
Collaborator

✅ Build finished in 18m 23s

Build command:

mvn clean verify -B -e -Daudit -Djs.no.sandbox -pl \
api,assemblies/pentaho-solutions,core

👌 All tests passed!

Tests run: 982, Failures: 0, Skipped: 0    Test Results


ℹ️ This is an automatic message

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