Skip to content
Merged
Prev Previous commit
Next Next commit
Fix the permission check
  • Loading branch information
lunny committed Jan 23, 2025
commit 7edb840eedf65e2b737dc4afcc57d6eaa9f47abb
4 changes: 2 additions & 2 deletions services/repository/transfer.go
Original file line number Diff line number Diff line change
Expand Up @@ -481,7 +481,7 @@ func RejectRepositoryTransfer(ctx context.Context, repo *repo_model.Repository,
}

func canUserCancelTransfer(ctx context.Context, r *repo_model.RepoTransfer, u *user_model.User) bool {
if u.ID == r.DoerID {
if u.IsAdmin || u.ID == r.DoerID {
return true
}

Expand All @@ -504,7 +504,7 @@ func canUserCancelTransfer(ctx context.Context, r *repo_model.RepoTransfer, u *u
log.Error("GetUserRepoPermission: %v", err)
return false
}
return perm.IsAdmin()
return perm.IsOwner()
}

// CancelRepositoryTransfer cancels the repository transfer process. The sender or
Expand Down