Fix: Remove insecure proxy fallback and prevent IP spoofing vulnerability #475

Triggered via pull request June 15, 2026 07:19

ArshVermaGit

opened #976

ArshVermaGit:fix-insecure-proxy-fallback-855

Status Failure

Total duration 32s

Artifacts –

ci.yml

on: pull_request

test

30s

Annotations

7 errors and 11 warnings

test

Process completed with exit code 1.

[backend] backend/tests/docs.test.js: backend/tests/docs.test.js#L16

Error: ENOENT: no such file or directory, open '/home/runner/work/peer-learning/docs/api.md' ❯ backend/tests/docs.test.js:16:28 ⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯ Serialized Error: { errno: -2, code: 'ENOENT', syscall: 'open', path: '/home/runner/work/peer-learning/docs/api.md' }

[backend] backend/tests/dispatchPushNotifications.test.js > dispatchPushNotifications — race condition > claimed notifications remain retryable after subscription fetch error: backend/tests/dispatchPushNotifications.test.js#L211

AssertionError: expected 500 to be 200 // Object.is equality - Expected + Received - 200 + 500 ❯ backend/tests/dispatchPushNotifications.test.js:211:23

[backend] backend/tests/dispatchPushNotifications.test.js > dispatchPushNotifications — race condition > returns sent=0, processed=0 when no pending notifications exist: backend/tests/dispatchPushNotifications.test.js#L181

AssertionError: expected 500 to be 200 // Object.is equality - Expected + Received - 200 + 500 ❯ backend/tests/dispatchPushNotifications.test.js:181:24

[backend] backend/tests/dispatchPushNotifications.test.js > dispatchPushNotifications — race condition > single call: returns sent=N, processed=N for N seeded rows: backend/tests/dispatchPushNotifications.test.js#L173

AssertionError: expected 500 to be 200 // Object.is equality - Expected + Received - 200 + 500 ❯ backend/tests/dispatchPushNotifications.test.js:173:24

[backend] backend/tests/dispatchPushNotifications.test.js > dispatchPushNotifications — race condition > all rows have push_sent_at set after concurrent invocations: backend/tests/dispatchPushNotifications.test.js#L168

AssertionError: expected [ { id: 'notif-0', …(6) }, …(4) ] to have a length of +0 but got 5 - Expected + Received - 0 + 5 ❯ backend/tests/dispatchPushNotifications.test.js:168:20

[backend] backend/tests/dispatchPushNotifications.test.js > dispatchPushNotifications — race condition > concurrent calls do not double-deliver: total sent === seeded count: backend/tests/dispatchPushNotifications.test.js#L149

AssertionError: expected 500 to be 200 // Object.is equality - Expected + Received - 200 + 500 ❯ backend/tests/dispatchPushNotifications.test.js:149:25

test

Node.js 20 actions are deprecated. The following actions are running on Node.js 20 and may not work as expected: actions/checkout@v4, actions/setup-node@v4. Actions will be forced to run with Node.js 24 by default starting June 16th, 2026. Node.js 20 will be removed from the runner on September 16th, 2026. Please check if updated versions of these actions are available that support Node.js 24. To opt into Node.js 24 now, set the FORCE_JAVASCRIPT_ACTIONS_TO_NODE24=true environment variable on the runner or in your workflow file. Once Node.js 24 becomes the default, you can temporarily opt out by setting ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION=true. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/

test: src/components/ui/sidebar.tsx#L636