Address minor comments

config/example.yaml

@@ -28,11 +28,11 @@ chihaya:
           shards: 1
 
     - name: "http"
-    config:
+      config:
-      addr: "localhost:6881"
+        addr: "localhost:6881"
-      requestTimeout: "10s"
+        requestTimeout: "10s"
-      readTimeout: "10s"
+        readTimeout: "10s"
-      writeTimeout: "10s"
+        writeTimeout: "10s"
 
     - name: "udp"
       config:

pkg/event/event.go

@@ -44,11 +44,9 @@ func init() {
 	eventToString[Stopped] = "stopped"
 	eventToString[Completed] = "completed"
 
-	stringToEvent[""] = None
+	for k, v := range eventToString {
-	stringToEvent["none"] = None
+		stringToEvent[v] = k
-	stringToEvent["started"] = Started
+	}
-	stringToEvent["stopped"] = Stopped
-	stringToEvent["completed"] = Completed
 }
 
 // New returns the proper Event given a string.

pkg/event/event_test.go

@@ -16,7 +16,7 @@ func TestNew(t *testing.T) {
 		expected    event
 		expectedErr error
 	}{
-		{"", None, nil},
+		{"", None, ErrUnknownEvent},
 		{"NONE", None, nil},
 		{"none", None, nil},
 		{"started", Started, nil},

server/http/query/query.go

@@ -196,11 +196,6 @@ func (q *Query) AnnounceRequest() (chihaya.AnnounceRequest, error) {
 		return nil, errors.New("failed to parse parameter: numwant")
 	}
 
-	request["ip"], err = q.Uint64("port")
-	if err != nil {
-		return nil, errors.New("failed to parse parameter: port")
-	}
-
 	request["port"], err = q.Uint64("port")
 	if err != nil {
 		return nil, errors.New("failed to parse parameter: port")

server/http/writer.go

@@ -69,7 +69,6 @@ func (w *writer) writeScrapeResponse(resp *chihaya.ScrapeResponse) error {
 		filesDict[infohash] = bencode.Dict{
 			"complete":   scrape.Complete,
 			"incomplete": scrape.Incomplete,
-			"downloaded": scrape.Downloaded,
 		}
 	}
 

server/store/memory/peer_store.go

@@ -52,8 +52,8 @@ func newPeerStoreConfig(storecfg *store.Config) (*peerStoreConfig, error) {
 	return &cfg, nil
 }
 
-const seedersSuffix = "-seeders"
+const seedersSuffix = "-s"
-const leechersSuffix = "-leechers"
+const leechersSuffix = "-l"
 
 type peer struct {
 	chihaya.Peer

tracker/middleware.go

@@ -13,10 +13,6 @@ import (
 // has been delivered to a client.
 type AnnounceHandler func(*config.TrackerConfig, chihaya.AnnounceRequest, *chihaya.AnnounceResponse) error
 
-func (h AnnounceHandler) handleAnnounce(cfg *config.TrackerConfig, req chihaya.AnnounceRequest, resp *chihaya.AnnounceResponse) error {
-	return h(cfg, req, resp)
-}
-
 // AnnounceMiddleware is higher-order AnnounceHandler used to implement modular
 // behavior processing an announce.
 type AnnounceMiddleware func(AnnounceHandler) AnnounceHandler
@@ -24,10 +20,7 @@ type AnnounceMiddleware func(AnnounceHandler) AnnounceHandler
 type announceChain struct{ mw []AnnounceMiddleware }
 
 func (c *announceChain) Append(mw ...AnnounceMiddleware) {
-	newMW := make([]AnnounceMiddleware, len(c.mw)+len(mw))
+	c.mw = append(c.mw, mw...)
-	copy(newMW[:len(c.mw)], c.mw)
-	copy(newMW[len(c.mw):], mw)
-	c.mw = newMW
 }
 
 func (c *announceChain) Handler() AnnounceHandler {
@@ -63,10 +56,6 @@ func RegisterAnnounceMiddleware(name string, mw AnnounceMiddleware) {
 // before it has been delivered to a client.
 type ScrapeHandler func(*config.TrackerConfig, chihaya.ScrapeRequest, *chihaya.ScrapeResponse) error
 
-func (h ScrapeHandler) handleScrape(cfg *config.TrackerConfig, req chihaya.ScrapeRequest, resp *chihaya.ScrapeResponse) error {
-	return h(cfg, req, resp)
-}
-
 // ScrapeMiddleware is higher-order ScrapeHandler used to implement modular
 // behavior processing a scrape.
 type ScrapeMiddleware func(ScrapeHandler) ScrapeHandler
@@ -74,10 +63,7 @@ type ScrapeMiddleware func(ScrapeHandler) ScrapeHandler
 type scrapeChain struct{ mw []ScrapeMiddleware }
 
 func (c *scrapeChain) Append(mw ...ScrapeMiddleware) {
-	newMW := make([]ScrapeMiddleware, len(c.mw)+len(mw))
+	c.mw = append(c.mw, mw...)
-	copy(newMW[:len(c.mw)], c.mw)
-	copy(newMW[len(c.mw):], mw)
-	c.mw = newMW
 }
 
 func (c *scrapeChain) Handler() ScrapeHandler {