Support Azure CLI Credentials (ActiveDirectoryAzCli)

Open
#256 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
4/5
Estimated time
3-5 days
Newbie friendliness
48/100
Issue type
Feature
Clarity
Mostly clear
Activity status
Active
Tech stack
azure, github-actions, sql, typescript
Domain
authentication, ci-cd, cloud

Research direction

Start with the dependent @tediousjs/connection-string parser at src/parser/sql-connection-string.ts lines 49-51, then inspect how sql-action handles connection strings after azure/login. Confirm how sqlcmd-go v1.4.0 supports Azure CLI credentials; done means the Active Directory Az CLI parameter is accepted and the action works without manually installing sqlcmd. Add or run focused parser and action tests if the repository provides them.

Written by the indexing model from the issue text.

Description

need-to-triage

Azure CLI Credential support was added to 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.

Dominant language
TypeScript
Stars
142
Forks
73
Avg merge
9d 18h
Merged PRs (30d)
2

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from Azure/sql-action

All issues in Azure/sql-action

Similar issues

More TypeScript issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.