Skip to content

Conversation

@Deekshitha2812
Copy link
Contributor

Description

The @azure/Identity is giving npm audit failure. So updated the it's version to ^4.2.1. But there was an error while building the package.

> C:\azure-pipelines-tasks-common-packages\common-npm-packages\az-blobstorage-provider\node_modules\.bin\tsc --outDir ./_build
,node_modules/@azure/msal-node/dist/config/Configuration.d.ts(4,8): error TS1192: Module '"http"' has no default export.
node_modules/@azure/msal-node/dist/config/Configuration.d.ts(5,8): error TS1192: Module '"https"' has no default export.
,
C:\azure-pipelines-tasks-common-packages\common-npm-packages\build-scripts\util.js:37
        throw new Error(`Command '${cl}' failed`);
        ^

Error: Command 'C:\azure-pipelines-tasks-common-packages\common-npm-packages\az-blobstorage-provider\node_modules\.bin\tsc --outDir ./_build' failed
    at Object.run (C:\azure-pipelines-tasks-common-packages\common-npm-packages\build-scripts\util.js:37:15)
    at Object.<anonymous> (C:\azure-pipelines-tasks-common-packages\common-npm-packages\az-blobstorage-provider\make.js:7:6)
    at Module._compile (node:internal/modules/cjs/loader:1198:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1252:10)
    at Module.load (node:internal/modules/cjs/loader:1076:32)
    at Function.Module._load (node:internal/modules/cjs/loader:911:12)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:81:12)
    at node:internal/main/run_main_module:22:4

So enabled the "esModuleInterop": true in tsconfig.json. (Azure/azure-sdk-for-js#33208).

📌 AB#2340626


Package Name

az-blobstorage-provider


Risk Assessment (Low / Medium / High)

Assess the level of risk and provide reasoning (e.g., scope, impact, backward compatibility).


Unit Tests Added or Updated

  • Unit tests added or updated
  • Manual tests performed

Additional Testing Performed

List all other tests performed (manual or automated, including integration, regression, scenario tests, etc.). Include links to test results or logs.


Documentation Changes Required (Yes / No)

Indicate whether related documentation needs to be updated. Provide links to the updated documentation if applicable.


Dependencies

"@azure/identity": "^4.2.1"


Checklist


@Deekshitha2812
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@Deekshitha2812
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

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.

2 participants