File as GitHub Issue repo: survev/survev

Push this scan report to `survev/survev`

Click the green button below to open GitHub’s new-issue form, pre-filled with the report title, summary table, top findings, and an embedded score-card image. No authentication needed — you review on GitHub before submitting. Repobility is credited as the scanner.

Embedded score card image

This image will render at the top of the issue body. Hosted on Repobility, refreshes automatically after re-scans.

Issue title

Server-Side Request Forgery (SSRF) — outbound HTTP from user input

Curate findings to include

Pick exactly which findings appear in the issue body. By default the top 5 are included. Uncheck noise, check what matters.

Top 5 (default)

Severity	Rule	Title	File:line
HIGH	`SEC085`	[SEC085] JS: child_process.exec with non-literal: child_process.exec with user-derived in…	`shared/utils/util.ts:256`
HIGH	`SEC135`	[SEC135] Auth/permission check missing on AI-generated endpoint: Mutating HTTP endpoint g…	`server/src/api/routes/user/UserRouter.ts:34`
HIGH	`SEC100`	[SEC100] CORS permissive Access-Control-Allow-Origin: : Permissive CORS policy (`` orig…	`server/src/api/index.ts:59`
HIGH	`SEC029`	[SEC029] Server-Side Request Forgery (SSRF) — outbound HTTP from user input: Outbound HTT…	`client/src/helpers.ts:208`
HIGH	`SEC029`	[SEC029] Server-Side Request Forgery (SSRF) — outbound HTTP from user input: Outbound HTT…	`client/src/crosshair.ts:16`
HIGH	`SEC029`	[SEC029] Server-Side Request Forgery (SSRF) — outbound HTTP from user input: Outbound HTT…	`client/src/api.ts:4`
HIGH	`SEC128`	[SEC128] Async function without await — fire-and-forget Promise (AI mistake): Async call …	`client/src/errorLogs.ts:71`
HIGH	`SEC128`	[SEC128] Async function without await — fire-and-forget Promise (AI mistake): Async call …	`client/src/debug/graph.ts:116`
HIGH	`SEC128`	[SEC128] Async function without await — fire-and-forget Promise (AI mistake): Async call …	`client/building-editor/src/main.ts:147`
HIGH	`SEC040`	[SEC040] innerHTML XSS — template literal with server-supplied data: Setting .innerHTML w…	`client/src/siteInfo.ts:128`
HIGH	`SEC040`	[SEC040] innerHTML XSS — template literal with server-supplied data: Setting .innerHTML w…	`client/atlas-builder/atlasBuilder.ts:355`
HIGH	`MINED115`	[MINED115] Action `pnpm/action-setup` pinned to mutable ref `@v4`: `uses: pnpm/action-set…	`.github/workflows/server.yml:31`
HIGH	`MINED115`	[MINED115] Action `actions/setup-node` pinned to mutable ref `@v6`: `uses: actions/setup-…	`.github/workflows/server.yml:25`
HIGH	`MINED115`	[MINED115] Action `actions/checkout` pinned to mutable ref `@v6`: `uses: actions/checkout…	`.github/workflows/server.yml:24`
HIGH	`MINED115`	[MINED115] Action `pnpm/action-setup` pinned to mutable ref `@v4`: `uses: pnpm/action-set…	`.github/workflows/tests.yml:32`
HIGH	`MINED115`	[MINED115] Action `actions/setup-node` pinned to mutable ref `@v6`: `uses: actions/setup-…	`.github/workflows/tests.yml:26`
HIGH	`MINED115`	[MINED115] Action `actions/checkout` pinned to mutable ref `@v6`: `uses: actions/checkout…	`.github/workflows/tests.yml:25`
HIGH	`MINED115`	[MINED115] Action `actions/cache/save` pinned to mutable ref `@v4`: `uses: actions/cache/…	`.github/workflows/client.yml:57`
HIGH	`MINED115`	[MINED115] Action `actions/cache` pinned to mutable ref `@v4`: `uses: actions/cache@v4` r…	`.github/workflows/client.yml:42`
HIGH	`MINED115`	[MINED115] Action `pnpm/action-setup` pinned to mutable ref `@v4`: `uses: pnpm/action-set…	`.github/workflows/client.yml:31`
HIGH	`MINED115`	[MINED115] Action `actions/setup-node` pinned to mutable ref `@v6`: `uses: actions/setup-…	`.github/workflows/client.yml:25`
HIGH	`MINED115`	[MINED115] Action `actions/checkout` pinned to mutable ref `@v6`: `uses: actions/checkout…	`.github/workflows/client.yml:24`
HIGH	`MINED115`	[MINED115] Action `actions/github-script` pinned to mutable ref `@v7`: `uses: actions/git…	`.github/workflows/deploy.yml:84`
HIGH	`MINED115`	[MINED115] Action `cloudflare/wrangler-action` pinned to mutable ref `@v3`: `uses: cloudf…	`.github/workflows/deploy.yml:74`
HIGH	`MINED115`	[MINED115] Action `actions/cache/save` pinned to mutable ref `@v4`: `uses: actions/cache/…	`.github/workflows/deploy.yml:67`
HIGH	`MINED115`	[MINED115] Action `actions/cache` pinned to mutable ref `@v4`: `uses: actions/cache@v4` r…	`.github/workflows/deploy.yml:53`
HIGH	`MINED115`	[MINED115] Action `pnpm/action-setup` pinned to mutable ref `@v4`: `uses: pnpm/action-set…	`.github/workflows/deploy.yml:46`
HIGH	`MINED115`	[MINED115] Action `actions/setup-node` pinned to mutable ref `@v4`: `uses: actions/setup-…	`.github/workflows/deploy.yml:41`
HIGH	`MINED115`	[MINED115] Action `actions/checkout` pinned to mutable ref `@v4`: `uses: actions/checkout…	`.github/workflows/deploy.yml:19`
HIGH	`MINED115`	[MINED115] Action `pnpm/action-setup` pinned to mutable ref `@v4`: `uses: pnpm/action-set…	`.github/workflows/bot.yml:32`
HIGH	`MINED115`	[MINED115] Action `actions/setup-node` pinned to mutable ref `@v6`: `uses: actions/setup-…	`.github/workflows/bot.yml:26`
HIGH	`MINED115`	[MINED115] Action `actions/checkout` pinned to mutable ref `@v6`: `uses: actions/checkout…	`.github/workflows/bot.yml:25`
HIGH	`MINED115`	[MINED115] Action `pnpm/action-setup` pinned to mutable ref `@v4`: `uses: pnpm/action-set…	`.github/workflows/lint.yml:33`
HIGH	`MINED115`	[MINED115] Action `actions/setup-node` pinned to mutable ref `@v6`: `uses: actions/setup-…	`.github/workflows/lint.yml:27`
HIGH	`MINED115`	[MINED115] Action `actions/checkout` pinned to mutable ref `@v6`: `uses: actions/checkout…	`.github/workflows/lint.yml:26`
HIGH	`MINED122`	[MINED122] package.json dep `uWebSockets.js` pulled from URL/Git: `dependencies.uWebSocke…	`server/package.json:1`
HIGH	`MINED113`	[MINED113] Express POST /api/report_error has no auth: Express route POST /api/report_err…	`server/src/api/index.ts:185`
HIGH	`MINED113`	[MINED113] Express POST /api/find_game has no auth: Express route POST /api/find_game dec…	`server/src/api/index.ts:85`
HIGH	`MINED113`	[MINED113] Express POST /api/find_game has no auth: Express route POST /api/find_game dec…	`server/src/gameServer.ts:190`
MED	`SEC045`	[SEC045] eval()/exec() on stored or user-supplied data: eval() and exec() on data — even …	`shared/utils/util.ts:256`
MED	`SEC087`	[SEC087] JS: weak Math.random for crypto: Math.random() is not cryptographically secure; …	`server/src/game/riverCreator.ts:17`
MED	`SEC087`	[SEC087] JS: weak Math.random for crypto: Math.random() is not cryptographically secure; …	`client/src/objects/smoke.ts:61`
MED	`SEC087`	[SEC087] JS: weak Math.random for crypto: Math.random() is not cryptographically secure; …	`client/src/errorLogs.ts:23`
MED	`ERR002`	[ERR002] Empty Catch Block: Empty catch blocks hide errors.	`client/src/helpers.ts:156`
MED	`ERR002`	[ERR002] Empty Catch Block: Empty catch blocks hide errors.	`client/src/device.ts:55`
MED	`ERR002`	[ERR002] Empty Catch Block: Empty catch blocks hide errors.	`client/src/config.ts:136`
MED	`AUC001`	[AUC001] No Repobility access matrix policy found: The repository uses web/API frameworks…	—
MED	`AGT007`	localStorage write failures are swallowed silently	`client/src/device.ts:54`
MED	`AGT007`	localStorage write failures are swallowed silently	`client/src/config.ts:159`
MED	`AGT006`	React interval is created without an explicit cleanup	`server/src/teamMenu.ts:373`
MED	`AGT006`	React interval is created without an explicit cleanup	`server/src/stressTest.ts:448`
MED	`AGT006`	React interval is created without an explicit cleanup	`server/src/gameServer.ts:417`
MED	`AGT006`	React interval is created without an explicit cleanup	`server/src/api/index.ts:208`
LOW	`AIC003`	Duplicated implementation block across source files	`shared/defs/maps/woodsSnowDefs.ts:74`
LOW	`AIC003`	Duplicated implementation block across source files	`shared/defs/maps/woodsDefs.ts:177`
LOW	`AIC003`	Duplicated implementation block across source files	`shared/defs/maps/woodsDefs.ts:38`
LOW	`AIC003`	Duplicated implementation block across source files	`shared/defs/maps/snowDefs.ts:82`
LOW	`AIC003`	Duplicated implementation block across source files	`shared/defs/maps/savannahDefs.ts:222`
LOW	`AIC003`	Duplicated implementation block across source files	`shared/defs/maps/potatoSpringDefs.ts:77`
LOW	`AIC003`	Duplicated implementation block across source files	`shared/defs/maps/potatoSpringDefs.ts:24`
LOW	`AIC003`	Duplicated implementation block across source files	`shared/defs/maps/potatoSpringDefs.ts:6`
LOW	`AIC003`	Duplicated implementation block across source files	`shared/defs/maps/potatoDefs.ts:188`
LOW	`AIC003`	Duplicated implementation block across source files	`shared/defs/maps/potatoDefs.ts:103`
LOW	`AIC003`	Duplicated implementation block across source files	`shared/defs/maps/potatoDefs.ts:48`
LOW	`AIC003`	Duplicated implementation block across source files	`shared/defs/maps/potatoDefs.ts:20`
LOW	`AIC003`	Duplicated implementation block across source files	`shared/defs/maps/mainSummerDefs.ts:33`
LOW	`AIC003`	Duplicated implementation block across source files	`shared/defs/maps/mainSummerDefs.ts:13`
LOW	`AIC003`	Duplicated implementation block across source files	`shared/defs/maps/mainSpringDefs.ts:51`
LOW	`AIC003`	Duplicated implementation block across source files	`shared/defs/maps/halloweenDefs.ts:199`
LOW	`AIC003`	Duplicated implementation block across source files	`shared/defs/maps/halloweenDefs.ts:147`
LOW	`AIC003`	Duplicated implementation block across source files	`shared/defs/maps/halloweenDefs.ts:103`
LOW	`AIC003`	Duplicated implementation block across source files	`shared/defs/maps/factionPotatoDefs.ts:29`
LOW	`AIC003`	Duplicated implementation block across source files	`shared/defs/maps/factionDefs.ts:241`
LOW	`AIC003`	Duplicated implementation block across source files	`shared/defs/maps/factionDefs.ts:239`
LOW	`AIC003`	Duplicated implementation block across source files	`shared/defs/maps/desertDefs.ts:39`
LOW	`AIC003`	Duplicated implementation block across source files	`shared/defs/maps/cobaltDefs.ts:123`
LOW	`AIC003`	Duplicated implementation block across source files	`shared/defs/maps/birthdayDefs.ts:23`
LOW	`AIC003`	Duplicated implementation block across source files	`shared/defs/maps/beachDefs.ts:42`
LOW	`AIC003`	Duplicated implementation block across source files	`server/src/api/routes/user/auth/google.…:28`
LOW	`AIC003`	Duplicated implementation block across source files	`client/src/ui/teamMenu.ts:114`
LOW	`AIC003`	Duplicated implementation block across source files	`client/src/ui/opponentDisplay.ts:28`
LOW	`AIC003`	Duplicated implementation block across source files	`client/src/main.ts:281`
LOW	`AIC003`	Duplicated implementation block across source files	`client/src/debug/editor.ts:225`
LOW	`AUC005`	[AUC005] No authorization-focused tests detected: No test files with common authorization…	—
INFO	`MINED043`	[MINED043] Http Not Https: Hardcoded http:// (not localhost) for endpoints that handle cr…	`config.ts:108`
INFO	`MINED043`	[MINED043] Http Not Https: Hardcoded http:// (not localhost) for endpoints that handle cr…	`client/vite.config.mts:51`
INFO	`MINED044`	[MINED044] Js Console Log Prod: console.log left in code. Should be replaced with logger …	`client/src/objects/objectPool.ts:118`
INFO	`MINED044`	[MINED044] Js Console Log Prod: console.log left in code. Should be replaced with logger …	`client/src/inputBinds.ts:139`
INFO	`MINED044`	[MINED044] Js Console Log Prod: console.log left in code. Should be replaced with logger …	`client/src/ambiance.ts:95`
INFO	`MINED045`	[MINED045] Ts Non Null Assertion: x! asserts not null - bypasses null checks - TypeError …	`client/atlas-builder/vitePlugin.ts:93`
INFO	`MINED045`	[MINED045] Ts Non Null Assertion: x! asserts not null - bypasses null checks - TypeError …	`client/atlas-builder/imageWorker.ts:59`
INFO	`MINED045`	[MINED045] Ts Non Null Assertion: x! asserts not null - bypasses null checks - TypeError …	`client/atlas-builder/atlasWorker.ts:103`
INFO	`MINED054`	[MINED054] Ts As Any: Casting to any (as any) bypasses type checking entirely.	`client/src/ui/localization.ts:136`
INFO	`MINED054`	[MINED054] Ts As Any: Casting to any (as any) bypasses type checking entirely.	`client/src/config.ts:188`
INFO	`MINED054`	[MINED054] Ts As Any: Casting to any (as any) bypasses type checking entirely.	`bot/src/commands/helpers.ts:68`
INFO	`MINED052`	[MINED052] Ts Any Typed: : any used as type annotation. Defeats TypeScript type safety.	`shared/utils/util.ts:297`
INFO	`MINED052`	[MINED052] Ts Any Typed: : any used as type annotation. Defeats TypeScript type safety.	`client/src/ui/localization.ts:95`
INFO	`MINED052`	[MINED052] Ts Any Typed: : any used as type annotation. Defeats TypeScript type safety.	`bot/src/commands/helpers.ts:79`

Reset to top 5 98 findings available (after auto-suppression of test files + won't-fix)

Issue body (markdown)

## Code-quality scan: `survev/survev`

**Score: 65/100 (B-)**  ·  98 findings  ·  scanned 2026-06-05 17:13 UTC  ·  127,332 LOC

| Severity | Count |
|---|---|
| CRITICAL | 0 |
| HIGH | 39 |
| MEDIUM | 14 |
| LOW | 31 |

📊 [Full filterable report](https://repobility.com/scan/8afd4c75-4bbf-4caa-92cb-3de68a091de5/)  ·  ![scorecard](https://repobility.com/scan/8afd4c75-4bbf-4caa-92cb-3de68a091de5/report.png?v=1780679627-s2)

### Top findings

1. **HIGH** `SEC085` — JS: child_process.exec with non-literal
   `shared/utils/util.ts:256`
2. **HIGH** `SEC135` — Auth/permission check missing on AI-generated endpoint
   `server/src/api/routes/user/UserRouter.ts:34` · CWE-862
3. **HIGH** `SEC100` — CORS permissive Access-Control-Allow-Origin: *
   `server/src/api/index.ts:59` · A05:2021 Security Misconfiguration
4. **HIGH** `SEC029` — Server-Side Request Forgery (SSRF) — outbound HTTP from user input
   `client/src/helpers.ts:208` · A10:2021 SSRF
5. **HIGH** `SEC029` — Server-Side Request Forgery (SSRF) — outbound HTTP from user input
   `client/src/crosshair.ts:16` · A10:2021 SSRF

---

**Security note**: this issue is public. If any flagged finding is a real, exploitable vulnerability, please redirect to your `SECURITY.md` policy or open a [private security advisory](https://docs.github.com/en/code-security/security-advisories/guidance-on-reporting-and-writing-information-about-vulnerabilities/privately-reporting-a-security-vulnerability) instead. We're happy to close this and re-submit privately.

---

_Filed automatically. Close this issue if not useful — we won't refile. Full report: https://repobility.com/scan/8afd4c75-4bbf-4caa-92cb-3de68a091de5/_

Megaproject â high spam risk

Could not determine 'survev/survev' star count (GitHub API rate-limited or unreachable). When in doubt about repo size, prefer opening a focused PR or a discussion rather than an issue.

Open in GitHub to file this issue Cancel

The button opens GitHubâs new-issue page in a new tab. You will see the title + body pre-filled â review, edit if you want, then click GitHubâs "Submit new issue" button. Repobility never posts anything on your behalf.

For real security findings on big repos: use the project's SECURITY.md or private advisory flow instead of a public issue.

Push this scan report to survev/survev

Embedded score card image

Issue title

Curate findings to include

Issue body (markdown)

Push this scan report to `survev/survev`