Skip to content

fix: update WhatsApp bridge dependencies - #52129

Closed
ljcarreira-galaico wants to merge 1 commit into
NousResearch:mainfrom
ljcarreira-galaico:galaico/whatsapp-bridge-deps-upstream
Closed

fix: update WhatsApp bridge dependencies#52129
ljcarreira-galaico wants to merge 1 commit into
NousResearch:mainfrom
ljcarreira-galaico:galaico/whatsapp-bridge-deps-upstream

Conversation

@ljcarreira-galaico

Copy link
Copy Markdown

Summary

  • Updates the WhatsApp bridge Baileys pin to rc13 to clear GHSA-qvv5-jq5g-4cgg.
  • Updates Express to 4.22.2.
  • Raises protobufjs and ws overrides to patched ranges.

Verification

  • cd scripts/whatsapp-bridge && npm audit --omit=dev → found 0 vulnerabilities.
  • node --check bridge.js → exit 0.
  • npm test --if-present → exit 0.

@ljcarreira-galaico
ljcarreira-galaico requested a review from a team June 24, 2026 20:27
@alt-glitch alt-glitch added type/security Security vulnerability or hardening comp/gateway Gateway runner, session dispatch, delivery platform/whatsapp WhatsApp Business adapter dependencies Pull requests that update a dependency file P2 Medium — degraded but workaround exists duplicate This issue or pull request already exists labels Jun 24, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

This was generated by AI during triage.

Duplicate of #43840, which bumps the WhatsApp bridge Baileys pin to v7.0.0-rc13 clearing the same critical GHSA-qvv5-jq5g-4cgg and was opened earlier (closed PRs #45957 and #43934 were already marked duplicates of it). This PR adds Express/protobufjs/ws audit bumps on top, which are worth folding into the canonical #43840 if useful. Maintainer to pick.

@teknium1

Copy link
Copy Markdown
Contributor

Thanks for the dependency-security update. This is already implemented on current main; closing as superseded by automated hermes-sweeper review.

  • scripts/whatsapp-bridge/package-lock.json:757-759 resolves the published @whiskeysockets/baileys 7.0.0-rc13. GHSA-qvv5-jq5g-4cgg is patched from 7.0.0-rc12, so this satisfies the reported critical advisory.
  • The same lockfile already resolves the additional audit targets at equal or newer versions: Express 4.22.2 (:1069), protobufjs 7.6.5 (:1654), and ws 8.21.0 (:2156).
  • Commit c30c9753b6efc08e154d66b6501a444739df3859 made the registry-based rc13 update and shipped in v2026.7.7.2.
  • This also addresses the earlier duplicate discussion pointing to fix(whatsapp): bump Baileys pin to v7.0.0-rc13 to fix LID-addressed group sends #43840 while preserving the newer registry-install approach on main.

@teknium1 teknium1 closed this Jul 15, 2026
@teknium1 teknium1 added the sweeper:implemented-on-main Sweeper: behavior already present on current main label Jul 15, 2026
@ljcarreira-galaico
ljcarreira-galaico deleted the galaico/whatsapp-bridge-deps-upstream branch August 9, 2026 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/gateway Gateway runner, session dispatch, delivery dependencies Pull requests that update a dependency file duplicate This issue or pull request already exists P2 Medium — degraded but workaround exists platform/whatsapp WhatsApp Business adapter sweeper:implemented-on-main Sweeper: behavior already present on current main type/security Security vulnerability or hardening

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants