Pass User type in context, not its pointer

Without this, the pointer to types.User is passed to the context.
This is a problem, as we do interface conversion.

I fixed it by ensuring we pass values instead of changing the
expected type at dest.

internal/middlewares/httpauth.go

@@ -47,7 +47,7 @@ func WithBasicAuth(authenticator Authenticator, next http.HandlerFunc) http.Hand
 				return
 			}
 
-			ctx := context.WithValue(r.Context(), UserContextKey, user) // This is ugly, but at least it cleans up the code and matches the usual patterns.
+			ctx := context.WithValue(r.Context(), UserContextKey, *user) // Store the user value directly in the context.
 			next.ServeHTTP(w, r.WithContext(ctx))
 			return
 		}