Currently, breadcrumbs are only attached when an http request is aborted with an error or panic. Sometimes we want to use handleError to report a message to sentry without aborting the request.

Breadcrumbs should be added there as well.

Adds the Authentication for ApiKey and OAuth2 to the generated ApiKey. This makes it necessary that each Api-Definition contains security schema configurations. But the generated code is still backwards compatible, because it can be used without the usage of the generated Authentication.

The pace Router object registers an error handler that normally intercepts panics:

// HandleRequest should be called with defer to recover panics in request handlers
func HandleRequest(handlerName string, w http.ResponseWriter, r *http.Request) {
	if rp := recover(); rp != nil {
		HandleError(&PanicWrap{rp}, handlerName, w, r)
	}
}

But panics are caught instead by the golang http server: http/server.go

// Serve a new connection.
func (c *conn) serve(ctx context.Context) {
	c.remoteAddr = c.rwc.RemoteAddr().String()
	ctx = context.WithValue(ctx, LocalAddrContextKey, c.rwc.LocalAddr())
	defer func() {
		if err := recover(); err != nil && err != ErrAbortHandler {
			const size = 64 << 10
			buf := make([]byte, size)
			buf = buf[:runtime.Stack(buf, false)]
			c.server.logf("http: panic serving %v: %v\n%s", c.remoteAddr, err, buf)
		}

Because of this, the recover() function in HandleRequest is always nil.

The old roundtripper package did not preserve the request body on retries. This has been solve by exchanging to another package providing that handling.

• remove insecure jwt dependency
• codahale/hdrhistogram fixes #274
• update github.com/go-redis/redis/v7
• update github.com/asaskevich/govalidator
• travis -> github action
• update github.com/mattn/goveralls
• remove coverage from action
• using github action actions-goveralls

Follow convention, but I am not sure why we do this for other entries. The grpc documentation does not indicate any need to do that: https://github.com/grpc/grpc-go/blob/master/Documentation/grpc-metadata.md

Metrics gathered by the queue backend were not working as expected due to the fact that they were gathered only once on package initialization. Added a ticker to the routine that is responsible with gathering the metrics.

Adds a new grpc client and server middleware setup that works together with the rest of pace bricks.

• adds testserver for testing and demonstration
• fixes golint to use vendored version (prevents crashes)
• updates readme

Motivation

Using long-polling/streaming features with couchdb somehow causes the healthcheck calls to be blocked.

Idea

Change https://github.com/pace/bricks/blob/master/backend/couchdb/health_check.go to use a second client dedicated to just performing health checks.

During active/passive failover, if setting the label of the pod fails, the logged message always contains the state undefined and not the actual state it tried to set.

brickserrors.Handle needs a check for context cancellation and if the context is canceled there needs to be a new context to continue.

https://github.com/pace/bricks/blob/74a4ee5afd9a836fe22d734fc8b647acd20480cf/maintenance/errors/error.go#L118