Merge pull request #4907 from glpatcern/publink-fix

public links: return error when owner could not be resolved
cs3org · Nov 4, 2024 · 1a51ab7 · 1a51ab7
2 parents a5f2cc2 + 742d606
commit 1a51ab7
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/changelog/unreleased/publink-fix.md b/changelog/unreleased/publink-fix.md
@@ -0,0 +1,3 @@
+Bugfix: public links: return error when owner could not be resolved
+
+https://github.com/cs3org/reva/pull/4907
diff --git a/pkg/auth/manager/publicshares/publicshares.go b/pkg/auth/manager/publicshares/publicshares.go
@@ -128,6 +128,9 @@ func (m *manager) Authenticate(ctx context.Context, token, secret string) (*user
 	if err != nil {
 		return nil, nil, err
 	}
+	if getUserResponse.Status.Code != rpcv1beta1.Code_CODE_OK {
+		return nil, nil, errtypes.NotFound(getUserResponse.Status.Message)
+	}
 
 	share := publicShareResponse.GetShare()
 	role := authpb.Role_ROLE_VIEWER