diff options
author | zeripath <art27@cantab.net> | 2020-08-01 15:45:26 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-08-01 10:45:26 -0400 |
commit | bac65f1b826be84d132420f1acd3a384dd3a8db8 (patch) | |
tree | 8743ef323cbec6b0eaae2d0ddff2809220cb1f94 /modules/indexer/stats | |
parent | b3e06523c1c4c960b9ef4fd2b2f409166100a9ea (diff) | |
download | gitea-bac65f1b826be84d132420f1acd3a384dd3a8db8.tar.gz gitea-bac65f1b826be84d132420f1acd3a384dd3a8db8.zip |
Fix incorrect error logging in Stats indexer and OAuth2 (#12387)
* Fix incorrect logging in oauth2.go
Fix #11945
Signed-off-by: Andrew Thornton <art27@cantab.net>
* Handle ErrAlreadyInQueue in stats indexer
Fix #12380
Signed-off-by: Andrew Thornton <art27@cantab.net>
* Fixes type in error message of indexer
Add the missing character in the error message.
Co-authored-by: techknowlogick <techknowlogick@gitea.io>
Co-authored-by: Lieven Hollevoet <hollie@lika.be>
Diffstat (limited to 'modules/indexer/stats')
-rw-r--r-- | modules/indexer/stats/queue.go | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/modules/indexer/stats/queue.go b/modules/indexer/stats/queue.go index 9c83f9b412..8309cfcd3b 100644 --- a/modules/indexer/stats/queue.go +++ b/modules/indexer/stats/queue.go @@ -21,7 +21,7 @@ func handle(data ...queue.Data) { for _, datum := range data { opts := datum.(int64) if err := indexer.Index(opts); err != nil { - log.Error("stats queue idexer.Index(%d) failed: %v", opts, err) + log.Error("stats queue indexer.Index(%d) failed: %v", opts, err) } } } @@ -39,5 +39,11 @@ func initStatsQueue() error { // UpdateRepoIndexer update a repository's entries in the indexer func UpdateRepoIndexer(repo *models.Repository) error { - return statsQueue.Push(repo.ID) + if err := statsQueue.Push(repo.ID); err != nil { + if err != queue.ErrAlreadyInQueue { + return err + } + log.Debug("Repo ID: %d already queued", repo.ID) + } + return nil } |