Azure / Azure/data-api-builder

[Bug]: Inconsistent and superflous logging?

Offen
#2,311 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
bug triage
Vorherrschende Sprache
C#
Sterne
1.5k
Forks
370
Ø Merge
3 T. 22 Std.
Gemergte PRs (30 T.)
9

Beschreibung

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

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Reproduziere die gemeldeten Befehle `dab init` und `dab add` mit `dab-config.json` und verfolge anschließend die Pfade zur Konfigurationssuche und zur Konsolenprotokollierung der CLI. Die Aufgabe ist abgeschlossen, wenn die unnötigen Konfigurationsmeldungen behoben sind und alle verbleibenden Ausgaben einheitliche Logging-Präfixe verwenden.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
azure, csharp, sql
Bereich
cli, observability-sre
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
38/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.