Skip to content

chore(security): fill the security-contact placeholders (owner-supplied values) - #33

Merged
hyperpolymath merged 1 commit into
mainfrom
chore/security-contact-fill
Jul 28, 2026
Merged

chore(security): fill the security-contact placeholders (owner-supplied values)#33
hyperpolymath merged 1 commit into
mainfrom
chore/security-contact-fill

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

Fills all 13 {{PLACEHOLDER}} tokens in .github/SECURITY.md + .well-known/security.txt with the owner-ruled values (email security@metadatastician.art, key URL on metadatastician.art, fingerprint ABF3ECD932C2A04345EC9B3D2E725C6B831EA2B8, website). Should turn openssf-compliance green.

Three judgement calls, detailed in the commit message: template-instructions block deleted (it says to), gpg examples re-keyed on the fingerprint (the key's UID is jonathan@, so email-keyed lookups would fail), and the optional Hiring: line dropped (no /careers page — dead link).

Key publication itself rides on the estate-wellknown worker deploy (prepared, awaiting wrangler login).

…ed values)

Fills the 13 {{PLACEHOLDER}} tokens that kept openssf-compliance red, with
values ruled by the owner on 2026-07-27/28:

  SECURITY_EMAIL   security@metadatastician.art
  PGP_KEY_URL      https://metadatastician.art/.well-known/pgp-key.txt
  PGP_FINGERPRINT  ABF3ECD932C2A04345EC9B3D2E725C6B831EA2B8
                   (ed25519, created 2026-07-28, expires 2027-07-28,
                    verified from the owner's exported public key)
  WEBSITE          metadatastician.art

Judgement calls, recorded:
- The TEMPLATE INSTRUCTIONS comment block is deleted (its own text says to
  delete it before publishing, and it contained literal {{...}} tokens that
  would keep the gate red).
- The two gpg example commands were keyed on the SECURITY EMAIL; the key's
  UID is jonathan@metadatastician.art, so 'gpg --fingerprint security@...'
  and '--recipient security@...' would fail. Both now key on the FINGERPRINT,
  which works regardless of UID.
- security.txt's optional 'Hiring:' line is dropped rather than filled — no
  /careers page exists, and RFC 9116 marks the field optional. Shipping a
  dead link helps nobody.

The key itself is being published at the PGP_KEY_URL via the estate-wellknown
Cloudflare worker (which generates /.well-known/security.txt dynamically for
the estate domains); until that deploy lands the URL 404s — the repo files
are correct either way.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@gitar-bot

gitar-bot Bot commented Jul 28, 2026

Copy link
Copy Markdown
CI failed: CI failed because the newly added '.githooks/' directory at the repository root violates the root shape check allowlist.

Overview

1 change-related failure found across 1 analyzed log. The root shape validation check rejected the newly introduced .githooks/ directory in the repository root.

Failures

Root Shape Check Failure (confidence: high)

  • Type: build
  • Affected jobs: 90186660655
  • Related to change: yes
  • Root cause: The script scripts/check-root-shape.sh failed because the newly added directory .githooks/ is not present on the allowed root entries list.
  • Suggested fix: Add .githooks/ to the root allowlist file (e.g., .machine_readable/root-allow.txt) or move the files into an appropriate existing subdirectory.

Summary

  • Change-related failures: 1 failure due to an unallowed directory (.githooks/) added at the repository root.
  • Infrastructure/flaky failures: 0
  • Recommended action: Update the root shape allowlist to include .githooks/ or relocate the files.
Code Review ✅ Approved

Replaces placeholders in SECURITY.md and security.txt with owner-supplied security contact details to achieve OpenSSF compliance. No issues found.

Auto-approved and auto-merge armed: No blocking issues found.
Please see Auto-approve Docs for details on setting custom approval criteria. — merges when pipeline and required approvals pass.

Tip

Comment Gitar fix CI or enable auto-apply: gitar auto-apply:on

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing less information.

Comment with these commands to change the behavior for this request:

Auto-apply Compact
gitar auto-apply:on         
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

@sonarqubecloud

Copy link
Copy Markdown

@gitar-bot

gitar-bot Bot commented Jul 28, 2026

Copy link
Copy Markdown

⚠️ Gitar auto-approved this PR but could not enable auto-merge: auto-merge is disabled for this repository — enable "Allow auto-merge" in the repository settings.

@gitar-bot gitar-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gitar has auto-approved this PR and enabled auto-merge (configure)

@gitar-bot gitar-bot Bot added the gitar-approved Added by Gitar label Jul 28, 2026
@hyperpolymath
hyperpolymath merged commit 0af1a2f into main Jul 28, 2026
42 of 45 checks passed
@hyperpolymath
hyperpolymath deleted the chore/security-contact-fill branch July 28, 2026 05:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gitar-approved Added by Gitar

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant