Azure / Azure/data-api-builder

[Bug]: Inconsistent and superflous logging?

Aperta
#2,311 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
bug triage
Lingua principale
C#
Stelle
1.5k
Fork
370
Merge medio
3g 22h
PR unite (30g)
9

Descrizione

### What happened?

```dos
dab init -c dab-config.json --database-type mssql --connection-string "@env('dab-connection-string')" --host-mode Development
dab add "Album" --source "[dbo].[Album]" --fields.include "AlbumId,Title,ArtistId" --permissions "anonymous:*"
```

Results in the following console output from the second command:

```dos
Information: Microsoft.DataApiBuilder 1.1.7
Information: Config not provided. Trying to get default config based on DAB_ENVIRONMENT...
Information: Environment variable DAB_ENVIRONMENT is (null)
Loading config file from dab-config.json.
```

In my opinion, there are multiple issues here:
- Given that there is a dab-config.json file in the folder, why do I need to see line 2 and 3.
- Why is line 4 not prefixed with "Information:" - looks like a Console.WriteLine rather than logging, and looks inconsistent

### Version

1.1.7

### What database are you using?

Azure SQL

### What hosting model are you using?

Local (including CLI)

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

REST

### Relevant log output

_No response_

### Code of Conduct

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

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Riproduci i comandi segnalati `dab init` e `dab add` con `dab-config.json`, quindi traccia i percorsi di individuazione della configurazione e di logging della console della CLI. Il lavoro è completato quando i messaggi di configurazione non necessari sono stati gestiti e tutto l'output rimanente utilizza prefissi di logging coerenti.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
azure, csharp, sql
Ambito
cli, observability-sre
Tipo di issue
Bug
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
38/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.