cursorius/server
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases 2 Wiki Activity

Fix index out of range issue with empty tag list printing

Browse Source
This commit is contained in:
restitux
2023-02-25 02:51:11 -07:00
parent 3ca1481632
commit 191b73fe41
2 changed files with 12 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files
pipeline_api/pipeline_api.go
+6 -4
View File
@@ -64,11 +64,13 @@ func (s *ApiServer) GetRunner(
} }
var runnerTagsStr strings.Builder var runnerTagsStr strings.Builder
fmt.Fprintf(&runnerTagsStr, "[%v", req.Msg.Tags[0]) if len(req.Msg.Tags) > 0 {
for _, tag := range req.Msg.Tags[1:] { fmt.Fprintf(&runnerTagsStr, "[%v", req.Msg.Tags[0])
fmt.Fprintf(&runnerTagsStr, ", %v", tag) for _, tag := range req.Msg.Tags[1:] {
fmt.Fprintf(&runnerTagsStr, ", %v", tag)
}
fmt.Fprintf(&runnerTagsStr, "]")
} }
fmt.Fprintf(&runnerTagsStr, "]")
response := <-respChan response := <-respChan
if response.Err != nil { if response.Err != nil {
runnermanager/runnermanager.go
+6 -4
View File
@@ -52,11 +52,13 @@ type runnerJob struct {
func (r *runnerManager) processRequest(req GetRunnerRequest) { func (r *runnerManager) processRequest(req GetRunnerRequest) {
var runnerTagsStr strings.Builder var runnerTagsStr strings.Builder
fmt.Fprintf(&runnerTagsStr, "[%v", req.Tags[0]) if len(req.Tags) > 0 {
for _, tag := range req.Tags[1:] { fmt.Fprintf(&runnerTagsStr, "[%v", req.Tags[0])
fmt.Fprintf(&runnerTagsStr, ", %v", tag) for _, tag := range req.Tags[1:] {
fmt.Fprintf(&runnerTagsStr, ", %v", tag)
}
fmt.Fprintf(&runnerTagsStr, "]")
} }
fmt.Fprintf(&runnerTagsStr, "]")
log.Infof("Got request for runner with tags \"%v\"", runnerTagsStr.String()) log.Infof("Got request for runner with tags \"%v\"", runnerTagsStr.String())
log.Debugf("Finding runner with tags %v", runnerTagsStr.String()) log.Debugf("Finding runner with tags %v", runnerTagsStr.String())