SSSD / sssd

A daemon to manage identity, authentication and authorization for centrally-managed systems.
https://sssd.io
GNU General Public License v3.0
612 stars 249 forks source link

sss_override does not take precedence over override_shell directive #7529

Open nassarmu opened 3 months ago

nassarmu commented 3 months ago

This is similar to issue #5589. sss_override shells are completely ignored if an override_shell is specified. The behavior of sssd is now inconsistent: sss_override homedir takes precedence over sssd.conf, but sss_override shell does not.

Tested with sssd Ubuntu 24.04 2.9.4-1.1ubuntu6.1, Ubuntu 20.04 2.6.3-1ubuntu3.3 and on CentOS8.10 2.9.4

sumit-bose commented 3 months ago

Hi,

thank you for the report. I tend to agree that a similar ordering as for the home directory options from #5589 would make sense. If no other developer has any concerns would you be interested to provide a patch?

bye, Sumit

nassarmu commented 3 months ago

Sumit, I looked at the code and i'm afraid it is just beyond my capabilities but i can try.