Merge pull request #3149 from vector-im/kegan/go-stds

rageshake: Abide by Go standards
This commit is contained in:
Kegsay 2017-02-06 15:40:58 +00:00 committed by GitHub
commit 1f0b4853be

View File

@ -13,6 +13,7 @@ import (
"log" "log"
"net/http" "net/http"
"os" "os"
"path/filepath"
"strconv" "strconv"
"time" "time"
) )
@ -36,9 +37,10 @@ func respond(code int, w http.ResponseWriter) {
w.Write([]byte("{}")) w.Write([]byte("{}"))
} }
func gzipAndSave(data []byte, filepath string) error { func gzipAndSave(data []byte, fpath string) error {
filepath = "bugs/" + filepath fpath = filepath.Join("bugs", fpath)
if _, err := os.Stat(filepath); err == nil {
if _, err := os.Stat(fpath); err == nil {
return fmt.Errorf("file already exists") // the user can just retry return fmt.Errorf("file already exists") // the user can just retry
} }
var b bytes.Buffer var b bytes.Buffer
@ -52,14 +54,14 @@ func gzipAndSave(data []byte, filepath string) error {
if err := gz.Close(); err != nil { if err := gz.Close(); err != nil {
return err return err
} }
if err := ioutil.WriteFile(filepath, b.Bytes(), 0644); err != nil { if err := ioutil.WriteFile(fpath, b.Bytes(), 0644); err != nil {
return err return err
} }
return nil return nil
} }
func main() { func main() {
http.HandleFunc("/", func(w http.ResponseWriter, req *http.Request) { http.HandleFunc("/api/submit", func(w http.ResponseWriter, req *http.Request) {
if req.Method != "POST" && req.Method != "OPTIONS" { if req.Method != "POST" && req.Method != "OPTIONS" {
respond(405, w) respond(405, w)
return return