Pre-launch — Gluecron is in final validation. Public signups and git hosting for non-owner users open after launch review.
CodeIssuesPull RequestsActionsSecurityInsightsSettings
✨ AI
More
claude/adoring-hopper-5x74bqclaude/affectionate-feynman-ykrf1hclaude/architecture-audit-design-wxprenclaude/build-status-update-3MXsfclaude/charming-meitner-mllb5rclaude/compare-gate-gluecron-s4mFQclaude/confident-faraday-tikcwbclaude/continue-work-XMTlIclaude/crontech-gluecron-deploy-7MIECclaude/crontech-platform-setup-SeKfwclaude/design-2026claude/ecstatic-ptolemy-jMdigclaude/enhance-github-integration-QNHdGclaude/fix-aa-loop-issue-PonMQclaude/fix-actions-and-processclaude/fix-desktop-errors-XqoW8claude/fix-red-workflowsclaude/fix-website-access-6FKJNclaude/gatetest-integration-hardeningclaude/github-audit-improvements-bDFr9claude/gluecron-launch-status-FoMRlclaude/hopeful-lamport-olfCTclaude/issue-to-pr-and-protectionsclaude/jolly-heisenberg-2sg1Qclaude/launch-preparation-QmTb6claude/new-session-xk1l7claude/plan-platform-architecture-kkN4yclaude/platform-analysis-roadmap-1nUGLclaude/platform-launch-assessment-8dWV8claude/polish-platform-release-AeDrUclaude/resume-previous-work-KzyLwclaude/review-crontech-handoff-qYEVqclaude/review-project-completeness-lHhS2claude/review-readme-docs-ulqPKclaude/serene-edison-rj87weclaude/setup-multi-repo-dev-BCwNQclaude/ship-fixes-and-tests-Jvz1cclaude/site-audit-competitive-pctlwgclaude/site-migration-vercel-XstpKclaude/standalone-product-repos-XHFTDcopilot/feat-smart-empty-states-keyboard-first-enhancementcopilot/feat-smart-morning-digest-review-context-restorecopilot/fix-and-process-workflowscopilot/update-ai-powered-code-reviewfeat/debt-mapfeat/push-policy-codeowners-hardeningfeat/smart-digest-contextfeat/stage-impactfeat/t1-secret-migrationfeat/u-polishfeat/w-self-hostfeat/w2-claude-configfix/agent-journey-orphan-sweepfix/audit-sweep-2026-07-26gatetest/auto-fix-1776586424172gatetest/auto-fix-1776586534814gatetest/auto-fix-1776590685143gatetest/auto-fix-1776590808199mainops/redeploy-retriggerstyle/dxt-cta-themeworktree-agent-a3377aad30d55da26worktree-agent-a7ef607b7ee1d6c74
graphql-email-privacy.test.ts2.6 KB · 68 lines
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
/**
 * GraphQL must not disclose other users' email addresses.
 *
 * `user(username: "...")` selected users.email and returned it verbatim. The
 * resolver is reachable anonymously (POST /api/graphql, softAuth only), so an
 * unauthenticated caller could read the address of any account on the
 * platform — bulk harvesting with one cheap query per username.
 *
 * Verified live 2026-07-28 before the fix:
 *   $ curl -s -X POST https://gluecron.com/api/graphql \
 *       -d '{"query":"{ user(username:\"ccantynz\"){ username email } }"}'
 *   {"data":{"user":{"username":"ccantynz","email":"ccantynz@gmail.com"}}}
 *
 * The `me` resolver remains the supported way to read your own address; it is
 * already scoped to ctx.user.id.
 */

import { describe, expect, it } from "bun:test";
import { readFileSync } from "fs";

const SRC = readFileSync("src/lib/graphql.ts", "utf8");
const userResolver = SRC.slice(
  SRC.indexOf("user: async (args, sel, _ctx)"),
  SRC.indexOf("repository: async (args, sel, _ctx)")
);

describe("user(username:) email exposure", () => {
  it("only returns email to the account holder", () => {
    expect(userResolver).toContain("_ctx?.user?.id === row.id");
    expect(userResolver).toContain("email: isSelf ? row.email : null");
  });

  it("does not pass the raw row through to selection resolution", () => {
    // The bug shape: `resolveSelections(u, sel)` where u came straight from
    // the DB with email attached.
    const idx = userResolver.indexOf("const u = {");
    expect(idx).toBeGreaterThan(-1);
    // The redacted object must be built before anything consumes it.
    const consume = userResolver.indexOf("resolveSelections(u");
    if (consume > -1) expect(idx).toBeLessThan(consume);
  });

  it("nulls the field rather than dropping it", () => {
    // Clients that request `email` should still get a valid response shape.
    expect(userResolver).toContain(": null");
  });
});

describe("the viewer resolver is unaffected", () => {
  it("still scopes to the authenticated user", () => {
    // `viewer`, not `me` — it is the self-lookup and is already gated on
    // ctx.user, so reading your own address keeps working.
    const viewer = SRC.slice(
      SRC.indexOf("viewer: async"),
      SRC.indexOf("user: async")
    );
    expect(viewer).toContain("ctx.user.id");
    expect(viewer).toContain("email: users.email");
    expect(viewer).toContain("if (!ctx.user) return null");
  });
});

describe("no other resolver selects email", () => {
  it("only viewer and user touch users.email", () => {
    const hits = SRC.match(/email: users\.email/g) ?? [];
    expect(hits.length).toBe(2); // viewer (scoped) + user (now redacted)
  });
});