Skip to content

[sec-check] fix: harden sshd drop-in — key-only auth, no root password login - #29

Merged
castrojo merged 1 commit into
mainfrom
sec/fix-sshd-hardening
Sep 5, 2026
Merged

castrojo merged 1 commit into
mainfrom
sec/fix-sshd-hardening

Conversation

@kubestellar-hive

Copy link
Copy Markdown
Contributor

Security Fix

Cluster: files/os/ssh/sshd_config.d/bluefin-server.conf, elements/bluefin-server/os-sshd-config.bst (sshd config only — disjoint from open PXE/installer PRs #22/#25/#26/#27).

Shipped sshd drop-in set PermitRootLogin yes + PasswordAuthentication yes, and zz-enable-sshd.preset enables sshd on first boot — exposing brute-forceable remote root login on every deployed host.

Changes:

  • PermitRootLogin prohibit-password (key-only root access)
  • PasswordAuthentication no
  • KbdInteractiveAuthentication no
  • Update element description to match (no longer 'temporary permissive bring-up' config).

Preset left untouched — sshd stays available, but key-only. Full removal of the bring-up preset is a maintainer decision tracked in the issue.

Refs #28


Filed by sec-check agent (ACMM L4/L5 — hold-gated mode). Hold-gated: human review required.

— hive: agent=sec-check backend=copilot model=kimi-k3

…disable password auth

PermitRootLogin yes + PasswordAuthentication yes shipped in the image
exposed brute-forceable remote root login. Switch to key-only auth
(PermitRootLogin prohibit-password, PasswordAuthentication no,
KbdInteractiveAuthentication no).

Refs #28

Signed-off-by: sec-check <sec-check[bot]@users.noreply.github.com>

@hanthor hanthor left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

PermitRootLogin prohibit-password, PasswordAuthentication no, KbdInteractiveAuthentication no — correct hardening. Confirmed only one sshd drop-in file exists in the repo, so there's no conflicting override. CI green.


Generated by Claude Code

@castrojo castrojo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Approved by @castrojo for Hive auto-merge on green CI.

@castrojo
castrojo merged commit f78f7fe into main Sep 5, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hold Work is intentionally paused.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants