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
F167884482: D57671.diff
Tue, Aug 25, 6:09 AM
F167833162: D57671.id.diff
Mon, Aug 24, 8:57 PM
Unknown Object (File)
Sat, Aug 22, 5:08 PM
Unknown Object (File)
Sat, Aug 22, 4:37 AM
Unknown Object (File)
Fri, Aug 21, 10:50 PM
Unknown Object (File)
Fri, Aug 21, 1:08 PM
Unknown Object (File)
Thu, Aug 20, 1:31 PM
Unknown Object (File)
Thu, Aug 20, 2:30 AM
View All 67 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...