Azure / Azure/data-api-builder

[Bug]: Implement httpprotobuf as open telemetry protocol.

Ouverte
#3,759 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
bug triage
Langage dominant
C#
Étoiles
1.5k
Forks
370
Merge moyen
3 j 22 h
PR mergées (30 j)
9

Description

### 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

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

Commencez dans src/Service/Program.cs et src/Service/Startup.cs, où sont configurés les exportateurs OTLP de logging, de métriques et de tracing. Suivez la façon dont exporter-protocol est chargé depuis le modèle CLI/configuration, puis vérifiez que chaque exportateur de service prend en charge à la fois grpc et httpprotobuf au lieu d’imposer grpc ; les tests d’exportateurs existants, s’il y en a, devraient couvrir les deux valeurs.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
csharp, grpc
Domaine
backend, observability
Type d'issue
Bug
Difficulté
3/5
Temps estimé
1-2 jours
Activité
Calme
Clarté
Plutôt claire
Accessibilité débutants
65/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.