Check that handlers are only called once in middleware/requestlog.

Fix a typo in middleware/httpgauge.

middleware/httpgauge/README.md

@@ -5,7 +5,7 @@
 
 Но мы будем писать упрощённую версию, которая будет хранить метрики в `map[string]int`.
 
-Ключём в такой `map` можно было бы сделать `Path` запроса, но это не очень хорошее решение.
+Ключом в такой `map` можно было бы сделать `Path` запроса, но это не очень хорошее решение.
 В пути часто встречаются изменяемые параметры. Например `/user/0`, `/user/1`, etc. В этом случае,
 мы не хотим отдельно измерять, что пришло 5 запросов к `/user/0` и 10 запросов к `/user/1`.
 Для мониторинга за сервером нам нужно знать, что пришло 15 запросов к пути `/user/{userID}`.

middleware/requestlog/requestlog_test.go

@@ -14,15 +14,36 @@ import (
  "gitlab.com/slon/shad-go/middleware/requestlog"
 )
 
+type oneShotHandler struct {
+ inner http.HandlerFunc
+
+ t *testing.T
+ used bool
+}
+
+func (h *oneShotHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
+ if h.used {
+ h.t.Errorf("handler used twice")
+ return
+ }
+
+ h.used = true
+ h.inner(w, r)
+}
+
 func TestRequestLog(t *testing.T) {
  core, obs := observer.New(zap.DebugLevel)
 
  m := chi.NewRouter()
  m.Use(requestlog.Log(zap.New(core)))
 
- m.Get("/simple", func(w http.ResponseWriter, r *http.Request) {
- w.WriteHeader(http.StatusOK)
- })
+ m.Get("/simple", (&oneShotHandler{
+ inner: func(w http.ResponseWriter, r *http.Request) {
+ w.WriteHeader(http.StatusOK)
+ },
+ t: t,
+ }).ServeHTTP)
+
  m.Post("/post", func(w http.ResponseWriter, r *http.Request) {
  w.WriteHeader(http.StatusOK)
  })