Skip to content

Commit cf9ec15

Browse files
making login calls silent (Azure#19)
Co-authored-by: Deepak Sattiraju <desattir@microsoft.com>
1 parent 3ec3e73 commit cf9ec15

2 files changed

Lines changed: 9 additions & 9 deletions

File tree

lib/main.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -49,8 +49,8 @@ function main() {
4949
throw new Error("Not all values are present in the creds object. Ensure clientId, clientSecret, tenantId and subscriptionId are supplied.");
5050
}
5151
// Attempting Az cli login
52-
yield executeAzCliCommand(`login --service-principal -u "${servicePrincipalId}" -p "${servicePrincipalKey}" --tenant "${tenantId}"`);
53-
yield executeAzCliCommand(`account set --subscription "${subscriptionId}"`);
52+
yield executeAzCliCommand(`login --service-principal -u "${servicePrincipalId}" -p "${servicePrincipalKey}" --tenant "${tenantId}"`, true);
53+
yield executeAzCliCommand(`account set --subscription "${subscriptionId}"`, true);
5454
isAzCLISuccess = true;
5555
if (enableAzPSSession) {
5656
// Attempting Az PS login
@@ -77,10 +77,10 @@ function main() {
7777
}
7878
});
7979
}
80-
function executeAzCliCommand(command) {
80+
function executeAzCliCommand(command, silent) {
8181
return __awaiter(this, void 0, void 0, function* () {
8282
try {
83-
yield exec.exec(`"${azPath}" ${command}`, [], {});
83+
yield exec.exec(`"${azPath}" ${command}`, [], { silent: !!silent });
8484
}
8585
catch (error) {
8686
throw new Error(error);

src/main.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -35,8 +35,8 @@ async function main() {
3535
throw new Error("Not all values are present in the creds object. Ensure clientId, clientSecret, tenantId and subscriptionId are supplied.");
3636
}
3737
// Attempting Az cli login
38-
await executeAzCliCommand(`login --service-principal -u "${servicePrincipalId}" -p "${servicePrincipalKey}" --tenant "${tenantId}"`);
39-
await executeAzCliCommand(`account set --subscription "${subscriptionId}"`);
38+
await executeAzCliCommand(`login --service-principal -u "${servicePrincipalId}" -p "${servicePrincipalKey}" --tenant "${tenantId}"`, true);
39+
await executeAzCliCommand(`account set --subscription "${subscriptionId}"`, true);
4040
isAzCLISuccess = true;
4141
if (enableAzPSSession) {
4242
// Attempting Az PS login
@@ -45,7 +45,7 @@ async function main() {
4545
await spnlogin.initialize();
4646
await spnlogin.login();
4747
}
48-
console.log("Login successful.");
48+
console.log("Login successful.");
4949
} catch (error) {
5050
if (!isAzCLISuccess) {
5151
core.error("Az CLI Login failed. Please check the credentials. For more information refer https://aka.ms/create-secrets-for-GitHub-workflows");
@@ -60,9 +60,9 @@ async function main() {
6060
}
6161
}
6262

63-
async function executeAzCliCommand(command: string) {
63+
async function executeAzCliCommand(command: string, silent?: boolean) {
6464
try {
65-
await exec.exec(`"${azPath}" ${command}`, [], {});
65+
await exec.exec(`"${azPath}" ${command}`, [], {silent: !!silent});
6666
}
6767
catch(error) {
6868
throw new Error(error);

0 commit comments

Comments
 (0)