Azure / Azure/data-api-builder
[Bug]: Inconsistent and superflous logging?
- 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