Skip to content

Fix 12 dependabot alerts: update transitive deps, replace vulnerable ip package, bump fast-xml-parser override#58

Merged
felickz merged 2 commits intomainfrom
copilot/fix-dependabot-alerts-again
Apr 7, 2026
Merged

Fix 12 dependabot alerts: update transitive deps, replace vulnerable ip package, bump fast-xml-parser override#58
felickz merged 2 commits intomainfrom
copilot/fix-dependabot-alerts-again

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 7, 2026

Resolves dependabot alerts 473–482, 484–485 spanning all 15 package manifests in the repo.

Transitive dependency updates (npm audit fix)

  • Root and all 7 function packages: patched ajv, brace-expansion, diff, flatted, js-yaml, minimatch, picomatch, semver, @smithy/config-resolver

fast-xml-parser override bump (getSecretDetails)

Replace ip with ipaddr.js (githubWebhookIPValidator)

  • ip has an unpatched SSRF vulnerability (GHSA-2p57-rm9w-gvfp) with no fix available
  • Switched to ipaddr.js@^2.2.0 which has built-in TypeScript types
// Before
import { cidrSubnet } from "ip";
const findIP = (keys: string[], ipToCheck: string) => {
  return keys.some((cidr) => cidrSubnet(cidr).contains(ipToCheck));
};

// After
import * as ipaddr from "ipaddr.js";
const findIP = (keys: string[], ipToCheck: string) => {
  const parsedIP = ipaddr.parse(ipToCheck);
  return keys.some((cidr) => {
    const [addr, prefixLength] = ipaddr.parseCIDR(cidr);
    return parsedIP.match(addr, prefixLength);
  });
};

Remove unused ip dep (githubWebhookSecretValidator)

  • ip and @types/ip were listed but never imported — removed both

All 15 package-lock.json manifests now report 0 audit vulnerabilities. TypeScript compilation verified on affected packages.

Copilot AI and others added 2 commits April 7, 2026 15:25
…paddr.js, update fast-xml-parser override to 5.5.10, remove unused ip dep from secretValidator

Agent-Logs-Url: https://github.com/advanced-security/GSSAR/sessions/aaf20009-ecfb-4c32-8a8e-fc7307710cd0

Co-authored-by: felickz <1760475+felickz@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

@felickz felickz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:octocat: 👍

@felickz felickz merged commit 552239f into main Apr 7, 2026
4 checks passed
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