pentaho / pentaho-platform

Pentaho BA Server Core
http://www.pentaho.com
Other
472 stars 723 forks source link

[BISERVER-14948] - Using LDAP authentication, Admin user is not able to view, identify and hence empty the non-admin user’s trash. #5601

Closed andreramos89 closed 6 months ago

andreramos89 commented 6 months ago

Change in the way the thrashed files are being returned to the PUC UI. Now all files should be visible for admin users.

@smmribeiro @befc @renato-s @joana-fb @miguelappleton

buildguy commented 6 months ago

:warning: Build finished in 11m

Build command:

mvn clean verify -B -e -Daudit -Djs.no.sandbox -pl repository

:no_entry: Failed Tests

:cloud_with_lightning_and_rain: 4 test(s) failed:

org.pentaho.platform.repository2.unified.jcr.DefaultDeleteHelperTest.testGetAllDeletedFiles_deletedUserHomeFolderNode on testGetAllDeletedFiles_deletedUserHomeFolderNode(org.pentaho.platform.repository2.unified.jcr.DefaultDeleteHelperTest) (click to expand)

``` ${result.errorDetails} ```

org.pentaho.platform.repository2.unified.jcr.DefaultDeleteHelperTest.testGetAllDeletedFiles on testGetAllDeletedFiles(org.pentaho.platform.repository2.unified.jcr.DefaultDeleteHelperTest) (click to expand)

``` ${result.errorDetails} ```

org.pentaho.platform.repository2.unified.jcr.DefaultDeleteHelperTest.testGetAllDeletedFiles_deletedUserHomeFolderNode (click to expand)

``` ${result.errorDetails} ```

org.pentaho.platform.repository2.unified.jcr.DefaultDeleteHelperTest.testGetAllDeletedFiles (click to expand)

``` ${result.errorDetails} ```

Tests run: 462, Failures: 4, Skipped: 0    Test Results


:information_source: This is an automatic message

buildguy commented 6 months ago

:white_check_mark: Build finished in 11m 52s

Build command:

mvn clean verify -B -e -Daudit -Djs.no.sandbox -pl repository

:ok_hand: All tests passed!

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


:information_source: This is an automatic message

buildguy commented 6 months ago

:white_check_mark: Build finished in 11m 38s

Build command:

mvn clean verify -B -e -Daudit -Djs.no.sandbox -pl repository

:ok_hand: All tests passed!

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


:information_source: This is an automatic message

buildguy commented 6 months ago

:white_check_mark: Build finished in 10m 8s

Build command:

mvn clean verify -B -e -Daudit -Djs.no.sandbox -pl repository

:ok_hand: All tests passed!

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


:information_source: This is an automatic message

buildguy commented 6 months ago

:white_check_mark: Build finished in 9m 37s

Build command:

mvn clean verify -B -e -Daudit -Djs.no.sandbox -pl repository

:ok_hand: All tests passed!

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


:information_source: This is an automatic message

hitachivantarasonarqube[bot] commented 6 months ago

SonarQube Quality Gate

Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

82.4% 82.4% Coverage
0.0% 0.0% Duplication

buildguy commented 6 months ago
[![👍 Frogbot scanned this pull request and found that it did not add vulnerable dependencies.](https://raw.githubusercontent.com/jfrog/frogbot/master/resources/v2/noVulnerabilityBannerPR.png)](https://docs.jfrog-applications.jfrog.io/jfrog-applications/frogbot)
Note: ---
**Frogbot** also supports **Contextual Analysis, Secret Detection, IaC and SAST Vulnerabilities Scanning**. This features are included as part of the [JFrog Advanced Security](https://jfrog.com/advanced-security) package, which isn't enabled on your system.

[🐸 JFrog Frogbot](https://docs.jfrog-applications.jfrog.io/jfrog-applications/frogbot)
buildguy commented 6 months ago

:white_check_mark: Build finished in 14m 11s

Build command:

mvn clean verify -B -e -Daudit -Djs.no.sandbox -pl repository

:ok_hand: All tests passed!

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


:information_source: This is an automatic message