Skip to content

Remove unused http-proxy-middleware package#3713

Merged
thewatermethod merged 2 commits into
mainfrom
mb/TTAHUB-5465/remove-http-proxy-middleware
Jun 18, 2026
Merged

Remove unused http-proxy-middleware package#3713
thewatermethod merged 2 commits into
mainfrom
mb/TTAHUB-5465/remove-http-proxy-middleware

Conversation

@thewatermethod

@thewatermethod thewatermethod commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator

Description of change

Package is flagged by yarn audit but we aren't using it since migration to Vite
Also updated dompurify, cheerio, nodemailer

How to test

CI passes/local dev functions

Jira Issue(s)

Checklists

Every PR

  • Linked Jira issue
  • JIRA issue status updated
  • Code is meaningfully tested
  • Meets accessibility standards (WCAG 2.1 Levels A, AA)
  • API Documentation updated
  • Boundary diagram updated
  • Logical Data Model updated
  • Architectural Decision Records written for major infrastructure decisions
  • UI review complete
  • QA review complete

Before merge to main

  • OHS demo complete
  • Ready to create production PR

Production Deploy

  • PR created as Draft
  • Staging smoke test completed
  • PR transitioned to Open (this ready_for_review transition triggers the Slack/Jira automation)
  • Reviewer added after the PR is Open (elainaparrish is the authorized approver under normal circumstances)
    • Sequence: Draft PR → Smoke test → Open PR (automation runs) → Add reviewer
    • Confirm that the Slack notification was sent after the PR was opened
    • Confirm that linked Jira ticket(s) transitioned as expected; if not, review the GitHub Actions workflow logs

After merge/deploy

  • Update JIRA ticket status

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Removes the unused http-proxy-middleware dependency from the frontend now that the app has migrated to Vite, reducing the dependency surface and addressing the related yarn audit finding.

Changes:

  • Removed http-proxy-middleware from frontend/package.json.
  • Pruned http-proxy-middleware and its transitive dependencies/types from frontend/yarn.lock.

Impact assessment:

  • Benefits: Low risk / medium benefit (dependency + audit noise reduction).
  • Risks: Low (verified no remaining references to http-proxy-middleware / createProxyMiddleware / setupProxy in frontend/).

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
frontend/package.json Removes http-proxy-middleware from declared dependencies.
frontend/yarn.lock Cleans lockfile entries for the removed package and its now-unused transitive deps.

@thewatermethod thewatermethod added this pull request to the merge queue Jun 18, 2026
@thewatermethod thewatermethod removed this pull request from the merge queue due to a manual request Jun 18, 2026
@thewatermethod thewatermethod added this pull request to the merge queue Jun 18, 2026
Merged via the queue into main with commit 442a2d8 Jun 18, 2026
15 checks passed
@thewatermethod thewatermethod deleted the mb/TTAHUB-5465/remove-http-proxy-middleware branch June 18, 2026 16:42
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.

3 participants