-
Notifications
You must be signed in to change notification settings - Fork 257
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Delete 'files provider' #7815
Delete 'files provider' #7815
Conversation
c854b46
to
46ecc56
Compare
since 'files provider' is being removed.
46ecc56
to
b8328c8
Compare
|
Added a patch to fix issue reported by 'black' |
:relnote:Previously deprecated `--with-files-provider` configure option and thus support of 'id_provider = files' were removed. For those who still were building SSSD with this feature: see https://sssd.io/docs/files-provider-deprecation.html for replacement details.
CONFDB_RESPONDER_CACHE_FIRST_DEFAILT -> CONFDB_RESPONDER_CACHE_FIRST_DEFAULT
455e574
to
54a58da
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Thank you.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Finally :-) Ack
First patch deletes 'local_negative_timeout' (it was an oversight that option support wasn't ifdef-ed by '--with-files-provider').
Patches are kept split to ease review but maybe squashed if needed.