Skip to content

fix: npm install in envs with stricter policies - #1565

Merged
aaudzei merged 1 commit into
mainfrom
users/antonaudzei/fix-keytar-install
Aug 31, 2026
Merged

fix: npm install in envs with stricter policies#1565
aaudzei merged 1 commit into
mainfrom
users/antonaudzei/fix-keytar-install

Conversation

@aaudzei

@aaudzei aaudzei commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Explicitly sets policy allowing post install scripts to be executed for selected packages.
It is required to fix error that some stricter envs may experience:

FAIL test/src/pat-auth.test.ts
  ● Test suite failed to run

    libsecret-1.so.0: cannot open shared object file: No such file or directory

      4 | import { AzureCliCredential, ChainedTokenCredential, DefaultAzureCredential, TokenCredential } from "@azure/identity";
      5 | import { AccountInfo, AuthenticationResult, PublicClientApplication } from "@azure/msal-node";
    > 6 | import { NativeBrokerPlugin } from "@azure/msal-node-extensions";
        | ^
      7 | import open from "open";
      8 | import { logger } from "./logger.js";
      9 |

      at CjsLoader.loadModule (node_modules/jest-runtime/build/index.js:312:29)
      at Object.<anonymous> (node_modules/keytar/lib/keytar.js:1:14)
      at Object.<anonymous> (node_modules/@azure/msal-node-extensions/lib/msal-node-extensions.cjs:9:14)
      at Object.require (src/auth.ts:6:1)
      at Object.<anonymous> (test/src/pat-auth.test.ts:31:1)

PR Checklist

  • I have read the contribution guidelines
  • I have read the code of conduct guidelines
  • Title of the pull request is clear and informative.
  • 👌 Code hygiene
  • 🔭 Telemetry added, updated, or N/A
  • 📄 Documentation added, updated, or N/A
  • 🛡️ Automated tests added, or N/A

🧪 How did you test it?

Reproduced on a machine not executing post install scripts by default. Applied diff and retried.

@aaudzei
aaudzei requested a review from a team as a code owner August 31, 2026 14:24
@github-actions

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@aaudzei
aaudzei merged commit f2064af into main Aug 31, 2026
7 checks passed
@aaudzei
aaudzei deleted the users/antonaudzei/fix-keytar-install branch August 31, 2026 14:46
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