Bug 60628 - Tweak login policy constants to handle IPA HBAC
Summary: Tweak login policy constants to handle IPA HBAC
Status: RESOLVED FIXED
Alias: None
Product: realmd
Classification: Unclassified
Component: General (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: Stef Walter
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 55041
Blocks: 56023 62518
  Show dependency treegraph
 
Reported: 2013-02-11 07:00 UTC by Stef Walter
Modified: 2013-04-12 11:59 UTC (History)
3 users (show)

See Also:
i915 platform:
i915 features:


Attachments
Support realm login policy (11.15 KB, patch)
2013-03-26 13:41 UTC, Stef Walter
Details | Splinter Review

Description Stef Walter 2013-02-11 07:00:15 UTC
We should add a login policy constant to the DBus API to reflect using IPA HBAC as a login policy. This will be the default deployed by realmd used with sssd and ipa.
Comment 1 Stef Walter 2013-03-26 13:41:43 UTC
Created attachment 77049 [details] [review]
Support realm login policy
Comment 2 Stef Walter 2013-04-10 13:24:46 UTC
Yassir, is this something you have time review?
Comment 3 Jakub Hrozek 2013-04-12 11:54:34 UTC
Comment on attachment 77049 [details] [review]
Support realm login policy

Review of attachment 77049 [details] [review]:
-----------------------------------------------------------------

::: service/realm-sssd-ad.c
@@ +229,4 @@
>  
>  	                                    "id_provider", "ad",
>  	                                    "auth_provider", "ad",
> +	                                    "access_provider", "ad",

Just a word of warning -- The AD access control provider checks if the account is expired. It has the same effect as the following configuration of the LDAP provider:

access_provider = ldap
ldap_access_order = expire
ldap_account_expire_policy = ad

The IPA changes look good to me.
Comment 4 Stef Walter 2013-04-12 11:57:22 UTC
(In reply to comment #3)
> > +	                                    "access_provider", "ad",
> 
> Just a word of warning -- The AD access control provider checks if the
> account is expired.

Good. I think that's what would be expected. To follow the login policy of the domain in this case.
Comment 5 Stef Walter 2013-04-12 11:59:08 UTC
Attachment 77049 [details] pushed as cf1602d - Support realm login policy


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.