mattermost-communityGH-30 use api logwarn instead of std log

hectorgabucio · Oct 9, 2020 · a99e504 · a99e504
1 parent 2bd6e1b
commit a99e504
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/server/api.go b/server/api.go
@@ -4,7 +4,6 @@ import (
 	"fmt"
 	"io"
 	"io/ioutil"
-	"log"
 	"net/http"
 	"net/url"
 	"os"
@@ -89,7 +88,7 @@ func (p *Plugin) handleJobCreation(w http.ResponseWriter, r *http.Request) {
 		jobInputs[k] = v.(string)
 	}
 	if err := p.sendJobCreateRequest(userID, request.ChannelId, jobInputs); err != nil {
-		log.Println("Error sending job creation request", err)
+		p.API.LogWarn("Error sending job creation request", "err", err)
 	}
 }
 

diff --git a/server/plugin.go b/server/plugin.go
@@ -4,7 +4,6 @@ import (
 	"encoding/json"
 	"fmt"
 	"io/ioutil"
-	"log"
 	"net/http"
 	"net/url"
 	"path/filepath"
@@ -74,7 +73,7 @@ func (p *Plugin) OnActivate() error {
 	}
 
 	if cmdErr := p.API.RegisterCommand(getCommand()); cmdErr != nil {
-		log.Println("Error registering Jenkins custom command", cmdErr)
+		p.API.LogWarn("Error registering Jenkins custom command", "err", cmdErr)
 	}
 
 	p.router = p.InitAPI()
@@ -321,7 +320,7 @@ func (p *Plugin) checkIfJobHasStarted(jenkins *gojenkins.Jenkins, jobName string
 		}
 		time.Sleep(pollingSleepTime * time.Second)
 		if _, err := task.Poll(); err != nil {
-			log.Println("Error polling jenkins job to check the build status", err)
+			p.API.LogWarn("Error polling jenkins job to check the build status", "err", err)
 		}
 	}
 	buildInfo, buildErr := jenkins.GetBuild(jobName, task.Raw.Executable.Number)