Skip to content

man: add details about 'an2ln'#8396

Merged
alexey-tikhonov merged 1 commit into
SSSD:masterfrom
sumit-bose:man_localauth_an2ln
Feb 6, 2026
Merged

man: add details about 'an2ln'#8396
alexey-tikhonov merged 1 commit into
SSSD:masterfrom
sumit-bose:man_localauth_an2ln

Conversation

@sumit-bose

Copy link
Copy Markdown
Contributor

With a recent security fix the 'an2ln' module was disabled in SSSD's configuration snippet for the localauth configuration of libkrb5. With this patch the related man page is update accordingly.

@gemini-code-assist gemini-code-assist 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.

Code Review

The pull request effectively updates the sssd_krb5_localauth_plugin.8.xml man page to reflect the recent security fix related to the an2ln module. The added details clearly explain why an2ln is disabled in certain SSSD configurations and the potential security implications of name collisions. It also clarifies the behavior when auth_provider = krb5 is used. The documentation is clear, concise, and provides valuable information to users regarding this change.

Comment thread src/man/sssd_krb5_localauth_plugin.8.xml Outdated
Comment thread src/man/sssd_krb5_localauth_plugin.8.xml Outdated
Comment thread src/man/sssd_krb5_localauth_plugin.8.xml Outdated
With a recent security fix the 'an2ln' module was disabled in SSSD's
configuration snippet for the localauth configuration of libkrb5. With
this patch the related man page is update accordingly.

Reviewed-by: Alexey Tikhonov <[email protected]>
Reviewed-by: Justin Stephenson <[email protected]>
@sssd-bot

Copy link
Copy Markdown
Contributor

The pull request was accepted by @justin-stephenson with the following PR CI status:


🟢 CodeQL (success)
🟢 osh-diff-scan:fedora-rawhide-x86_64:upstream (success)
🟢 rpm-build:centos-stream-10-x86_64:upstream (success)
🟢 rpm-build:fedora-42-x86_64:upstream (success)
🟢 rpm-build:fedora-43-x86_64:upstream (success)
🟢 rpm-build:fedora-rawhide-x86_64:upstream (success)
🟢 Analyze (target) / cppcheck (success)
🟢 Build / freebsd (success)
🟢 Build / make-distcheck (success)
🟢 ci / intgcheck (centos-10) (success)
🟢 ci / intgcheck (fedora-42) (success)
🟢 ci / intgcheck (fedora-43) (success)
🟢 ci / intgcheck (fedora-44) (success)
🟢 ci / prepare (success)
🟢 ci / system (centos-10) (success)
🟢 ci / system (fedora-42) (success)
🟢 ci / system (fedora-43) (success)
🔴 ci / system (fedora-44) (failure)
➖ Coverity scan / coverity (skipped)
🟢 Static code analysis / codeql (success)
🟢 Static code analysis / pre-commit (success)
🟢 Static code analysis / python-system-tests (success)


There are unsuccessful or unfinished checks. Make sure that the failures are not related to this pull request before merging.

@sssd-bot
sssd-bot force-pushed the man_localauth_an2ln branch from 08dd2da to 5a6ce36 Compare January 23, 2026 14:11
@alexey-tikhonov
alexey-tikhonov merged commit bcd9998 into SSSD:master Feb 6, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants