Upstream/harbor
1
0
Fork 0
mirror of https://github.com/goharbor/harbor.git synced 2024-11-19 16:55:16 +01:00
Code Issues Projects Releases Wiki Activity

Merge pull request #8575 from ywk253100/190806_jobservice

Browse Source 
Ignore the arguments when logging the job detail
This commit is contained in:
Wang Yan 2019-08-07 11:07:10 +08:00 committed by GitHub
commit 0bbd17388e
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/jobservice/worker/cworker/c_worker.go
View File

@ -17,6 +17,7 @@ package cworker
import (
"fmt"
"reflect"
"sync"
"time"
"github.com/gocraft/work"
@ -30,7 +31,6 @@ import (
"github.com/goharbor/harbor/src/jobservice/worker"
"github.com/gomodule/redigo/redis"
"github.com/pkg/errors"
"sync"
)
var (
@ -66,7 +66,10 @@ type workerContext struct{}
// log the job
func (rpc *workerContext) logJob(job *work.Job, next work.NextMiddlewareFunc) error {
jobInfo, _ := utils.SerializeJob(job)
jobCopy := *job
// as the args may contain sensitive information, ignore them when logging the detail
jobCopy.Args = nil
jobInfo, _ := utils.SerializeJob(&jobCopy)
logger.Infof("Job incoming: %s", jobInfo)
return next()