Repository: syncthing/syncthing
File: cmd/syncthing/gui.go (Go)

Code before:
	name string
	data []byte
}

func (s *apiService) getSupportBundle(w http.ResponseWriter, r *http.Request) {
	var arrayFile []fileEntry

	// Redacted configuration as a JSON
	jsonConfig, _ := json.MarshalIndent(getRedactedConfig(s), "", "  ")
	arrayFile = append(arrayFile, fileEntry{name: "config.json", data: jsonConfig})

	// Log as a text
	var buff bytes.Buffer
	for _, line := range s.systemLog.Since(time.Time{}) {
		fmt.Fprintf(&buff, "%s: %s\n", line.When.Format(time.RFC3339), line.Message)
	}
	arrayFile = append(arrayFile, fileEntry{name: "log.txt", data: buff.Bytes()})

	// Errors as a JSON
	if errs := s.guiErrors.Since(time.Time{}); len(errs) > 0 {
		jsonError, _ := json.MarshalIndent(errs, "", "  ")
		arrayFile = append(arrayFile, fileEntry{name: "errors.json", data: jsonError})
	}

	// Errors as a JSON
	panicFiles, ok := checkPanicFile(baseDirs["config"])
	if ok {
		for _, f := range panicFiles {
			panicF, err := ioutil.ReadFile(f)
			if err != nil {
				arrayFile = append(arrayFile, fileEntry{name: filepath.Base(f), data: panicF})
			}
		}
	}

	// Version and platform information as a JSON
	versionPlatform, _ := json.MarshalIndent(map[string]string{
		"version":     Version,
		"codename":    Codename,
		"longVersion": LongVersion,
		"os":          runtime.GOOS,
		"arch":        runtime.GOARCH,
	}, "", "  ")
	arrayFile = append(arrayFile, fileEntry{name: "versionPlatform.json", data: versionPlatform})

	// Report Data as a JSON
	version := usageReportVersion
	if val, _ := strconv.Atoi(r.URL.Query().Get("version")); val > 0 {
		version = val
	}


Review comment (suggestion):
Comment here is cut/paste from above. Also I would suggest skipping the comma-ok return from checkPanicFile. In fact, I'd skip the entire function and

```
if files, err := filepath.Glob(filepath.Join(basedirs["config"], "panic*")); err == nil {
  for _, f := range files {
    ...
```


Code after:
		l.Debugln(err)
	}
	w.Header().Set("Content-Type", "text/plain; charset=utf-8")

	for _, line := range s.systemLog.Since(since) {
		fmt.Fprintf(w, "%s: %s\n", line.When.Format(time.RFC3339), line.Message)
	}
}

type fileEntry struct {
	name string
	data []byte
}

func (s *apiService) getSupportBundle(w http.ResponseWriter, r *http.Request) {
	var files []fileEntry

	// Redacted configuration as a JSON
	if jsonConfig, err := json.MarshalIndent(getRedactedConfig(s), "", "  "); err == nil {
		files = append(files, fileEntry{name: "config.json.txt", data: jsonConfig})
	} else {
		l.Warnln("Support bundle: failed to create config.json:", err)
	}

	// Log as a text
	var buflog bytes.Buffer
	for _, line := range s.systemLog.Since(time.Time{}) {
		fmt.Fprintf(&buflog, "%s: %s\n", line.When.Format(time.RFC3339), line.Message)
	}
	files = append(files, fileEntry{name: "log.txt", data: buflog.Bytes()})

	// Errors as a JSON
	if errs := s.guiErrors.Since(time.Time{}); len(errs) > 0 {
		if jsonError, err := json.MarshalIndent(errs, "", "  "); err != nil {
			files = append(files, fileEntry{name: "errors.json.txt", data: jsonError})
		} else {
			l.Warnln("Support bundle: failed to create errors.json:", err)
		}
	}

	// Panic files as a JSON
	if panicFiles, err := filepath.Glob(filepath.Join(baseDirs["config"], "panic*")); err == nil {
		for _, f := range panicFiles {
			if panicFile, err := ioutil.ReadFile(f); err != nil {
				l.Warnf("Support bundle: failed to load %s: %s", filepath.Base(f), err)
			} else {
				files = append(files, fileEntry{name: filepath.Base(f), data: panicFile})
			}
		}
	}