Skip to content

chore: bump reva to 1605a62 - #12850

Open
LukasHirt wants to merge 1 commit into
masterfrom
chore/bump-reva-ocisdev-861
Open

chore: bump reva to 1605a62#12850
LukasHirt wants to merge 1 commit into
masterfrom
chore/bump-reva-ocisdev-861

Conversation

@LukasHirt

Copy link
Copy Markdown
Contributor

Summary

Bumps reva to 1605a62, which includes owncloud/reva#712.

That fix bounds the per-foreign-link gateway Stat fan-out in ListPublicShares (the jsoncs3/production public share manager) behind resource-level deduplication and a bounded worker pool, fixing request timeouts (code = Canceled) on tenants with many public links (OCISDEV-861). Visibility/permissions behavior is unchanged; only the cost of the check is bounded.

No new configuration is introduced — the reva PR is explicit about this, and it's confirmed here: the jsoncs3/json public sharing driver config already in services/sharing/pkg/config/config.go is unaffected, and ocis only talks to ListPublicShares through the gateway gRPC client.

Test plan

  • go mod tidy && go mod vendor
  • go build ./...
  • go vet ./... (diffed against pre-bump baseline: identical set of pre-existing findings, no new issues)
  • go test ./services/sharing/... ./services/graph/...
  • go mod verify

Bumps reva to 1605a62d7be2ebf58364afeb8e4e84b65b464746, which bounds the
per-foreign-link gateway Stat fan-out in ListPublicShares (jsoncs3 public
share manager) behind deduplication and a worker pool, fixing request
timeouts on tenants with many public links (OCISDEV-861). No new
configuration is introduced by this change.

Signed-off-by: Lukas Hirt <info@hirt.cz>
@LukasHirt
LukasHirt requested a review from a team as a code owner August 26, 2026 06:59
@LukasHirt LukasHirt self-assigned this Aug 26, 2026
@kw-security

kw-security commented Aug 26, 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.

@LukasHirt
LukasHirt enabled auto-merge (squash) August 26, 2026 07:03
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