Skip to content

Commit 8115cf1

Browse files
committed
Allow ignoring broken package repository link
1 parent ab61e45 commit 8115cf1

File tree

1 file changed

+9
-8
lines changed

1 file changed

+9
-8
lines changed

services/packages/package_update.go

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -44,16 +44,17 @@ func UnlinkFromRepository(ctx context.Context, pkg *packages_model.Package, doer
4444
}
4545

4646
repo, err := repo_model.GetRepositoryByID(ctx, pkg.RepoID)
47-
if err != nil {
47+
if err != nil && !repo_model.IsErrRepoNotExist(err) {
4848
return fmt.Errorf("error getting repository %d: %w", pkg.RepoID, err)
4949
}
50-
51-
perms, err := access_model.GetUserRepoPermission(ctx, repo, doer)
52-
if err != nil {
53-
return fmt.Errorf("error getting permissions for user %d on repository %d: %w", doer.ID, repo.ID, err)
54-
}
55-
if !perms.CanWrite(unit.TypePackages) {
56-
return util.ErrPermissionDenied
50+
if err == nil {
51+
perms, err := access_model.GetUserRepoPermission(ctx, repo, doer)
52+
if err != nil {
53+
return fmt.Errorf("error getting permissions for user %d on repository %d: %w", doer.ID, repo.ID, err)
54+
}
55+
if !perms.CanWrite(unit.TypePackages) {
56+
return util.ErrPermissionDenied
57+
}
5758
}
5859

5960
user, err := user_model.GetUserByID(ctx, pkg.OwnerID)

0 commit comments

Comments
 (0)