Skip to content

fix: [OCISDEV-1045] Show vault-shared mountpoints in the vault drive list - #12644

Merged
2403905 merged 1 commit into
stable-8.1from
fix/OCISDEV-1045
Jul 27, 2026
Merged

fix: [OCISDEV-1045] Show vault-shared mountpoints in the vault drive list#12644
2403905 merged 1 commit into
stable-8.1from
fix/OCISDEV-1045

Conversation

@2403905

@2403905 2403905 commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Description

Related PR owncloud/reva#666

Shares of vault resources are mountpoints hosted on the shares storage provider
that grant into the vault storage provider. When listing spaces with the vault
storage_id, the storage registry skipped the shares provider entirely, so the
vault share mountpoint was missing from the vault drive list while still showing
up in the regular drive list.

The registry now also queries the shares provider when the vault storage id is
requested and segregates share mountpoints by their grantStorageID, so a vault
share only appears in the vault drive list and a regular share only in the
regular one.

Related Issue

  • Fixes <issue_link>

Motivation and Context

How Has This Been Tested?

  • test environment:
  • test case 1:
  • test case 2:
  • ...

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised:

@kw-security

kw-security commented Jul 24, 2026

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@2403905
2403905 force-pushed the fix/OCISDEV-1045 branch from ab170c6 to fe87362 Compare July 27, 2026 08:48
@2403905
2403905 enabled auto-merge (squash) July 27, 2026 10:35
@mzner
mzner self-requested a review July 27, 2026 11:27
@2403905
2403905 merged commit 683312a into stable-8.1 Jul 27, 2026
118 of 120 checks passed
@2403905
2403905 deleted the fix/OCISDEV-1045 branch July 27, 2026 11:27
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.

4 participants