mirror of
https://github.com/go-gitea/gitea
synced 2024-12-25 10:04:27 +00:00
d1353e1f7c
* update code.gitea.io/sdk/gitea v0.13.1 -> v0.13.2 * update github.com/go-swagger/go-swagger v0.25.0 -> v0.26.0 * update github.com/google/uuid v1.1.2 -> v1.2.0 * update github.com/klauspost/compress v1.11.3 -> v1.11.7 * update github.com/lib/pq 083382b7e6fc -> v1.9.0 * update github.com/markbates/goth v1.65.0 -> v1.66.1 * update github.com/mattn/go-sqlite3 v1.14.4 -> v1.14.6 * update github.com/mgechev/revive 246eac737dc7 -> v1.0.3 * update github.com/minio/minio-go/v7 v7.0.6 -> v7.0.7 * update github.com/niklasfasching/go-org v1.3.2 -> v1.4.0 * update github.com/olivere/elastic/v7 v7.0.21 -> v7.0.22 * update github.com/pquerna/otp v1.2.0 -> v1.3.0 * update github.com/xanzy/go-gitlab v0.39.0 -> v0.42.0 * update github.com/yuin/goldmark v1.2.1 -> v1.3.1
95 lines
3.8 KiB
Markdown
Vendored
95 lines
3.8 KiB
Markdown
Vendored
# httpsnoop
|
|
|
|
Package httpsnoop provides an easy way to capture http related metrics (i.e.
|
|
response time, bytes written, and http status code) from your application's
|
|
http.Handlers.
|
|
|
|
Doing this requires non-trivial wrapping of the http.ResponseWriter interface,
|
|
which is also exposed for users interested in a more low-level API.
|
|
|
|
[![GoDoc](https://godoc.org/github.com/felixge/httpsnoop?status.svg)](https://godoc.org/github.com/felixge/httpsnoop)
|
|
[![Build Status](https://travis-ci.org/felixge/httpsnoop.svg?branch=master)](https://travis-ci.org/felixge/httpsnoop)
|
|
|
|
## Usage Example
|
|
|
|
```go
|
|
// myH is your app's http handler, perhaps a http.ServeMux or similar.
|
|
var myH http.Handler
|
|
// wrappedH wraps myH in order to log every request.
|
|
wrappedH := http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
|
|
m := httpsnoop.CaptureMetrics(myH, w, r)
|
|
log.Printf(
|
|
"%s %s (code=%d dt=%s written=%d)",
|
|
r.Method,
|
|
r.URL,
|
|
m.Code,
|
|
m.Duration,
|
|
m.Written,
|
|
)
|
|
})
|
|
http.ListenAndServe(":8080", wrappedH)
|
|
```
|
|
|
|
## Why this package exists
|
|
|
|
Instrumenting an application's http.Handler is surprisingly difficult.
|
|
|
|
However if you google for e.g. "capture ResponseWriter status code" you'll find
|
|
lots of advise and code examples that suggest it to be a fairly trivial
|
|
undertaking. Unfortunately everything I've seen so far has a high chance of
|
|
breaking your application.
|
|
|
|
The main problem is that a `http.ResponseWriter` often implements additional
|
|
interfaces such as `http.Flusher`, `http.CloseNotifier`, `http.Hijacker`, `http.Pusher`, and
|
|
`io.ReaderFrom`. So the naive approach of just wrapping `http.ResponseWriter`
|
|
in your own struct that also implements the `http.ResponseWriter` interface
|
|
will hide the additional interfaces mentioned above. This has a high change of
|
|
introducing subtle bugs into any non-trivial application.
|
|
|
|
Another approach I've seen people take is to return a struct that implements
|
|
all of the interfaces above. However, that's also problematic, because it's
|
|
difficult to fake some of these interfaces behaviors when the underlying
|
|
`http.ResponseWriter` doesn't have an implementation. It's also dangerous,
|
|
because an application may choose to operate differently, merely because it
|
|
detects the presence of these additional interfaces.
|
|
|
|
This package solves this problem by checking which additional interfaces a
|
|
`http.ResponseWriter` implements, returning a wrapped version implementing the
|
|
exact same set of interfaces.
|
|
|
|
Additionally this package properly handles edge cases such as `WriteHeader` not
|
|
being called, or called more than once, as well as concurrent calls to
|
|
`http.ResponseWriter` methods, and even calls happening after the wrapped
|
|
`ServeHTTP` has already returned.
|
|
|
|
Unfortunately this package is not perfect either. It's possible that it is
|
|
still missing some interfaces provided by the go core (let me know if you find
|
|
one), and it won't work for applications adding their own interfaces into the
|
|
mix.
|
|
|
|
However, hopefully the explanation above has sufficiently scared you of rolling
|
|
your own solution to this problem. httpsnoop may still break your application,
|
|
but at least it tries to avoid it as much as possible.
|
|
|
|
Anyway, the real problem here is that smuggling additional interfaces inside
|
|
`http.ResponseWriter` is a problematic design choice, but it probably goes as
|
|
deep as the Go language specification itself. But that's okay, I still prefer
|
|
Go over the alternatives ;).
|
|
|
|
## Performance
|
|
|
|
```
|
|
BenchmarkBaseline-8 20000 94912 ns/op
|
|
BenchmarkCaptureMetrics-8 20000 95461 ns/op
|
|
```
|
|
|
|
As you can see, using `CaptureMetrics` on a vanilla http.Handler introduces an
|
|
overhead of ~500 ns per http request on my machine. However, the margin of
|
|
error appears to be larger than that, therefor it should be reasonable to
|
|
assume that the overhead introduced by `CaptureMetrics` is absolutely
|
|
negligible.
|
|
|
|
## License
|
|
|
|
MIT
|