domi/forgejo-domi
1
0
Fork 0
forked from forgejo/forgejo
Code Pull requests Activity

Fix dump-repo git init, fix wrong error type for NullDownloader (#20182) (#20186)

Browse source 
* Fix `dump-repo` git init

* Fix wrong error type for NullDownloader
This commit is contained in:
wxiaoguang 2022-07-02 00:01:05 +08:00 • committed by GitHub
parent bf43db10a9
commit d22826a28e
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
3 changed files with 17 additions and 11 deletions
Download patch file Download diff file Expand all files Collapse all files

2
modules/indexer/code/elastic_search.go
View file

@ -284,7 +284,7 @@ func (b *ElasticSearchIndexer) Index(ctx context.Context, repo *repo_model.Repos
reqs := make([]elastic.BulkableRequest, 0)
if len(changes.Updates) > 0 {
// Now because of some insanity with git cat-file not immediately failing if not run in a valid git directory we need to run git rev-parse first!
if err := git.EnsureValidGitRepository(git.DefaultContext, repo.RepoPath()); err != nil {
if err := git.EnsureValidGitRepository(ctx, repo.RepoPath()); err != nil {
log.Error("Unable to open git repo: %s for %-v: %v", repo.RepoPath(), repo, err)
return err
}