Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

v2 - tenant-id and client-id are not optional #119

Closed
zijchen opened this issue Aug 25, 2022 · 0 comments · Fixed by #124
Closed

v2 - tenant-id and client-id are not optional #119

zijchen opened this issue Aug 25, 2022 · 0 comments · Fixed by #124
Assignees
@zijchen
Labels
bug Something isn't working v2-candidate
Milestone
v2

Comments

@zijchen
Copy link
Contributor

zijchen commented Aug 25, 2022

When not provided, it throws this error:
image

Note: Using AAD Service Principal Auth
@zijchen zijchen added need-to-triage Requires investigation v2-candidate and removed need-to-triage Requires investigation labels Aug 25, 2022
@zijchen zijchen self-assigned this Aug 25, 2022
@dzsquared dzsquared added this to the v2 milestone Aug 29, 2022
@zijchen zijchen added the bug Something isn't working label Sep 1, 2022
@zijchen zijchen mentioned this issue Sep 7, 2022
Merged
@zijchen zijchen linked a pull request Sep 7, 2022 that will close this issue
v2 - Remove client-id and tenant-id as inputs #124
Merged
@zijchen zijchen closed this as completed Sep 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zijchen zijchen

Labels
bug Something isn't working v2-candidate
Projects
None yet
Milestone
v2
Development

Successfully merging a pull request may close this issue.

v2 - Remove client-id and tenant-id as inputs Azure/sql-action
2 participants
@dzsquared @zijchen