Azure / Azure/data-api-builder

[Bug]: Implement httpprotobuf as open telemetry protocol.

Abierto
#3,759 0 comentarios 0 reacciones 0 asignados Ver en GitHub
bug triage
Lenguaje dominante
C#
Estrellas
1.5k
Forks
370
Merge medio
3 d 22 h
PR fusionados (30 d)
9

Descripción

### What happened?

While CLI implements the option to configure the protocol, the service implementation looks hard coded to GRPC. After spending many hours to get it to use httpprotobuf it looks like at a cold level its implemented to grpc.

Copilot validating:

> You’re right: service code is currently hard-coded to gRPC.
>
> Confirmed in these service paths (all set configure.Protocol = OtlpExportProtocol.Grpc; explicitly):
>
> src/Service/Program.cs (logging OTLP exporter)
> src/Service/Startup.cs (logging, metrics, tracing OTLP exporters)
> Meanwhile, config model/schema/CLI do expose exporter-protocol with grpc | httpprotobuf, but the service wiring does not read/use that value when creating exporters.
>
> So current state is:
>
> CLI/schema/config: supports httpprotobuf
> runtime exporter setup: effectively always gRPC

### Version

2.0.9

### What database are you using?

Azure SQL

### What hosting model are you using?

Local (including CLI)

### Which API approach are you accessing DAB through?

MCP

### Relevant log output

```Text

```

### Code of Conduct

- [x] I agree to follow this project's Code of Conduct

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Comienza en src/Service/Program.cs y src/Service/Startup.cs, donde se configuran los exportadores OTLP de logging, métricas y tracing. Sigue cómo se carga exporter-protocol desde el modelo de CLI/configuración y, a continuación, verifica que cada exportador de servicio admita tanto grpc como httpprotobuf en lugar de imponer grpc; las pruebas existentes de los exportadores, si las hay, deberían cubrir ambos valores.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
csharp, grpc
Área
backend, observability
Tipo de issue
Error
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Tranquilo
Claridad
Bastante claro
Aptitud para principiantes
65/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.