Skip to content

No-op: Unauthenticated is already the default auth.provider for dab init#3453

Closed
Copilot wants to merge 2 commits intomainfrom
copilot/set-unauthenticated-as-default
Closed

No-op: Unauthenticated is already the default auth.provider for dab init#3453
Copilot wants to merge 2 commits intomainfrom
copilot/set-unauthenticated-as-default

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 16, 2026

Why make this change?

Investigation of the issue requesting Unauthenticated be set as the default auth.provider for dab init revealed this is already the case — no code changes required.

What is this change?

No changes. Both InitOptions.cs and AuthenticationOptions.cs already default to "Unauthenticated":

  • InitOptions.cs: [Option("auth.provider", Default = "Unauthenticated", ...)]
  • AuthenticationOptions.cs: public record AuthenticationOptions(string Provider = "Unauthenticated", ...)

How was this tested?

  • Integration Tests
  • Unit Tests

Existing tests in Cli.Tests/InitTests.cs consistently use "Unauthenticated" as the expected default provider value.

Sample Request(s)

# auth.provider defaults to Unauthenticated without explicit flag
dab init --database-type mssql --connection-string "@env('connection_string')"

Copilot AI linked an issue Apr 16, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Set Unauthenticated as the dab init default No-op: Unauthenticated is already the default auth.provider for dab init Apr 16, 2026
Copilot AI requested a review from Aniruddh25 April 16, 2026 07:27
@Aniruddh25 Aniruddh25 marked this pull request as ready for review April 16, 2026 18:46
Copilot AI review requested due to automatic review settings April 16, 2026 18:46
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.

@Aniruddh25 Aniruddh25 closed this Apr 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Enh]: Set Unauthenticated as the dab init default.

3 participants