Skip to content

Add PermitRootLogin prohibit-password and forced-commands-only - #1111

Merged
ejohnstown merged 1 commit into
wolfSSL:masterfrom
stenslae:pr-permit-root-login
Jul 31, 2026
Merged

Add PermitRootLogin prohibit-password and forced-commands-only#1111
ejohnstown merged 1 commit into
wolfSSL:masterfrom
stenslae:pr-permit-root-login

Conversation

@stenslae

@stenslae stenslae commented Jul 20, 2026

Copy link
Copy Markdown
Member

Expanded PermitRootLogin configuration to fully support OpenSSH settings prohibit-password (including legacy without-password) and forced-commands-only.

Added additional test coverage for this new feature.

@stenslae stenslae self-assigned this Jul 20, 2026

@wolfSSL-Fenrir-bot wolfSSL-Fenrir-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.

Fenrir Automated Review — PR #1111

Scan targets checked: wolfssh-bugs, wolfssh-src

Findings: 1
1 finding(s) posted as inline comments (see file-level comments below)

This review was generated automatically by Fenrir. Findings are non-blocking.

Comment thread apps/wolfsshd/auth.c Outdated
Comment thread apps/wolfsshd/auth.c Outdated
@stenslae
stenslae force-pushed the pr-permit-root-login branch 3 times, most recently from 9e1bbd9 to 71e1e28 Compare July 21, 2026 16:18

@wolfSSL-Fenrir-bot wolfSSL-Fenrir-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.

Fenrir Automated Review — PR #1111

Scan targets checked: wolfssh-bugs, wolfssh-src

No new issues found in the changed files. ✅

@stenslae stenslae assigned wolfSSL-Bot and unassigned stenslae Jul 21, 2026
@stenslae
stenslae force-pushed the pr-permit-root-login branch from 71e1e28 to 9937fff Compare July 21, 2026 17:41
@stenslae

stenslae commented Jul 21, 2026

Copy link
Copy Markdown
Member Author

Rebased onto latest master, which added a72b944 (roots PermitRootLogin on uid 0). Extracted into a shared IsRootUser() and pointed my prohibit-password / forced-commands-only checks at it too.

Also dropped my fake-crypt timing mitigation for invalid users, will make a dedicated PR for the timing oracle mitigations.

@stenslae stenslae assigned stenslae and unassigned wolfSSL-Bot Jul 21, 2026

@wolfSSL-Fenrir-bot wolfSSL-Fenrir-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.

Fenrir Automated Review — PR #1111

Scan targets checked: wolfssh-bugs, wolfssh-src

No new issues found in the changed files. ✅

@stenslae stenslae assigned wolfSSL-Bot and unassigned stenslae Jul 21, 2026
@stenslae
stenslae force-pushed the pr-permit-root-login branch 2 times, most recently from ebb3394 to 9cc97ed Compare July 23, 2026 16:37
@stenslae
stenslae requested review from wolfSSL-Fenrir-bot and removed request for wolfSSL-Fenrir-bot July 23, 2026 17:20

@wolfSSL-Fenrir-bot wolfSSL-Fenrir-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.

Fenrir Automated Review — PR #1111

Scan targets checked: wolfssh-bugs, wolfssh-src

Findings: 2
2 finding(s) posted as inline comments (see file-level comments below)

This review was generated automatically by Fenrir. Findings are non-blocking.

Comment thread apps/wolfsshd/wolfsshd.c
Comment thread apps/wolfsshd/auth.c Outdated
Comment thread apps/wolfsshd/wolfsshd.c
Comment thread apps/wolfsshd/auth.c Outdated
@stenslae
stenslae force-pushed the pr-permit-root-login branch from 9cc97ed to c05b9c4 Compare July 27, 2026 16:44

@wolfSSL-Fenrir-bot wolfSSL-Fenrir-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.

Fenrir Automated Review — PR #1111

Scan targets checked: wolfssh-bugs, wolfssh-src

Findings: 1
1 finding(s) posted as inline comments (see file-level comments below)

This review was generated automatically by Fenrir. Findings are non-blocking.

Comment thread apps/wolfsshd/wolfsshd.c Outdated
@stenslae
stenslae force-pushed the pr-permit-root-login branch from c05b9c4 to ad3840e Compare July 27, 2026 17:01
Comment thread apps/wolfsshd/wolfsshd.c Outdated

@wolfSSL-Fenrir-bot wolfSSL-Fenrir-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.

Fenrir Automated Review — PR #1111

Scan targets checked: wolfssh-bugs, wolfssh-src

No new issues found in the changed files. ✅

@stenslae
stenslae force-pushed the pr-permit-root-login branch from ad3840e to 3c3933c Compare July 29, 2026 21:00

@wolfSSL-Fenrir-bot wolfSSL-Fenrir-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.

Fenrir Automated Review — PR #1111

Scan targets checked: wolfssh-bugs, wolfssh-src

No new issues found in the changed files. ✅

@stenslae
stenslae force-pushed the pr-permit-root-login branch from 3c3933c to 58f7070 Compare July 31, 2026 21:17

@wolfSSL-Fenrir-bot wolfSSL-Fenrir-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.

Fenrir Automated Review — PR #1111

Scan targets checked: wolfssh-bugs, wolfssh-src

No new issues found in the changed files. ✅

@stenslae
stenslae force-pushed the pr-permit-root-login branch from 58f7070 to 80bfe96 Compare July 31, 2026 22:12
@ejohnstown
ejohnstown merged commit 2206342 into wolfSSL:master Jul 31, 2026
148 of 149 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants