Page MenuHomeFreeBSD
Differential D57671

periodic/security: Introduce $security_status_loginfail_ignore for 800.loginfail
ClosedPublic
Actions

Authored by michaelo on Jun 19 2026, 5:24 PM.
Tags
None
Referenced Files
F175299367: D57671.id180195.diff
Fri, Oct 9, 6:55 PM
F175293817: D57671.diff
Fri, Oct 9, 5:56 PM
Unknown Object (File)
Thu, Oct 8, 11:38 PM
Unknown Object (File)
Mon, Oct 5, 11:11 PM
Unknown Object (File)
Mon, Oct 5, 4:13 AM
Unknown Object (File)
Sun, Oct 4, 3:22 AM
Unknown Object (File)
Sat, Oct 3, 2:30 AM
Unknown Object (File)
Thu, Oct 1, 11:20 AM
View All Files
Subscribers
imp
mail_fbsd2.e4m.org
ziaee

Details

Reviewers
jrm
Commits
rG1807800b7986: periodic/security: Introduce $security_status_loginfail_ignore for 800.loginfail
rG624e0bf29d72: periodic/security: Introduce $security_status_loginfail_ignore for 800.loginfail
rG6a7c8c922455: periodic/security: Introduce $security_status_loginfail_ignore for 800.loginfail
Summary

Introduce a new variable "security_status_loginfail_ignore" to filter unwanted
login failure messages from the daily security checks.

Co-authored-by: Michael Osipov <michaelo@FreeBSD.org>
PR: 295191
MFC after: 1 week

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 74068
Build 70951: arc lint + arc unit

Event Timeline

michaelo created this revision.Jun 19 2026, 5:24 PM
Herald added subscribers: ziaee, imp. · View Herald TranscriptJun 19 2026, 5:24 PM
michaelo requested review of this revision.Jun 19 2026, 5:24 PM
Harbormaster completed remote builds in B74012: Diff 180076.Jun 19 2026, 5:24 PM
mail_fbsd2.e4m.org added a subscriber: mail_fbsd2.e4m.org.Jun 20 2026, 4:55 AM
jrm added a comment.EditedJun 20 2026, 7:14 PM

The commit message says "are no" instead of "are not". You might consider something simpler because users can really filter out whatever they want.

Introduce a new variable "security_status_loginfail_ignore" to filter unwanted login failure messages from the daily security checks.

share/man/man5/periodic.conf.5
1041–1043
  1. Rather than egrep(1), say extended regular expression, since egrep is deprecated in POSIX in favor of grep -E.
  2. Simplify the text. Users can really filter out whatever they want.
michaelo updated this revision to Diff 180195.Jun 20 2026, 7:26 PM

@jrm, reasonable points. Thank you!

Harbormaster completed remote builds in B74068: Diff 180195.Jun 20 2026, 7:26 PM
michaelo edited the summary of this revision. (Show Details)Jun 20 2026, 7:26 PM
michaelo marked an inline comment as done.
jrm accepted this revision.Jun 20 2026, 8:24 PM
This revision is now accepted and ready to land.Jun 20 2026, 8:24 PM
Closed by commit rG6a7c8c922455: periodic/security: Introduce $security_status_loginfail_ignore for 800.loginfail (authored by mail_fbsd2.e4m.org, committed by michaelo). · Explain WhyJun 20 2026, 8:35 PM
This revision was automatically updated to reflect the committed changes.
michaelo added a commit: rG6a7c8c922455: periodic/security: Introduce $security_status_loginfail_ignore for 800.loginfail.
michaelo added a commit: rG624e0bf29d72: periodic/security: Introduce $security_status_loginfail_ignore for 800.loginfail.Jun 28 2026, 8:19 PM
michaelo added a commit: rG1807800b7986: periodic/security: Introduce $security_status_loginfail_ignore for 800.loginfail.Jun 28 2026, 8:22 PM

Revision Contents
Changeset List

PathSize
share/
man/
man5/
6 lines
usr.sbin/
periodic/
etc/
security/
1 line

Diff 180195

View Options

share/man/man5/periodic.conf.5

Loading...
View Options

usr.sbin/periodic/etc/security/800.loginfail

Loading...