Grades are a hygiene check, not a security guarantee.

MCP, MCP server

OpenGrok MCP Server

OpenGrok MCP Server, an MCP server listed in the PlugVet directory.

Compatibility: Not stated.

Grade C

One Medium finding. Read the findings before installing.

Graded by PlugVetRubric v2

Not a guarantee. A hygiene check of the source at the commit shown, not a security audit.

Rescanned at commit a97583ce5c06, unchanged since the check below.

NETWORK_HOST_FROM_INPUT

Why this grade

  • NETWORK_HOST_FROM_INPUT Medium

    A file takes a URL or endpoint as input and has no fixed host list.

    src/extension.ts, line 433 * unlike the global fetch() in VS Code's Node.js context.

Any High, or two or more Medium findings, give D. One Medium gives C. Low findings or a security Note give B. Each code names a finding behind the letter.

If you think this grade is wrong, email [email protected] with the entry link and your evidence. We aim to acknowledge within two working days and decide within seven. If we confirm an error, we aim to correct it within 24 hours of confirming it. These are aims, not guarantees.

Grades are a hygiene check of the source at the commit shown, on the date checked, not a security audit or a guarantee. The free directory grades are automated. Before release, a PlugVet engineer reviews every paid report and any grade we hold back, including all F grades, against the source code and the evidence we quote.

What was checked

Plain notes from the hygiene checks.

  • Permissions

    Concern

    can delete files.

    PERMISSION_DELETE_FILES

    esbuild.js, line 284

    try { fs.unlinkSync(path.join(dir, f)); } catch { /* ignore */ }

    scripts/build-vsix.js, line 57

    fs.unlinkSync(backupPath);

    scripts/generate-spec-runner.js, line 27

    fs.rmSync(outfile, { force: true });

    Show more

    src/extension.ts, line 209

    try { fs.unlinkSync(tmpPath); } catch { /* ignore cleanup errors */ }

  • Install and update

    Pass

    No install-time script found, and dependencies are pinned or absent.

  • Network

    Serious

    accepts a host from input, with no fixed list in this file.

    NETWORK_HOST_FROM_INPUT

    src/extension.ts, line 433

    * unlike the global fetch() in VS Code's Node.js context.

    src/server/cli/status.ts, line 215

    const res = await fetch('https://registry.npmjs.org/opengrok-mcp-server/latest', {

    src/server/client/opengrok-client.ts, line 364

    // NOTE: must use undici's fetch (same package as the Agent below).

    Show more

    src/server/client/opengrok-client.ts, line 1091

    // Use a single direct fetch (no pRetry) so 429/5xx don't trigger retry backoff.

    src/server/sandbox/sandbox.ts, line 284

    // Full-file fetch (cached after first call) - a mid-file window can

    src/server/tools/executors.ts, line 525

    // Warn on full-file fetch (no line range) - returned separately so it doesn't contaminate the file text.

    src/server/tools/executors.ts, line 527

    const warning = `Full file fetch (${content.lineCount} lines). Use opengrok_get_file_symbols first, then fetch only the lines you need.`;

    CHANGELOG.md, line 396

    - **`testConnection()` 429 fix**: Rewrote the method to bypass `pRetry` entirely. Previously it called `this.request()` which retried 429 responses 3× with exponential backoff (~7

    CHANGELOG.md, line 417

    `testConnection()` used `this.request()` which throws `AbortError` for any 4xx response (including 401 Unauthorized). The catch block returned `false`, so servers requiring authent

  • Secrets

    Concern

    reads a credential or environment file.

    SECRETS_ENV_READ SECRETS_PATH_DOTENV

    src/server/main.ts, line 35

    const envPassword = process.env['OPENGROK_PASSWORD'] ?? '';

    src/server/main.ts, line 36

    const passwordFile = process.env['OPENGROK_PASSWORD_FILE'] ?? '';

    src/server/cli/status.ts, line 30

    const envPassword = process.env['OPENGROK_PASSWORD'] ?? configOverrides['OPENGROK_PASSWORD'] ?? '';

    Show more

    src/server/cli/status.ts, line 31

    const passwordFile = process.env['OPENGROK_PASSWORD_FILE'] ?? '';

    src/server/cli/status.ts, line 146

    const passwordFilePath = process.env['OPENGROK_PASSWORD_FILE'] ?? configOverrides['OPENGROK_PASSWORD_FILE'] ?? '';

    src/server/cli/setup/configure.ts, line 96

    // Claude Code (~/.claude.json) - projects[cwd].mcpServers['opengrok-mcp'].env

    src/server/cli/setup/configure.ts, line 104

    const env = project.mcpServers?.['opengrok-mcp']?.env;

    src/server/cli/setup/configure.ts, line 117

    const env = data.mcpServers?.['opengrok-mcp']?.env;

    src/server/cli/setup/configure.ts, line 382

    return { ...env, [entry.env]: entry.value };

    src/server/cli/setup/configure.ts, line 445

    if (srv?.env) {

    src/server/cli/setup/configure.ts, line 463

    return data.mcpServers?.['opengrok-mcp']?.env ?? null;

    src/server/cli/setup/configure.ts, line 468

    if (data.mcpServers?.['opengrok-mcp']) data.mcpServers['opengrok-mcp'].env = env;

    src/server/cli/setup/configure.ts, line 500

    if (data.servers?.['opengrok-mcp']?.env) {

  • Agent instructions

    Pass

    No instruction to ignore earlier instructions or to hide a step.

  • Obfuscation

    Note

    a file was over the read cap and was not fully read.

    OBFUSCATION_FILE_OVER_READ_CAP

    package-lock.json, line 1

    {

  • Licence

    Pass

    A licence file is present (LICENSE).

  • Maintainer activity

    Pass

    The latest commit is within the last 365 days.

Get the full report on this tool, £49

Note

Hygiene check of commit a97583ce5c06. can delete files. accepts a host from input, with no fixed list in this file. reads a credential or environment file. a file was over the read cap and was not fully read.

Date and source

Date checked:

View source

github.com

Commit graded: a97583ce5c0694abdc2d2c5ab2cd5abad770b88b

This link goes to the original host. PlugVet does not host or check what you download from there, and installing it is your own decision and risk.

Read how this letter is chosen