mirror of
https://github.com/hanzoai/o11y.git
synced 2026-10-15 12:58:33 +00:00
A path segment is a VALUE. Which router matched the route is not, and 140 handler bodies named gorilla/mux to read one — so the router was braided into every piece of business logic that addresses a resource by id. coretypes now owns the read (routevalue.go): Param for a path segment, SetParams for the test that has no router, RoutePath for the route TEMPLATE the audit record must be keyed on. One reader, one writer, one place that knows who filled them. PathParam — the authz extractor — reads through the same Param, so the gate and the handler cannot disagree about which segment they mean. Converted end to end, both ends of one resource plus the shared plumbing: llmpricingrule's handler, errortracking's handler AND its test injector, the SelfAccess gate's id, and audit's route template. Nine tests, and they are not vacuous: stubbing Param to "" turns eight of them red. Param answers for a PATH segment only. A query value stays r.URL.Query().Get — conflating the two is how a route starts reading the wrong half of the URL, and TestParamIsNotAQueryRead pins it. RoutePath answers "" with no matched route where the old mux.CurrentRoute(req).GetPathTemplate() dereferenced nil. Route table unchanged: 367 method+path pairs before and after (353 typed ops, 11 hatches, 3 probes). No registration was touched — that flip is atomic across the whole tree and is the next commit, not this one. Co-authored-by: Hanzo Dev <[email protected]>
187 lines
5.2 KiB
Go
187 lines
5.2 KiB
Go
package middleware
|
|
|
|
import (
|
|
"log/slog"
|
|
"net"
|
|
"net/http"
|
|
"time"
|
|
|
|
semconv "go.opentelemetry.io/otel/semconv/v1.26.0"
|
|
"go.opentelemetry.io/otel/trace"
|
|
|
|
"github.com/hanzoai/o11y/pkg/auditor"
|
|
"github.com/hanzoai/o11y/pkg/errors"
|
|
"github.com/hanzoai/o11y/pkg/http/render"
|
|
"github.com/hanzoai/o11y/pkg/types/audittypes"
|
|
"github.com/hanzoai/o11y/pkg/types/authtypes"
|
|
"github.com/hanzoai/o11y/pkg/types/coretypes"
|
|
)
|
|
|
|
const (
|
|
logMessage = "::RECEIVED-REQUEST::"
|
|
)
|
|
|
|
type Audit struct {
|
|
logger *slog.Logger
|
|
excludedRoutes map[string]struct{}
|
|
auditor auditor.Auditor
|
|
}
|
|
|
|
func NewAudit(logger *slog.Logger, excludedRoutes []string, auditor auditor.Auditor) *Audit {
|
|
excludedRoutesMap := make(map[string]struct{})
|
|
for _, route := range excludedRoutes {
|
|
excludedRoutesMap[route] = struct{}{}
|
|
}
|
|
|
|
return &Audit{
|
|
logger: logger.With(slog.String("pkg", pkgname)),
|
|
excludedRoutes: excludedRoutesMap,
|
|
auditor: auditor,
|
|
}
|
|
}
|
|
|
|
func (middleware *Audit) Wrap(next http.Handler) http.Handler {
|
|
return http.HandlerFunc(func(rw http.ResponseWriter, req *http.Request) {
|
|
start := time.Now()
|
|
host, port, _ := net.SplitHostPort(req.Host)
|
|
// The route TEMPLATE, not the request path: an audit record keyed on
|
|
// /v1/o11y/rules/7 is one record shape per id. Falls back to the
|
|
// request path when no route matched (a 404 is still audited).
|
|
path := coretypes.RoutePath(req)
|
|
if path == "" {
|
|
path = req.URL.Path
|
|
}
|
|
|
|
fields := []any{
|
|
string(semconv.ClientAddressKey), req.RemoteAddr,
|
|
string(semconv.UserAgentOriginalKey), req.UserAgent(),
|
|
string(semconv.ServerAddressKey), host,
|
|
string(semconv.ServerPortKey), port,
|
|
string(semconv.HTTPRequestSizeKey), req.ContentLength,
|
|
string(semconv.HTTPRouteKey), path,
|
|
}
|
|
|
|
responseBuffer := &byteBuffer{}
|
|
writer := newResponseCapture(rw, responseBuffer)
|
|
|
|
// Capture the body only when a resolved resource derives an id from it (e.g. a create).
|
|
if coretypes.ShouldCaptureResponseBody(req.Context()) {
|
|
writer.EnableBodyCapture()
|
|
}
|
|
|
|
next.ServeHTTP(writer, req)
|
|
|
|
statusCode, writeErr := writer.StatusCode(), writer.WriteError()
|
|
|
|
// Logging or Audit: skip if the matched route is in the excluded list. This allows us to exclude noisy routes (e.g. health checks) from both logging and audit.
|
|
if _, ok := middleware.excludedRoutes[path]; ok {
|
|
return
|
|
}
|
|
|
|
middleware.emitAuditEvent(req, writer, path)
|
|
|
|
fields = append(fields,
|
|
string(semconv.HTTPResponseStatusCodeKey), statusCode,
|
|
string(semconv.HTTPServerRequestDurationName), time.Since(start),
|
|
)
|
|
if writeErr != nil {
|
|
fields = append(fields, errors.Attr(writeErr))
|
|
middleware.logger.ErrorContext(req.Context(), logMessage, fields...)
|
|
} else {
|
|
if statusCode >= 400 && responseBuffer.Len() != 0 {
|
|
fields = append(fields, "response.body", responseBuffer.String())
|
|
}
|
|
|
|
middleware.logger.InfoContext(req.Context(), logMessage, fields...)
|
|
}
|
|
})
|
|
}
|
|
|
|
func (middleware *Audit) emitAuditEvent(req *http.Request, writer responseCapture, routeTemplate string) {
|
|
if middleware.auditor == nil {
|
|
return
|
|
}
|
|
|
|
resolved, err := coretypes.ResolvedResourcesFromContext(req.Context())
|
|
if err != nil || len(resolved) == 0 {
|
|
return
|
|
}
|
|
|
|
claims, _ := authtypes.ClaimsFromContext(req.Context())
|
|
statusCode := writer.StatusCode()
|
|
span := trace.SpanFromContext(req.Context())
|
|
|
|
var errorType, errorCode string
|
|
if statusCode >= 400 {
|
|
errorType = render.ErrorTypeFromStatusCode(statusCode)
|
|
errorCode = render.ErrorCodeFromBody(writer.BodyBytes())
|
|
}
|
|
|
|
extractorCtx := coretypes.ExtractorContext{Request: req, ResponseBody: writer.BodyBytes()}
|
|
|
|
for _, resource := range resolved {
|
|
resource.ResolveResponse(extractorCtx)
|
|
verb, category := resource.Verb(), resource.Category()
|
|
|
|
switch typed := resource.(type) {
|
|
case coretypes.ResolvedResourceWithTargetResource:
|
|
for _, sourceID := range typed.SourceIDs() {
|
|
for _, targetID := range typed.TargetIDs() {
|
|
attributesList := []audittypes.ResourceAttributes{
|
|
audittypes.NewRelatedResourceAttributes(
|
|
typed.SourceResource(),
|
|
sourceID,
|
|
typed.TargetResource(),
|
|
targetID,
|
|
),
|
|
}
|
|
|
|
// Sibling peers are symmetric, so mirror the event from the target's side too.
|
|
if !typed.IsParentChild() {
|
|
attributesList = append(attributesList, audittypes.NewRelatedResourceAttributes(
|
|
typed.TargetResource(),
|
|
targetID,
|
|
typed.SourceResource(),
|
|
sourceID,
|
|
))
|
|
}
|
|
|
|
for _, attributes := range attributesList {
|
|
middleware.auditor.Audit(req.Context(), audittypes.NewAuditEventFromHTTPRequest(
|
|
req,
|
|
routeTemplate,
|
|
statusCode,
|
|
span.SpanContext().TraceID(),
|
|
span.SpanContext().SpanID(),
|
|
verb,
|
|
category,
|
|
claims,
|
|
attributes,
|
|
errorType,
|
|
errorCode,
|
|
))
|
|
}
|
|
}
|
|
}
|
|
default:
|
|
for _, id := range resource.SourceIDs() {
|
|
attributes := audittypes.NewResourceAttributes(resource.SourceResource(), id)
|
|
|
|
middleware.auditor.Audit(req.Context(), audittypes.NewAuditEventFromHTTPRequest(
|
|
req,
|
|
routeTemplate,
|
|
statusCode,
|
|
span.SpanContext().TraceID(),
|
|
span.SpanContext().SpanID(),
|
|
verb,
|
|
category,
|
|
claims,
|
|
attributes,
|
|
errorType,
|
|
errorCode,
|
|
))
|
|
}
|
|
}
|
|
}
|
|
}
|