Borehole · Project · free tier
halo-dev/halo
Free reading: not for reliance
0688f95 · 6,376 commits · 3,320 files
Read 2026-10-23

halo-dev/halo: technical due diligence, read from its repository

What Borehole read in halo-dev/halo's code and history at 0688f95: its licence, security advisories, releases, review practice, tests, dependencies and operations. Every finding cites the file and line it rests on. The full report also covers the team and asks the questions the code cannot answer.

DimensionRating
Architecture & Codebase — including AI provenancestrong
Engineering Process & SDLC Maturitystrong
Product & Engineering Maturitystrong
Security & Reliability Postureadequate
Observability & Operationsadequate
Fit with the Acquirernot assessable

What the repository says

  • Releases are cut from another branch than the one surveyed
  • .github/pull_request_template.md:9: states a policy on AI-generated contributions

Architecture & Codebase — including AI provenance

strong
01.6

ui/packages/api-client/src/api/user-v1alpha1-console-api.ts is 1,497 lines, 25× this codebase's median file

adequate

The median source file here is 59 lines. 8 files exceed 1,000 lines, the largest being 1,497. Size is not a defect on its own — some problems genuinely live in one place — but these are where merge conflicts, review fatigue and single-owner knowledge concentrate, and they are the first thing that slows an inheriting engineer down.

Engineering Process & SDLC Maturity

strong

No finding.

Product & Engineering Maturity

strong
04.62

Releases are cut from another branch than the one surveyed

adequate

The newest release, v2.27.1, is not on the branch this survey read; the newest release on it is v2.27.0. The shipped product may differ from what this report describes. Survey the release branch too.

Evidence
tag v2.27.1

Security & Reliability Posture

adequate
05.6

1 Dockerfile runs the application as root

adequate

No USER instruction switches away from root, so the process runs with full privileges inside the container. It is a one-line fix and an ordinary expectation, which is why its absence is worth asking about: it usually says more about whether anyone has reviewed the deployment than about this container specifically.

Evidence
Dockerfile — runs as root
05.25

1 environment file is committed

adequate

ui/.env.production is tracked in git. Environment files are where credentials usually live, and committing one usually means nobody decided how secrets reach the running application. Whatever is in it is in the history for good. Check 05.1 says whether it holds anything credential-shaped.

Evidence
ui/.env.production — committed environment file
05.22

1 deserialiser that can run code

adequate

1 places read data with a deserialiser that executes what it reads (Java object streams). That is safe on data the system wrote itself and remote code execution on anything else. Ask where each one's input comes from.

05.26

28 locked dependencies have published vulnerabilities, 17 high or critical in what ships

adequate

28 of 1,457 locked dependencies match 77 published advisories in OSV, 37 of them rated high or critical. A lockfile says which version is installed, not whether the flawed code is reachable from this product, so each needs a look. Most are fixed by an upgrade, and the advisory names the version.

Evidence
ui/pnpm-lock.yaml — shell-quote 1.9.0: GHSA-pqg4-j6r4-53mv (critical), fixed in 1.11.0; a development dependency
ui/pnpm-lock.yaml — storybook 10.5.7: GHSA-f2ph-cmg5-wxc8 (critical), fixed in 10.6.2; a development dependency
ui/pnpm-lock.yaml — @vue/server-renderer 3.5.41: GHSA-g2v6-rqmx-r4w6 (high), fixed in 3.5.42
05.48

5 places insert HTML from a variable without sanitising it

adequate

5 places set HTML from a value that is neither a literal nor passed through a sanitiser. Where the value can come from a user, that is stored cross-site scripting, and it is the first thing a buyer's security review looks for in a frontend.

05.49

The production build publishes source maps

adequate

The build configuration emits source maps for production, so the original source, with its comments and structure, is downloadable by every visitor. Upload them to an error tracker and keep them off the public server instead.

Evidence
ui/packages/components/vite.config.ts:59 — source maps in a production build
05.65

17 advisories against this project in the last 24 months, 5 high or critical

adequate

The newest was published on 2026-10-21. A project that publishes advisories is telling its users about its fixes, and the count and pace say how much security work it carries. Whether 13 of them are fixed in the code you ship could not be told from this history: its version is only known to be at least 2.27.0. Answer lives in: the release you run, set against each fixed version. The record is incomplete: NVD: 18 records name halo with nothing tying them to this repository (CVE-2004-1539, CVE-2004-1667, CVE-2005-1741, …).

Evidence
2026-10-21: GHSA-7mxx-m76m-ff49 (high), fixed in 2.27.0: Users with delegated role-assignment permissions can escalate privileges to super administrator
2026-09-15: CVE-2026-91772 (moderate): Halo through 2.26.1 contains an open redirect vulnerability in the anonymous thumbnail endpoint that fails to validate the uri query parameter. Attackers can cr
2026-09-08: CVE-2026-78971 (moderate): In Halo <= 2.25.4, the plugin management feature allows users to install/update malicious plugins, which could let attackers execute any command with Halo proce
2026-08-18: CVE-2026-67920 (high): An issue in Halo 2.25.4 allows a remote attacker to execute arbitrary code via the run.halo.app.migration.impl.MigrationServiceImpl.restoreWorkdir(), and org.sp
2026-08-18: CVE-2026-67921 (critical): Cross-Site Request Forgery (CSRF) vulnerability exists in Halo CMS versions up to 2.25.4 via the CorsConfigurer.java and the CsrfConfigurer.java components. Thi

Observability & Operations

adequate
06.2

No structured logging library is in use

adequate

Nothing references a structured logging library. Logs that are written to be read by a person are not searchable by a machine, which means incident response here is proportional to how well someone remembers the codebase rather than to what the tooling can find.

Evidence
searched 3,225 files for 10 structured logging libraries; 0 matched
06.9

No error reporter is wired in

adequate

Nothing in the service initialises an error reporter (Sentry, Rollbar, Bugsnag, Honeybadger or an APM agent). An exception in production reaches a person only if someone is reading the logs, which usually means a customer reports it first.

Evidence
no error reporter initialised
06.17

The code branches on the environment's name at 12 places

adequate

12 places outside configuration test whether they are running in production or staging. Staging then does not exercise production's code, and each deploy is the first run of those branches. Differences belong in configuration the code reads, not in the code.

Evidence
ui/packages/ui-plugin-bundler-kit/src/rsbuild.ts:100 — branch on the environment name
ui/packages/ui-plugin-bundler-kit/src/rsbuild.ts:102 — branch on the environment name
ui/packages/ui-plugin-bundler-kit/src/vite.ts:99 — branch on the environment name

Fit with the Acquirer

not assessable

Not rated: structurally not assessable from a repository.

Read more, or read your own

The full report on halo-dev/halo: every dimension, the questions for the room, and the audit log.

Survey a repository: free on public repositories. Private code is read on your own machine by a collector whose source you can read first.