Merge pull request #435 from soulseen/master

fix cron check

pkg/models/devops/devops.go

@@ -421,18 +421,26 @@ func CheckScriptCompile(req *http.Request) ([]byte, error) {
 }
 
 func CheckCron(req *http.Request) (*CheckCronRes, error) {
-	baseUrl := jenkins.Server + CheckCronUrl + req.URL.RawQuery
-	log.Infof("Jenkins-url: " + baseUrl)
-	req.SetBasicAuth(jenkins.Requester.BasicAuth.Username, jenkins.Requester.BasicAuth.Password)
+	newurl, err := url.Parse(jenkins.Server + CheckCronUrl + req.URL.RawQuery)
+
+	reqJenkins := &http.Request{
+		Method: http.MethodGet,
+		URL:    newurl,
+		Header: http.Header{},
+	}
 	var res = new(CheckCronRes)
+	client := &http.Client{Timeout: 30 * time.Second}
+
+	reqJenkins.SetBasicAuth(jenkins.Requester.BasicAuth.Username, jenkins.Requester.BasicAuth.Password)
 
-	resp, err := http.Get(baseUrl)
+	resp, err := client.Do(reqJenkins)
 	if err != nil {
 		log.Error(err)
 		return res, err
 	}
 	defer resp.Body.Close()
 
+
 	doc, err := goquery.NewDocumentFromReader(resp.Body)
 	if err != nil {
 		log.Error(err)