Azure / Azure/sql-action

Support Azure CLI Credentials (ActiveDirectoryAzCli)

Offen
#256 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
need-to-triage
Vorherrschende Sprache
TypeScript
Sterne
142
Forks
73
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

Azure CLI Credential support [was added to](https://github.com/microsoft/go-sqlcmd/commit/0a4cd1949a9ff220649a4f7bd0d9f775488df8e1) the sqlcmd-go tool back in v1.4.0 but the `@tediousjs/connection-string` library does not support the `Active Directory Az CLI` Authentication parameter in the connection string.

https://github.com/tediousjs/connection-string/blob/v0.5.0/src/parser/sql-connection-string.ts#L49-L51

Since many use cases of this action would involve first doing an `azure/login`, it makes sense to support the CLI credentials.

I am completely aware that this is a problem in a dependent package, but it would appreciate any update on plans to support this, or guidance to achieve this that does not involve manually installing `sqlcmd`.

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginne mit dem abhängigen @tediousjs/connection-string-Parser in src/parser/sql-connection-string.ts in den Zeilen 49–51 und untersuche anschließend, wie sql-action Verbindungszeichenfolgen nach azure/login verarbeitet. Bestätige, wie sqlcmd-go v1.4.0 Anmeldedaten der Azure CLI unterstützt; als erledigt gilt, wenn der Active Directory Az CLI-Parameter akzeptiert wird und die Action funktioniert, ohne sqlcmd manuell zu installieren. Füge fokussierte Parser- und Action-Tests hinzu oder führe sie aus, falls das Repository solche bereitstellt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
azure, github-actions, sql, typescript
Bereich
authentication, ci-cd, cloud
Issue-Typ
Feature
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Aktiv
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
48/100

Neue Issues direkt in Ihr Postfach

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