fix http middlewares

This commit is contained in:
A.Unger
2020-04-22 12:31:12 +02:00
parent 55db2f72d0
commit 96b97ec00d
5 changed files with 18 additions and 7 deletions

View File

@@ -77,10 +77,11 @@ func (h *webBundleServiceHandler) ListSettingsBundles(w http.ResponseWriter, r *
resp := &ListSettingsBundlesResponse{}
//if err := json.NewDecoder(r.Body).Decode(&req); err != nil {
// http.Error(w, err.Error(), http.StatusPreconditionFailed)
// return
//}
// TODO no need to parse any body on a get request. This code shoud be removed fro GET endpoints
// if err := json.NewDecoder(r.Body).Decode(&req); err != nil {
// http.Error(w, err.Error(), http.StatusPreconditionFailed)
// return
// }
if err := h.h.ListSettingsBundles(
context.Background(),

View File

@@ -6,7 +6,10 @@ import (
// NewInstrument returns a service that instruments metrics.
func NewInstrument(next Service, metrics *metrics.Metrics) Service {
return Service{}
return Service{
manager: next.manager,
config: next.config,
}
}
type instrument struct {

View File

@@ -6,7 +6,10 @@ import (
// NewLogging returns a service that logs messages.
func NewLogging(next Service, logger log.Logger) Service {
return Service{}
return Service{
manager: next.manager,
config: next.config,
}
}
type logging struct {

View File

@@ -2,6 +2,7 @@ package svc
import (
"context"
"github.com/owncloud/ocis-settings/pkg/settings"
store "github.com/owncloud/ocis-settings/pkg/store/filesystem"

View File

@@ -2,7 +2,10 @@ package svc
// NewTracing returns a service that instruments traces.
func NewTracing(next Service) Service {
return Service{}
return Service{
manager: next.manager,
config: next.config,
}
}
type tracing struct {