Remove the temporary patch for "listing commits failed" error

Contributes to #452488

**Problem**

gitaly!6590 introduced a
bug that was raising a `listing commits` error if the repository was
empty.

I fixed it by adding an error handler for this error.

**Solution**

This original bug was fixed on Gitaly side:
gitaly!6801. The Rails
patch is not necessary anymore.

Changelog: other

app/models/repository.rb

@@ -171,11 +171,6 @@ def commits(ref = nil, opts = {})
     commits = Commit.decorate(commits, container) if commits.present?
 
     CommitCollection.new(container, commits, ref)
-  rescue Gitlab::Git::CommandError => e
-    # Temporary fix to address a new Gitaly internal error: https://gitlab.com/gitlab-org/gitlab/-/issues/452488
-    return CommitCollection.new(container, [], ref) if e.message.include?('listing commits failed')
-
-    raise
   end
 
   def commits_between(from, to, limit: nil)

spec/models/repository_spec.rb

@@ -386,26 +386,6 @@ def expect_to_raise_storage_error
       end
     end
 
-    context 'when Gitaly raises a CommandError error' do
-      let(:error_message) { 'Boom' }
-
-      before do
-        expect(Gitlab::Git::Commit).to receive(:where).and_raise(Gitlab::Git::CommandError, error_message)
-      end
-
-      it 're-raises an error' do
-        expect { repository.commits('master', limit: 60) }.to raise_error(Gitlab::Git::CommandError, error_message)
-      end
-
-      context 'when error contains "listing commits failed" message' do
-        let(:error_message) { 'listing commits failed' }
-
-        it 'returns an empty collection' do
-          expect(repository.commits('master', limit: 60).to_a).to eq([])
-        end
-      end
-    end
-
     context 'when ref is passed' do
       it 'returns every commit from the specified ref' do
         expect(repository.commits('master', limit: 60).size).to eq(37)