Skip to content

fix(security): exclude credential and dev-only files from npm package via .npmignore - #366

Merged
sridhargk merged 2 commits into
masterfrom
security-vuln-fixes
Oct 22, 2026
Merged

sridhargk merged 2 commits into
masterfrom
security-vuln-fixes

Conversation

@sridhargk

Copy link
Copy Markdown
Collaborator

…

PR summary

Fixes:
npm replaces .gitignore with .npmignore entirely when both are present. The repo's .npmignore had no *.env rule, so plaintext credential files present on the release machine were bundled into every published tarball.

PR Checklist

Please make sure that your PR fulfills the following requirements:

  • The commit message follows the Angular Commit Message Guidelines.
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

PR Type

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • New tests
  • Build/CI related changes
  • Documentation content changes
  • Other (please describe)

What is the current behavior?

What is the new behavior?

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

… via .npmignore

Signed-off-by: Sridhar G K <gksridhar09@gmail.com>
Signed-off-by: Sridhar G K <gksridhar09@gmail.com>
@sridhargk sridhargk self-assigned this Oct 22, 2026
@sridhargk
sridhargk merged commit 0839b1c into master Oct 22, 2026
6 checks passed
@sridhargk
sridhargk deleted the security-vuln-fixes branch October 22, 2026 15:41
@DevxNetworkServices

Copy link
Copy Markdown
Member

🎉 This PR is included in version 0.42.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants