Skip to content

fix: resolve npm security advisories across all sub-packages#5

Draft
Copilot wants to merge 3 commits intomainfrom
copilot/resolve-security-advisory-notifications
Draft

fix: resolve npm security advisories across all sub-packages#5
Copilot wants to merge 3 commits intomainfrom
copilot/resolve-security-advisory-notifications

Conversation

Copy link

Copilot AI commented Mar 12, 2026

Multiple high/critical CVEs across all four npm packages (48+ total vulnerabilities), stemming from outdated direct and transitive dependencies.

Dependency upgrades

  • All packages: axios1.13.6, express4.22.1 (fixes CSRF, SSRF, DoS, body-parser, cookie vulns)
  • notifier: twilio4.23.0
  • pangeagpt-client: parcel2.16.4
  • pangeagpt-server: openai 3.x4.x (v3 bundled a vulnerable internal axios); body-parser1.20.4
  • teams-bot: @microsoft/teamsfx 2.x4.x (fixes @azure/identity EoP); botbuilder4.23.3; added overrides for tough-cookie and xml2js to patch prototype pollution in azurite's transitive deps

Code changes for breaking API upgrades

pangeagpt-server/server.js — openai v3 → v4:

// Before
const { Configuration, OpenAIApi } = require("openai");
const openai = new OpenAIApi(new Configuration({ apiKey: process.env.OPENAI_API_KEY }));
const resp = await openai.createChatCompletion({ model, messages, temperature });
resp.data.choices[0].message.content;

// After
const { OpenAI } = require("openai");
const openai = new OpenAI({ apiKey: process.env.OPENAI_API_KEY });
const resp = await openai.chat.completions.create({ model, messages, temperature });
resp.choices[0].message.content;

teams-bot/.../initialize.js — teamsfx v2 → v4 renamed the adapter:

// Before
const { BotBuilderCloudAdapter } = require("@microsoft/teamsfx");
// After
const { AgentBuilderCloudAdapter } = require("@microsoft/teamsfx");

Remaining issues

5 vulnerabilities remain in teams-bot, all inside the azurite devDependency (@azure/ms-rest-js pinning old axios 0.x and @azure/identity 3.x). These are unfixable without an upstream azurite release and are dev-only (not deployed).


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 2 commits March 12, 2026 04:30
Co-authored-by: kerisha <62780934+kerisha@users.noreply.github.com>
Co-authored-by: kerisha <62780934+kerisha@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix security advisory notification issues fix: resolve npm security advisories across all sub-packages Mar 12, 2026
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