<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN" "http://www.w3.org/TR/REC-html40/loose.dtd">
<html lang="en">
<head>
<meta content="text/html; charset=US-ASCII" http-equiv="Content-Type">
<title>
GitLab
</title>



<style>img {
max-width: 100%; height: auto;
}
</style>
</head>
<body>
<div class="content">

<h3>
Timo Aaltonen pushed to branch experimental
at <a href="https://salsa.debian.org/sssd-team/sssd">Debian SSSD packaging / sssd</a>
</h3>
<h4>
Commits:
</h4>
<ul>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/2f34087cf593389557c7d5e66a2947fb80e0901d">2f34087c</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-08-13T14:25:01Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">Updating the version before the 2.0 release
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/7e9f0a0c99bc8eace31f44c4c5e9ce7e13d76680">7e9f0a0c</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-08-13T14:59:01Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">include stdarg.h directly in debug.h

It expects to be already included which may break compilation
if  it is not true.

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/40e3863ef226eac58d74ea7fcb25dcfa99672dc5">40e3863e</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-08-13T14:59:04Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">pam_add_response: fix talloc context

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/c2ed0caee434accef1999e5794daa12322c73bb7">c2ed0cae</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-08-13T14:59:09Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sss_ptr_hash: add sss_ptr_get_value to make it useful in delete callbacks

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/9c9a432833476053c46859ae67b0cfa19a41efa2">9c9a4328</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-08-13T14:59:12Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sss_ptr_list: add linked list of talloc pointers

A list item is automatically freed and removed from the list when
the talloc pointer is freed.

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/e347b55571e1f08b48865e06a86d2f009442e10a">e347b555</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-08-13T14:59:14Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sbus: move sbus code to standalone library

This will allow to use both old and new sbus implementation during
conversion.

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/564c0798adbcfe132509d5245ea5b7fa2e4d3b51">564c0798</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-08-13T14:59:17Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sbus: add sbus sssd error codes

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/b49ee1bfc3b541b08d7c7b7188d8bde3dbd1bd44">b49ee1bf</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-08-13T14:59:21Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sbus: add new implementation

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/7f3ed078768d39153ef7929bd97fae047cf18965">7f3ed078</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-08-13T14:59:24Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sbus: build new sbus implementation

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/f91e90a7690e928e127a6f1020bb125c79377992">f91e90a7</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-08-13T14:59:27Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sbus: disable generating old api

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/06631b456011c630029f6fd2d1c3e97d95bab710">06631b45</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-08-13T14:59:29Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sbus: fix indirect includes in sssd

Some header files were included indirectly throught data provider
interface headers. When we switch backend to new sbus, we got lots
of undefined symbols errors.

We now include the files directly so we can convert the backend.

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/2963f2d917bd904f72b11daf400293ba518c74a0">2963f2d9</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-08-13T14:59:33Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sbus: add sss_iface library

This will merge sbus interface of all processes into one place.

SSSD is already a lot interconnected through and almost any process
needs acces to interface of all other processes (all processes talks
to monitor and data provider, data provider talks to some responders).
We would also like to make responders talks between each other.

Therefore it is much simpler to create one place for all the internal
D-Bus API. This will make it simpler to use, easier to build and it
will also reduce amount of generated code since most of the invokers
can be reused.

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/924f8098333e07d7bc2675f93cf4e57ec489a194">924f8098</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-08-13T14:59:47Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sbus: convert monitor

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/c7e2d7a56d8969f26921811e5122f78b257fb51e">c7e2d7a5</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-08-13T14:59:54Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sbus: convert backend

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/e50fb8ace7e1af34bd0742a3683343e5f0055ae9">e50fb8ac</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-08-13T14:59:57Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sbus: convert responders

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/de3a63c4b8e55ae2dbd3fbf12ca868c854ed8052">de3a63c4</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-08-13T15:00:00Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sbus: convert proxy provider

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/fbe2476a3dd9be83ffa85c29dca26f734618d72d">fbe2476a</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-08-13T15:00:03Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sbus: convert infopipe

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/aaecabf2d0e7a0732e4169b9b7c09a266603abd8">aaecabf2</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-08-13T15:00:06Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sbus: convert sssctl

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/5edba6ce412f5be57a36fac1a7bd2b884c36ddf3">5edba6ce</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-08-13T15:00:09Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sbus: remove old implementation

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/7c1dd71c31faa00039f764af45a28e8668731769">7c1dd71c</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-08-13T15:00:12Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sbus: add new internal libraries to specfile

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/3d1b6458568f3df4d5c192f432e73d65e4a9d293">3d1b6458</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-08-13T15:00:14Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sbus: make tests run

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/c0c8499b6ac36b44e3db7e8c1d44ce32f38662b5">c0c8499b</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-08-13T15:00:17Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">tests: disable parse_inp_call_dp, parse_inp_call_attach in responder-get-domains-tests

New sbus implementation broke responder-get-domains-tests and we could
not find how to fix it in time.

Ticket to track fixing these tests:
https://pagure.io/SSSD/sssd/issue/3806

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/1adf2f982c28096ca03b9910a412a8a63559ec10">1adf2f98</a></strong>
<div>
<span>by amitkuma</span>
<i>at 2018-08-13T15:03:45Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">confdb: Remove CONFDB_DOMAIN_LEGACY_PASS

As CONFDB_DOMAIN_LEGACY_PASS is legacy parameter that is rooted in
pre-1.0 SSSD, let's just nuke it now as we're already removing other
legacy code.

Resolves:
https://pagure.io/SSSD/sssd/issue/3530

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/a2d543f61266f7ec0095a5395a4ea38c8e1426d5">a2d543f6</a></strong>
<div>
<span>by Amit Kumar</span>
<i>at 2018-08-13T15:06:57Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">providers: disable ldap_sudo_include_regexp by default

Using wildcard in sudoHost attribute is very costly to evaluate on LDAP
server side and this features seems to be rarely used. Considering this,
let's make it **not** enabled by default.

Whoever is still interested on using it, can just add to their
sssd.conf:
`ldap_sudo_include_regexp = true`.

Signed-off-by: Fabiano Fidêncio <fidencio@redhat.com>

Reviewed-by: Pavel Březina <pbrezina@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/65bd6bf05d75c843e525f8bf89e9b75b02a2bfb7">65bd6bf0</a></strong>
<div>
<span>by Fabiano Fidêncio</span>
<i>at 2018-08-13T15:19:47Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">providers: drop ldap_{init,}groups_use_matching_rule_in_chain support

Resolves:
https://pagure.io/SSSD/sssd/issue/3492

Signed-off-by: Fabiano Fidêncio <fidencio@redhat.com>

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/5dafa8177326e032503a3f8195eb0d0dd4cb1b15">5dafa817</a></strong>
<div>
<span>by Fabiano Fidêncio</span>
<i>at 2018-08-13T15:21:02Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">ldap: remove parallel requests from rfc2307bis

As this branch of code is not well tested, we've decided to just nuke it
and, in the future, spend more time on improving the performance for the
general case instead of maintaining this old "hack".

Resolves:
https://pagure.io/SSSD/sssd/issue/3494

Signed-off-by: Fabiano Fidêncio <fidencio@redhat.com>

Reviewed-by: Pavel Březina <pbrezina@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/7d483737f3763947a5b8a43efbfaf31f54f956a4">7d483737</a></strong>
<div>
<span>by Fabiano Fidêncio</span>
<i>at 2018-08-13T16:29:12Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">tests: adapt common_dom to files_provider

Also start to consider the "files" provider when cleaning up the files.
This change will let us start to moving to "files" provider in our test
suite.

Related:
https://pagure.io/SSSD/sssd/issue/3304

Signed-off-by: Fabiano Fidêncio <fidencio@redhat.com>

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
Reviewed-by: Pavel Březina <pbrezina@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/2243b348976705ca81da954cb1ead1f406d01918">2243b348</a></strong>
<div>
<span>by Fabiano Fidêncio</span>
<i>at 2018-08-13T16:29:19Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">tests: adapt test_sysdb_views to files provider

Let's take advantage of the files provider and start to get rid of the
local provider references in our code.

Apart from the obvious change from "local" to "files" and from "LOCAL"
to "FILES", we're also passing an on-the-fly created uid/gid to the
sysdb_store_user() function in order to avoid calling
sysdb_get_new_id(), which only should be used with the "local" provider.

Related:
https://pagure.io/SSSD/sssd/issue/3304

Signed-off-by: Fabiano Fidêncio <fidencio@redhat.com>

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
Reviewed-by: Pavel Březina <pbrezina@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/35a200d5bd84d6a6742e77ffda9945e37134d721">35a200d5</a></strong>
<div>
<span>by Fabiano Fidêncio</span>
<i>at 2018-08-13T16:29:35Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">tests: adapt sysdb-tests to files_provider

Let's take advantage of the files provider and start to get rid of the
local provider references in our code.

Apart from the obvious change from "local" to "files" and from "LOCAL"
to "FILES", we're also passing a static uid/gid to the sysdb_add_user()
function in order to avoid calling sysdb_get_new_id(), which only should
be used with the "local" provider. Another change doneis to explicitly
set mpg to true as it was enabled by default when using the "local"
provider.

Related:
https://pagure.io/SSSD/sssd/issue/3304

Signed-off-by: Fabiano Fidêncio <fidencio@redhat.com>

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
Reviewed-by: Pavel Březina <pbrezina@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/6ebcc59b9458c5a118cd182c07a840fa6756a43e">6ebcc59b</a></strong>
<div>
<span>by Fabiano Fidêncio</span>
<i>at 2018-08-13T16:29:42Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">tests: adapt sysdb_ssh tests to files provider

Let's take advantage of the files provider and start to get rid of the
local provider references in our code.

Apart from the obvious change from "local" to "files" and from "LOCAL"
to "FILES", we also had to change the cleanup function as this test
suite doesn't rely on test_multidom_suite_cleanup().

Related:
https://pagure.io/SSSD/sssd/issue/3304

Signed-off-by: Fabiano Fidêncio <fidencio@redhat.com>

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
Reviewed-by: Pavel Březina <pbrezina@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/064ca0b46eed06977dbd32c2bd894f71dc43be2e">064ca0b4</a></strong>
<div>
<span>by Fabiano Fidêncio</span>
<i>at 2018-08-13T16:29:49Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">tests: adapt auth-tests to files provider

Let's take advantage of the files provider and start to get rid of the
local provider references in our code.

Apart from the obvious change from "local" to "files" and from "LOCAL"
to "FILES", we also had to change the cleanup function as this test
suite doesn't rely on test_multidom_suite_cleanup().

Related:
https://pagure.io/SSSD/sssd/issue/3304

Signed-off-by: Fabiano Fidêncio <fidencio@redhat.com>

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
Reviewed-by: Pavel Březina <pbrezina@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/a8a9e66a852ba8aa1371e2e1897d49f3feee76da">a8a9e66a</a></strong>
<div>
<span>by Fabiano Fidêncio</span>
<i>at 2018-08-13T16:29:55Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">tests: adapt tests_fqnames to files provider

Let's take advantage of the files provider and start to get rid of the
local provider references in our code.

Only a simple change from "local" to "files" was required.

Related:
https://pagure.io/SSSD/sssd/issue/3304

Signed-off-by: Fabiano Fidêncio <fidencio@redhat.com>

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
Reviewed-by: Pavel Březina <pbrezina@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/99b5bb54447287da5031dcd2bfebeef8b47d52d7">99b5bb54</a></strong>
<div>
<span>by Fabiano Fidêncio</span>
<i>at 2018-08-13T16:30:09Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sysdb: sanitize the dn on cleanup_dn_filter

When trying to get rid of the "local" provider references in
ldap_id_cleanup tests by switching to "files" provider, it's been
noticed that the linearized dn wasn't coming sanitized, which would make
the test to fail.

While I'm not sure here's the right place to have it fixed, this is the
simplest proposal I could come up with.

Related:
https://pagure.io/SSSD/sssd/issue/3304

Signed-off-by: Fabiano Fidêncio <fidencio@redhat.com>

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
Reviewed-by: Pavel Březina <pbrezina@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/728e4be10fc7a7b3173b74b1f597e835914c9abf">728e4be1</a></strong>
<div>
<span>by Fabiano Fidêncio</span>
<i>at 2018-08-13T16:30:16Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sysdb: pass subfilter and ts_subfilter to sysdb_search_*_by_timestamp()

The reason for specifically passing a ts_subfilter is because when using
a provider that relies on a cache and on a timestamp cache, the search
done each cache is different.

The difference in the search is that on timestamp cache we add a
(dateExpireTimestamp <= XXX), but it shouldn't be added to the cache
search.

This commit is needed in order to have the ldap_id_cleanup test running
when switching from local provider (which has no timestamp cache) to the
files provider (which exposed this issue).

Related:
https://pagure.io/SSSD/sssd/issue/3304

Signed-off-by: Fabiano Fidêncio <fidencio@redhat.com>

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
Reviewed-by: Pavel Březina <pbrezina@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/2e8fe6a3dd022721e1ff476c8fd7af317a535d63">2e8fe6a3</a></strong>
<div>
<span>by Fabiano Fidêncio</span>
<i>at 2018-08-13T16:30:27Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">tests: adapt test_ldap_id_cleanup to files provider

Let's take advantage of the files provider and start to get rid of the
local provider references in our code.

The only changes requred were the obvious change from "local" to "files"
and from "LOCAL" to "FILES".

Related:
https://pagure.io/SSSD/sssd/issue/3304

Signed-off-by: Fabiano Fidêncio <fidencio@redhat.com>

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
Reviewed-by: Pavel Březina <pbrezina@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/a24f0c202ed7ea159ad04eb19b506b7f21ad41cc">a24f0c20</a></strong>
<div>
<span>by Fabiano Fidêncio</span>
<i>at 2018-08-13T16:30:35Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">tests: remove LOCAL_SYSDB_FILE reference from test_sysdb_certmap

Let's take advantage of the files provider and start to get rid of the
local provider references in our code.

Let's just replace "LOCAL_SYSDB_FILE" for NULL as This test suite
doesn't create local sysdb file.

Related:
https://pagure.io/SSSD/sssd/issue/3304

Signed-off-by: Fabiano Fidêncio <fidencio@redhat.com>

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
Reviewed-by: Pavel Březina <pbrezina@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/5a87af9128c5468ce1f2e55602c9d3ec05987b8d">5a87af91</a></strong>
<div>
<span>by Fabiano Fidêncio</span>
<i>at 2018-08-13T16:30:41Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">tests: remove LOCAL_SYSDB_FILE reference from test_sysdb_domain_resolution_order_

Let's take advantage of the files provider and start to get rid of the
local provider references in our code.

Let's just replace "LOCAL_SYSDB_FILE" for NULL as This test suite
doesn't create local sysdb file.

Related:
https://pagure.io/SSSD/sssd/issue/3304

Signed-off-by: Fabiano Fidêncio <fidencio@redhat.com>

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
Reviewed-by: Pavel Březina <pbrezina@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/15342ebe8c22738e947728f06c6833b5be09eca8">15342ebe</a></strong>
<div>
<span>by Fabiano Fidêncio</span>
<i>at 2018-08-13T16:30:54Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">tests: remove LOCAL_SYSDB_FILE reference from test_sysdb_subdomains

Let's take advantage of the files provider and start to get rid of the
local provider references in our code.

Let's just replace "LOCAL_SYSDB_FILE" for NULL as This test suite
doesn't create local sysdb file.

Related:
https://pagure.io/SSSD/sssd/issue/3304

Signed-off-by: Fabiano Fidêncio <fidencio@redhat.com>

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
Reviewed-by: Pavel Březina <pbrezina@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/c075e2865b2fbedb6a7eeb6a90780c6dbffeb5fd">c075e286</a></strong>
<div>
<span>by Fabiano Fidêncio</span>
<i>at 2018-08-13T16:31:01Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">tests: remove LOCAL_SYSDB_FILE reference from common_dom

As all tests are now taking advantage of the "files" provider instead of
the "local" one, let's just remove the last reference of
LOCAL_SYSDB_FILE from our tests.

Together with the reference, let's also remove the whole if-block as
we're not relying on "local" provider anymore.

Related:
https://pagure.io/SSSD/sssd/issue/3304

Signed-off-by: Fabiano Fidêncio <fidencio@redhat.com>

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
Reviewed-by: Pavel Březina <pbrezina@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/b8946c46e04e8b5329e4d95f9a67b4a76f883bb2">b8946c46</a></strong>
<div>
<span>by Fabiano Fidêncio</span>
<i>at 2018-08-13T16:31:08Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">local: build local provider conditionally

Let's "get rid" of the local provider and only build it conditionally.

The local provider is only used by our integration tests and those will
be ran in the CI enabling the local provider.

If someone, for some reason, still needs to use it,
"--enable-local-provider" has been added as a configure option and the
provider can be built using that (as done in our integration tests).

Resolves:
https://pagure.io/SSSD/sssd/issue/3304

Signed-off-by: Fabiano Fidêncio <fidencio@redhat.com>

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
Reviewed-by: Pavel Březina <pbrezina@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/82d51b7feba8df49df3f7a023d2b16aba26a858e">82d51b7f</a></strong>
<div>
<span>by Fabiano Fidêncio</span>
<i>at 2018-08-13T16:31:16Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">pysss: fix typo in comment

"local" -> "password"

Signed-off-by: Fabiano Fidêncio <fidencio@redhat.com>

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
Reviewed-by: Pavel Březina <pbrezina@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/0e211b8ba30c3adcdeef21ca1339b194cbfffb04">0e211b8b</a></strong>
<div>
<span>by Fabiano Fidêncio</span>
<i>at 2018-08-13T16:31:23Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">pysss: remove pysss.local

The pysss.local interface has been de-emphasized in favour of the files
domain. As there's no current consumer of this API, let's just remove
it.

Resolves:
https://pagure.io/SSSD/sssd/issue/3493

Signed-off-by: Fabiano Fidêncio <fidencio@redhat.com>

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
Reviewed-by: Pavel Březina <pbrezina@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/4d7f07893fb3bcf9c39ec0ff318c73529f7f8278">4d7f0789</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-08-13T16:39:45Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">TESTS: the sys package was used but not imported

The test used sys.exit but never imported the sys module.

Related:
https://pagure.io/SSSD/sssd/issue/3685

Reviewed-by: Fabiano Fidêncio <fidencio@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/aafaacd59d3ce935c4ca0a73a140f1e6f4cd49b0">aafaacd5</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-08-13T16:39:51Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">TESTS: Remove tests database in teardown

This is to avoid one test indirectly stepping on another if they are run
individually. For example, some of the tests added a principal and ran
kinit which populated the database, another test started execution by
checking there is nothing stored in the secrets database.

In general, it's better to run the tests in isolation.

Related:
https://pagure.io/SSSD/sssd/issue/3685

Reviewed-by: Fabiano Fidêncio <fidencio@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/0294bcf7cdf21fe6702423f77554a880699c85bc">0294bcf7</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-08-13T16:39:56Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">TESTS: Properly set argv[0] when starting the secrets responder

This was a nasty one. Because argv[0] was unsert, instead of its typical
setting as the program path, parsing the arguments produced garbage
which resulted in uid and gid being set to a random (?) value. As an
effect, the secrets responder was not running as fake-root in tests and
this triggered permission errors in libsss_secrets and only root is able
to set arbitary secrets.

Related:
https://pagure.io/SSSD/sssd/issue/3685

Reviewed-by: Fabiano Fidêncio <fidencio@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/80811f941160e481112adc6933f59e3d2e5c5c85">80811f94</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-08-13T16:40:00Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">KCM: Move a confusing DEBUG message

We only want to print the message if KCM really does not find the nextID
on that try, but we used to print the message always.

Related:
https://pagure.io/SSSD/sssd/issue/3685

Reviewed-by: Fabiano Fidêncio <fidencio@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/ca73eedba23f4746d57161f185a5da5e1c5b6b7c">ca73eedb</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-08-13T16:40:05Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">KCM: Fix a typo

Related:
https://pagure.io/SSSD/sssd/issue/3685

Reviewed-by: Fabiano Fidêncio <fidencio@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/24b151e076313441518692a6afe7d3ec50e8c02f">24b151e0</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-08-13T16:40:10Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">UTIL: Add libsss_secrets

Splits pieces of the secrets responder and makes them into a library.
This is done so that other processes than sssd_secrets can access the
secrets database without going through the whole REST API.

Related:
https://pagure.io/SSSD/sssd/issue/3685

Reviewed-by: Fabiano Fidêncio <fidencio@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/fdfa36ae07dfc9bb7488d8dcde1e37ff95aeb3be">fdfa36ae</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-08-13T16:40:15Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">SECRETS: Use libsss_secrets

Instead of its own code for handling the secrets, use libsss_secrets.
It's pretty much the same code anyway, just moved to a different
location.

Related:
https://pagure.io/SSSD/sssd/issue/3685

Reviewed-by: Fabiano Fidêncio <fidencio@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/e0bf64a736dc0fa244dc8023970ed4e65b4d7f31">e0bf64a7</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-08-13T16:40:19Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">KCM; Hide the secret URL as implementation detail instead of exposing it in the JSON-marshalling API

There were several functions in the kcmsrv_ccache_json.c module that
were really specific to storing ccaches in the secrets responder. Since
we are about to introduce a new ccache back end which will on one hand
use the JSON representation but on the other hand not use the secrets
responder, we move several functions and definitions to
kcmsrv_ccache_secrets.c

A function to create the key for the secret is on the other made public
so that it could be reused in the future.

Related:
https://pagure.io/SSSD/sssd/issue/3685

Reviewed-by: Fabiano Fidêncio <fidencio@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/0b9001e3a8c820f137cbe07f853317474dac53ac">0b9001e3</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-08-13T16:40:24Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">UTIL: libsss_secrets: Add an update function

With the secrets responder, we were bound to what API did Custodia
provide, because one of the core functions of the secrets responder was
forwarding to Custodia. Since the library is not bound by any such
limitation, we can add a function to update a secret.

Related:
https://pagure.io/SSSD/sssd/issue/3685

Reviewed-by: Fabiano Fidêncio <fidencio@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/24ba2120697d95139ba6c0107608f31196532097">24ba2120</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-08-13T16:40:29Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">KCM: Add a new back end that uses libsss_secrets directly

Add a new KCM back end that stores the ccaches in libsss_secrets
directly by using its API. The back end is only fake-async as all the
libsss_secrets functions are synchronous.

Related:
https://pagure.io/SSSD/sssd/issue/3685

Reviewed-by: Fabiano Fidêncio <fidencio@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/f91adcc8eb58294d0a69b5ce938d959f059a8a84">f91adcc8</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-08-13T16:40:34Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">TESTS: Get rid of KCM_PEER_UID

In normal operation, libsss_secret only allows root to write to the KCM
namespace. This was impractical in tests, because the identity of the
client is read from the socket and even with the socket and uid
wrapping, still the real UID was used. So we #define-d the client ID to
something else during tests.

But this is also problematic for tests that do not use the secrets
responder, but only the libsecrets, beacause there the ID of the caller
is root, because of uid_wrapper.

This patch changes the approach so that the secrets responder tests
preload getsockopt_wrapper() and any communication to the secrets socket
rewrites the client credentials to be root. In combination with
uid_wrapper which pretends to libsss_secrets that the caller is root, we
can get rid of test-specific #define in the code.

Related:
https://pagure.io/SSSD/sssd/issue/3685

Reviewed-by: Fabiano Fidêncio <fidencio@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/7dd1991c93a8c0ac5c887573a1be18523a861522">7dd1991c</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-08-13T16:40:41Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">TESTS: Add tests for the KCM libsss_secrets back end

Related:
https://pagure.io/SSSD/sssd/issue/3685

Reviewed-by: Fabiano Fidêncio <fidencio@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/f74feb08b86e72eb3e098ed097d0574f815f6de3">f74feb08</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-08-13T16:40:47Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">KCM: Change the default ccache storage from the secrets responder to libsecrets

Related:
https://pagure.io/SSSD/sssd/issue/3685

Reviewed-by: Fabiano Fidêncio <fidencio@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/fcbedf46fcfc66f443afa3171036b4bc7bbd380b">fcbedf46</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-08-13T16:40:52Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">BUILD: Do not build the secrets responder by default

The secrets responder is now built only conditionally and defaults to
'do not build'. However, libsss_secrets.so is built whenever either KCM
or secrets are selected.

The KCM secrets responder tests are skipped if the secrets responder is
not built.

This patch also avoids two BuildRequires in the default set,
libcurl-devel and http-parser-devel are no longer required by SSSD.

Related:
https://pagure.io/SSSD/sssd/issue/3685

Reviewed-by: Fabiano Fidêncio <fidencio@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/c0374e1cb5793a7ad1e22eba32110c8657184965">c0374e1c</a></strong>
<div>
<span>by Niranjan M.R</span>
<i>at 2018-08-17T12:39:54Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">Python3 changes to multihost tests

1. Python3 related changes
2. converting ldap attributes to use bytes

Signed-off-by: Niranjan M.R <mrniranjan@redhat.com>
Reviewed-by: Madhuri Upadhye <mupadhye@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/aba6fe4471e3de66b3fac2661dd6255b1d185a10">aba6fe44</a></strong>
<div>
<span>by Niranjan M.R</span>
<i>at 2018-08-20T11:23:12Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">Minor fixes related to converting of ldap attributes to bytes

Minor fixes related to converting attribute values to bytes
in function related to adding posix group

Signed-off-by: Niranjan M.R <mrniranjan@redhat.com>
Reviewed-by: Madhuri Upadhye <mupadhye@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/101934f29e6b76931b1499adc19ae7f7a976789d">101934f2</a></strong>
<div>
<span>by Lukas Slebodnik</span>
<i>at 2018-08-20T11:24:33Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">BUILD: Fix issue with installation of libsss_secrets

libsss_secret.so is linked with libsss_util.so therefore it shoudl be
added into pkglib_LTLIBRARIES after libsss_util.so.
Otherwise there can failure in linking phase.

libtool: warning: relinking 'libsss_secrets.la'
libtool: install: (cd /home/build/sssd/ci-build-debug/intg/bld; /bin/sh
  "/home/build/sssd/ci-build-debug/intg/bld/libtool"  --tag CC
  --mode=relink gcc -Wall -Wshadow -Wstrict-prototypes -Wpointer-arith
  -Wcast-qual -Wcast-align -Wwrite-strings -Wundef
  -Werror-implicit-function-declaration -Winit-self -Wmissing-include-dirs
  -fno-strict-aliasing -std=gnu99 -O2 -g -g3 -O2 -Werror
  -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE
  -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE
  -avoid-version -o libsss_secrets.la -rpath
  /tmp/sssd-intg.l7nl5pgb/lib/sssd
  src/util/secrets/libsss_secrets_la-secrets.lo
  src/util/secrets/libsss_secrets_la-config.lo -ltalloc -lldb
  libsss_crypt.la libsss_debug.la libsss_util.la )
libtool: relink: gcc -shared  -fPIC -DPIC
  src/util/secrets/.libs/libsss_secrets_la-secrets.o
  src/util/secrets/.libs/libsss_secrets_la-config.o   -Wl,-rpath
  -Wl,/tmp/sssd-intg.l7nl5pgb/lib/sssd -ltalloc -lldb
  -L/tmp/sssd-intg.l7nl5pgb/lib/sssd -lsss_crypt -lsss_debug -lsss_util
  -O2 -g -g3 -O2   -Wl,-soname -Wl,libsss_secrets.so -o
  .libs/libsss_secrets.so
/usr/bin/ld: cannot find -lsss_util
collect2: error: ld returned 1 exit status
libtool:   error: error: relink 'libsss_secrets.la' with the above
command before installing it

Merges: https://pagure.io/SSSD/sssd/pull-request/3808

Reviewed-by: Pavel Březina <pbrezina@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/677a93372e4b7359d19d7e55467fa5ccea4a80a3">677a9337</a></strong>
<div>
<span>by Lukas Slebodnik</span>
<i>at 2018-08-20T11:24:52Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">BUILD: Add missing deps to libsss_sbus*.so

It indirectly caused failures when linking unit test.

  CCLD     test_sbus_opath
/usr/bin/ld: ./.libs/libsss_sbus.so: undefined reference to `g_utf8_validate'
/usr/bin/ld: ./.libs/libsss_sbus.so: undefined reference to `g_free'
/usr/bin/ld: ./.libs/libsss_sbus.so: undefined reference to `g_malloc'
/usr/bin/ld: ./.libs/libsss_sbus.so: undefined reference to `g_utf8_casefold'
/usr/bin/ld: ./.libs/libsss_sbus.so: undefined reference to `g_utf8_collate'
/usr/bin/ld: ./.libs/libsss_sbus.so: undefined reference to `g_utf8_strlen'
/usr/bin/ld: ./.libs/libsss_sbus.so: undefined reference to `g_utf8_strdown'

Merges: https://pagure.io/SSSD/sssd/pull-request/3808

Reviewed-by: Pavel Březina <pbrezina@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/53ed60b878d3737d4c174644b69df960595479da">53ed60b8</a></strong>
<div>
<span>by Lukas Slebodnik</span>
<i>at 2018-08-20T11:24:58Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">BUILD: Reduce compilation of unnecessary files

We safe compilation of 52 files 2 times with linking to existing
internal libraries.

It also fixes issue with multiple definition of the same symbol

  CCLD     responder_common-tests
/usr/bin/ld: .libs/libsss_debug.so and ../../../src/util/responder_common_tests-debug.o:
  warning: multiple common of `sss_logger'
collect2: error: ld returned 1 exit status

Merges: https://pagure.io/SSSD/sssd/pull-request/3808

Reviewed-by: Pavel Březina <pbrezina@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/7dce2461e5c290453b3a8c5b0ff1b4b0da8c6188">7dce2461</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-08-21T12:17:18Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">Updating the version to track 2.1 development

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/81dce19792cf300950411722d16b72f8816aecb0">81dce197</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-08-29T13:04:48Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">KCM: Don't error out if creating a new ID as the first step

We need to handle the case where the nextID operation is ran, but the
secdb is totally empty, otherwise logins with sssd's krb5_child would
fail.

Resolves:
https://pagure.io/SSSD/sssd/issue/3815

Reviewed-by: Michal Židek <mzidek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/55d5b43543b5ef62322fe635fe8108410cb4ea77">55d5b435</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-09-06T14:38:25Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sbus: register filter on new connection

The filter is not again registered on new connection when the old connection
was lost. This caused a segfault when the router is destroyed during shutdown.

It also would not allow to recieve and process any messages as the filter
function is needed for that. However, this was not very visible with
current sssd architecture.

Steps to reproduce:
1. Run SSSD
2. pkill sssd_be
3. Wait for responders to reconnect to backend
4. Shutdown SSSD
5. It will crash without this patch

Resolves:
https://pagure.io/SSSD/sssd/issue/3821

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/8c8f74b0dfa29643279d31b12300ced47d5c2ab5">8c8f74b0</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-09-06T14:38:38Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sbus: fix typo

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/30f4adf874aff174734ad77902a79fc5727ab495">30f4adf8</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-09-06T14:38:42Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sbus: check for null message in sbus_message_bound

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/ca50c40511f08c0f7c786598e5793a06789c6cce">ca50c405</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-09-06T14:38:45Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sbus: replace sbus_message_bound_ref with sbus_message_bound_steal

The memory context used to new message reference accidentally overwrote
the one use by the initial sbus_message_bound call. This caused a memory
leak of message as its reference counter got increased but number of
talloc contexts bound this this message decreased at the same time.

Fixing this is non-trival and it would require separate data slot for
each reference. Because we do not have any existing use case for this
and we use it only as an equivalent of talloc_steal it is better to
provide a real equivalent for this talloc function.

Resolves:
https://pagure.io/SSSD/sssd/issue/3810

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/c895fa2449900f4abd1dce6bb62a45c52bbb12cf">c895fa24</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-09-06T14:38:54Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sbus: add unit tests for public sbus_message module

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/945865ae16120ffade267227ca48cefd58822fd2">945865ae</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-09-11T19:46:14Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">SELINUX: Always add SELinux user to the semanage database if it doesn't exist

Previously, we tried to optimize too much and only set the SELinux user
to Linux user mapping in case the SELinux user was different from the
system default. But this doesn't work for the case where the Linux user
has a non-standard home directory, because then SELinux would not have
any idea that this user's home directory should be labeled as a home
directory.

This patch relaxes the optimization in the sense that on the first
login, the SELinux context is saved regardless of whether it is the same
as the default or different.

Resolves:
https://pagure.io/SSSD/sssd/issue/3819

Reviewed-by: Michal Židek <mzidek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/e573f57799d1ac5230732290c8689f5ca36f38f6">e573f577</a></strong>
<div>
<span>by Niranjan M.R</span>
<i>at 2018-09-11T20:10:44Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">test-library: fixes related to KCM, TLS on Directory server

Fixes related to enabling of ssl in Directory Server
Minor fixes on KCM, adding new exceptions

Signed-off-by: Niranjan M.R <mrniranjan@redhat.com>

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/ac622b771c4879d6fefbca1cadcaa9f5d1788852">ac622b77</a></strong>
<div>
<span>by Niranjan M.R</span>
<i>at 2018-09-11T20:10:47Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">Multihost-SanityTests: New test case for ssh login with KCM as default

1. Minor fix with regarding to spaces between package names
2. Install sssd-dbus

Signed-off-by: Niranjan M.R <mrniranjan@redhat.com>

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/1e2398870e8aa512ead3012d46cbe6252429467a">1e239887</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-09-19T09:49:29Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">intg: flush the SSSD caches to sync with files

To make sure that SSSD has synced with the latest data added to the
passwd file sss_cache is called in two places where the current sync
scheme was not reliable. This was mainly observed when running the
integration tests on Debian.

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/b03179ead11db7dbfd6a00d3eeef3dac0990f826">b03179ea</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-09-19T09:51:01Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sbus: dectect python binary for sbus_generate.sh

We already detect python2 and python3 binaries during configure. With
this patch PYTHON_EXEC is set to the python3 binary if python3 bindings
are generated and to the python2 binary otherwise. With the help of an
environment variable sbus_generate.sh is made aware of it.

Related to https://pagure.io/SSSD/sssd/issue/3807

Reviewed-by: Pavel Březina <pbrezina@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/d7f0b58e2896ed2ef9ed5a390815c1e4df6caaee">d7f0b58e</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-09-19T09:54:14Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sudo: respect case sensitivity in sudo responder

If the domain is not case sensitive and the case of the original user
or group name differs from the name in the rule we failed to find the
rule.

Now we filter the rule only with lower cased values in such domain.

Steps to reproduce:
1. Add user/group with upper case, e.g. USER-1
2. Add sudo rule with lower cased name, e.g. sudoUser: user-1
3. Login to system with lower case, e.g. user-1
4. Run sudo -l

Without the patch, rule is not found.

Resolves:
https://pagure.io/SSSD/sssd/issue/3820

Reviewed-by: Michal Židek <mzidek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/3bd67c772c951f33422261ef658a104ccecc9561">3bd67c77</a></strong>
<div>
<span>by Michal Židek</span>
<i>at 2018-09-19T09:55:58Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">GPO: Add gpo_implicit_deny option

This option (when set to True) can be used to deny access to
users even if there is not applicable GPO. Normally users are
allowed access in this situation.

Resolves:
https://pagure.io/SSSD/sssd/issue/3701

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/10fa27eddb9bbe135277d587c6a2de4b311da6df">10fa27ed</a></strong>
<div>
<span>by Michal Židek</span>
<i>at 2018-09-21T16:22:20Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">CONFDB: Skip 'local' domain if not supported

When SSSD is built without the support for local
domain, we should gracegully skip local domains
and let other domains start.

Resolves:
https://pagure.io/SSSD/sssd/issue/3828

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/7c619ae08f05a7595d15cf11b64461a7d19cfaa7">7c619ae0</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-09-25T12:56:48Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sysdb: extract sysdb_ldb_msg_attr_to_certmap_info() call

Related to https://pagure.io/SSSD/sssd/issue/3500

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/d1dd7f7703b4f40d2fbb830e28969b31b8a1673e">d1dd7f77</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-09-25T12:56:52Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sysdb_ldb_msg_attr_to_certmap_info: set SSS_CERTMAP_MIN_PRIO

Make sure that priority is always set.

Related to https://pagure.io/SSSD/sssd/issue/3500

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/0bf709ad348ca115443bd21e4e369abd5d7698c4">0bf709ad</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-09-25T12:57:12Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sysdb: add attr_map attribute to sysdb_ldb_msg_attr_to_certmap_info()

Allow more flexible attribute mapping in
sysdb_ldb_msg_attr_to_certmap_info()

Related to https://pagure.io/SSSD/sssd/issue/3500

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/d9cc38008a51a8a5189904f175e4d10cbde4a974">d9cc3800</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-09-25T12:57:20Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">confdb: add confdb_certmap_to_sysdb()

Add a function to write certificate mapping and matching rules from the
config database to the cache of a domain.

Related to https://pagure.io/SSSD/sssd/issue/3500

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/15301db1dc1e5e2aafc1805a30e3b28756218c9b">15301db1</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-09-25T12:57:24Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">AD/LDAP: read certificate mapping rules from config file

Related to https://pagure.io/SSSD/sssd/issue/3500

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/06f7005d38d164879b727708feff80004b422f91">06f7005d</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-09-25T12:57:27Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sysdb: sysdb_certmap_add() handle domains more flexible

sysdb_ldb_msg_attr_to_certmap_info() creates an empty list if there are
no domains defined, sysdb_certmap_add() should be able to handle both a
missing or an empty domains list.

Related to https://pagure.io/SSSD/sssd/issue/3500

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/9386ef605ffbc03abe2bc273efddbc099441fe3b">9386ef60</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-09-25T12:57:31Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">confdb: add special handling for rules for the files provider

To make the configuration more simple there are some special assumption
for local users, i.e. user managed by the files provider.

Related to https://pagure.io/SSSD/sssd/issue/3500

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/275eeed24adc31f3df51cf278f509a4be76a3a3c">275eeed2</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-09-25T12:57:36Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">files: add support for Smartcard authentication

To support certificate based authentication the files provider must be
able to map a certificate to a user during a BE_REQ_BY_CERT request.

Additionally the authentication request should be handled by the PAM
responder code which is responsible for the local Smartcard
authentication. To be consistent with the other backend an authentication
handler is added to the files provider which unconditionally returns the
offline error code telling the PAM responder to handle the
authentication if it has access to the needed credentials.

Related to https://pagure.io/SSSD/sssd/issue/3500

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/9fdc5f1d87a133885e6a22810a7eb980c60dcb55">9fdc5f1d</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-09-25T12:57:40Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">responder: make sure SSS_DP_CERT is passed to files provider

Currently the files provider is only contacted once in a while to update
the full cache with fresh data from the passwd file. To allow rule based
certificate mapping the lookup by certificate request must be always
send to the file provider so that it can evaluate the rules and add the
certificate to cached entry of the matching user.

Related to https://pagure.io/SSSD/sssd/issue/3500

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/d42f44d54453d3ddb54875374c1b61dc1e7cd821">d42f44d5</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-09-25T12:57:45Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">PAM: add certificate matching rules from all domains

Currently the PAM responder only reads the certificate mapping and
matching rules from the first domain. To support Smartcard
authentication for local and remote users all configured domains must be
taken into account.

Related to https://pagure.io/SSSD/sssd/issue/3500

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/0c739e969a617bdb4c06cdfd63772bf6d283c518">0c739e96</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-09-25T12:57:50Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">doc: add certificate mapping section to man page

Related to https://pagure.io/SSSD/sssd/issue/3500

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/16941c47a6f0fc2f1679725d55cde221f3c3a6ef">16941c47</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-09-25T12:57:53Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">intg: user default locale

Some checks depend on english error messages so checks should be always
run with the default locale.

Related to https://pagure.io/SSSD/sssd/issue/3500

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/442ae7b1d0704cdd667d4f1ba4c165ce3f3ffed4">442ae7b1</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-09-25T12:57:57Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">PAM: use better PAM error code for failed Smartcard authentication

If the user enters a wrong PIN the PAM responder currently returns
PAM_USER_UNKNOWN better is PAM_AUTH_ERR.

Related to https://pagure.io/SSSD/sssd/issue/3500

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/91aea762d02731193eb66a00b930ff1fe8bc5ab8">91aea762</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-09-25T12:58:01Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">test_ca: test library only for readable

On Debian libraries typically do not have the execute-bit set so it is
better to only check for readability.

Related to https://pagure.io/SSSD/sssd/issue/3500

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/a45a410dc7fa7cf84bcac541e693ee8781e25431">a45a410d</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-09-25T12:58:05Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">test_ca: set a password/PIN to nss databases

To make sure the PIN is properly checked during tests the NSS databases
need a password.

Related to https://pagure.io/SSSD/sssd/issue/3500

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/d332c8a0e7a4c7f0b3ee1b2110145a23cbd61c2a">d332c8a0</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-09-25T12:58:09Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">getsockopt_wrapper: add support for PAM clients

PAM clients expect that the private socket of the PAM responder is
handled by root. With this patch getsockopt_wrapper can return the
expected UID and GID to PAM clients.

Related to https://pagure.io/SSSD/sssd/issue/3500

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/657f3b89bca9adfb13f0867c91f1d76845d2d6dd">657f3b89</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-09-25T12:58:15Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">intg: add Smartcard authentication tests

Two test for Smartcard authentication of a local user, i.e. a user
managed by the files provider, are added. One for a successful
authentication, the other for a failed authentication with a wrong PIN.

Related to https://pagure.io/SSSD/sssd/issue/3500

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/74f24e9b34cc063c2d70dca3a3d0f35bd4e9778f">74f24e9b</a></strong>
<div>
<span>by Niranjan M.R</span>
<i>at 2018-09-26T11:53:21Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">pytest: Remove installing idm module

Signed-off-by: Niranjan M.R <mrniranjan@redhat.com>

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/4ffe3ab9023ff858410256bc5c38a03d9cd88cf9">4ffe3ab9</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-09-28T08:20:25Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">proxy: access provider directly not through be_ctx

Modules are initialized as part of dp_init_send() but be_ctx->provider is set
only after this request is finished therefore it is not available here.

Resolves:
https://pagure.io/SSSD/sssd/issue/3812

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/4c5a1afa0df41aac05d34455c6e54a6f52a8dd28">4c5a1afa</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-09-28T08:20:29Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">dp: set be_ctx->provider as part of dp_init request

Backend context is overused inside sssd code even during its initialization.
Some parts of initialization code requires access to be_ctx->provider so we
must make it available as soon as possible.

Better solution would be to always use 'provider' directly in initialization
but this makes it safer for any future changes as one does not have to keep
in mind when it is safe to use be_ctx->provider and when not. Now it is
always safe.

Resolves:
https://pagure.io/SSSD/sssd/issue/3812

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/9245bf1afe6767a0412212bc0040e606ee850e7d">9245bf1a</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-09-28T08:20:33Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sbus: read destination after sender is set

dbus_message_set_sender may reallocate internal fields which will yield pointer
obtained by dbus_message_get_* invalid.

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/b821ee3ca93beb94a7a9b22b6f7a205e4900212e">b821ee3c</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-09-28T08:20:36Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sbus: do not try to remove signal listeners when disconnecting

This may cause some troubles if the dbus connection was dropped
as dbus will try to actually send the messages. Also when the
connectin is being freed, tevent integration is already disabled
so there is no point in doing this.

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/f1f9af528f71f42ac41bb7a272f4f7d940fd3a0f">f1f9af52</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-09-28T08:20:42Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sbus: free watch_fd->fdevent explicitly

We never reproduced this with gdb but valgrind shows invalid read in sbus_watch_handler
after the watch_fd was freed. This should not be needed since watch_fd is memory parent
of fdevent but it seems to help.

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/de8c9caf61e7b971cda9563cc5851ea222db5830">de8c9caf</a></strong>
<div>
<span>by Tomas Halman</span>
<i>at 2018-10-01T19:12:07Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">doc: remove local provider reference from manpages

Introduce new condition for documentation build. Related part of
documentation is excluded, if build is done without local provider.

Resolves https://pagure.io/SSSD/sssd/issue/3826

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/4276b3f73d26f9fb7fdd39e1443df5301f96a099">4276b3f7</a></strong>
<div>
<span>by Niranjan M.R</span>
<i>at 2018-10-01T19:27:55Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">pytest/testlib: Add function to create organizational Unit

Signed-off-by: Niranjan M.R <mrniranjan@redhat.com>

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/a62caa0e396eac08d4d225142a63003b2b2ba061">a62caa0e</a></strong>
<div>
<span>by Niranjan M.R</span>
<i>at 2018-10-01T19:27:58Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">pytest/testlib: Fix related to removing kerberos database

Stop krb5kdc and kadmin services. Earlier we were
not stopping kadmin services. The current code use
loop to stop the services

Based on review comments, used tuple to define krb5kdc and kadmin

Signed-off-by: Niranjan M.R <mrniranjan@redhat.com>

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/2ac3efd1173af62c3bb4b8fb3a823330529bbc22">2ac3efd1</a></strong>
<div>
<span>by Niranjan M.R</span>
<i>at 2018-10-01T19:28:02Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">pytest: Add test for sudo: search with lower cased name for case insensitive domains

1. Add test case for sudo: search with lower
   cased name for case insensitive domains

2. Add fixtures:
    disable case_sensitive
    enable sudo service
    add suders: sss in nsswitch.conf

Signed-off-by: Niranjan M.R <mrniranjan@redhat.com>

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/58d11ae61dfc5d5fb95ee38de10f3637bc8edd53">58d11ae6</a></strong>
<div>
<span>by Niranjan M.R</span>
<i>at 2018-10-01T19:28:12Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">pytest/testlib: function to create sudorules in ldap

Adds function add_sudo_rule to create sudo rules in
Directory Server.

Signed-off-by: Niranjan M.R <mrniranjan@redhat.com>

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/78333065761450873a1855aa41e28761299851e2">78333065</a></strong>
<div>
<span>by Niranjan M.R</span>
<i>at 2018-10-01T19:28:15Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">pytest/testlib: remove space in CA DN

Minor fix of removing space in DN when creating self signed CA.

Signed-off-by: Niranjan M.R <mrniranjan@redhat.com>

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/56842e706c26cd4cce6a3fbd2e175e844f75887f">56842e70</a></strong>
<div>
<span>by Niranjan M.R</span>
<i>at 2018-10-01T19:28:18Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">pytest/conftest.py: Delete krb5.keytab as part of cleanup

Signed-off-by: Niranjan M.R <mrniranjan@redhat.com>

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/0e8f9ffe9d34ff664304a8b5f18827b175285eae">0e8f9ffe</a></strong>
<div>
<span>by Niranjan M.R</span>
<i>at 2018-10-01T19:28:20Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">pytest: split kcm test cases in to separate file.

Signed-off-by: Niranjan M.R <mrniranjan@redhat.com>

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/e18c67c38a3f645549ebd5117b49a4d3e1c94018">e18c67c3</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-10-03T09:24:18Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">ci: add http-parser-devel for Fedora

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/ec7665973936897ab6be58308e655f08d91bec5c">ec766597</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-10-03T09:27:15Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">pep8: Ignore W504 and W605 to silence warnings on Debian

This code:
    pkcs11_txt.write("library=libsoftokn3.so\nname=soft\n" +
                     "parameters=configdir='sql:" + config.ABS_BUILDDIR +
                     "/../test_CA/p11_nssdb' " +
                     "dbSlotDescription='SSSD Test Slot' " +
                     "dbTokenDescription='SSSD Test Token' " +
                     "secmod='secmod.db' flags=readOnly)\n\n")
    pkcs11_txt.close()

Was producing warnings such as:
./src/tests/intg/test_pam_responder.py:143:22: W504 line break after binary operator

Even though it looks OK visually and conforms to pep8's written form.

Additionaly, this regular expression compilation:
 Template = re.compile(
            ' *<template name="(\S+)">(.*?)</template>\r?\n?',
            re.MULTILINE | re.DOTALL
        )

Was producing a warning such as:
./src/sbus/codegen/sbus_Template.py:156:29: W605 invalid escape sequence '\S'

Since the \S literal is part of a regular expression, let's suppress
this warning as well.

Reviewed-by: Michal Židek <mzidek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/081b18e75c746f9a2ad1fb412c825293090311f8">081b18e7</a></strong>
<div>
<span>by Tomas Halman</span>
<i>at 2018-10-03T09:29:43Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">confdb: log an error when domain is misconfigured

We need to inform user that there is misconfiguration
and particular domain will not be started.

Resolves:
https://pagure.io/SSSD/sssd/issue/3827

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/dfa7bf1133f002a9fbbd3495a70909913db25b16">dfa7bf11</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-10-03T09:33:05Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">be: use be_is_offline for the main domain when asking for domain status

The DOM_ACTIVE/INACTIVE flag is not used with the main domain as it
is used only for subdomains.

Resolves:
https://pagure.io/SSSD/sssd/issue/3830

Reviewed-by: Michal Židek <mzidek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/e29b82077a78157a1e4d90e2308c1272d7612f3d">e29b8207</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-10-08T18:56:22Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">p11: handle multiple certs during auth with OpenSSL

This patch adds missing code already available in the NSS version to
select a certificate for authentication if multiple certificates are
available on the Smartcard. A unit test to check this feature is added
as well.

Related to https://pagure.io/SSSD/sssd/issue/3489

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/0be037bbedd0aed6a7eccead6aabe0d07258242a">0be037bb</a></strong>
<div>
<span>by Tomas Halman</span>
<i>at 2018-10-08T18:56:27Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">doc: Add nsswitch.conf note to manpage

We want to add note about nsswitch.conf configuration
into sssd-files manpage.

Resolves:
https://pagure.io/SSSD/sssd/issue/3750

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
Reviewed-by: Justin Stephenson <jstephen@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/e5dc30e0092b240a32f2004966eeecdc57d50fb8">e5dc30e0</a></strong>
<div>
<span>by Lukas Slebodnik</span>
<i>at 2018-10-09T21:11:25Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">MAN: Fix typo in ad_gpo_implicit_deny default value

Merges: https://pagure.io/SSSD/sssd/pull-request/3846

Reviewed-by: Michal Židek <mzidek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/42f69e26e5b858dd03492cc2a148d02c2ccc2161">42f69e26</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-10-10T21:02:36Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">p11_child: add --wait_for_card option

The --wait_for_card option will let the p11_child wait until a
Smartcard/token is available in a slot with the removable flag.

Related to  https://pagure.io/SSSD/sssd/issue/3650

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/2e4ecf5a866b212bef44e262fd90c67a88dc616a">2e4ecf5a</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-10-10T21:02:46Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">PAM: add p11_wait_for_card_timeout option

If the --wait_for_card is used to call p11_child the PAM responder
should be prepared to wait longer until p11_child can return
successfully.

Related to https://pagure.io/SSSD/sssd/issue/3650

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/d33a8bed5aad9135426c9ebdf101cf600685ab81">d33a8bed</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-10-10T21:02:51Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">pam_sss: make flags public

To allow the PAM responder to act on the config flags set for pam_sss
the flags have to be made public first.

Related to https://pagure.io/SSSD/sssd/issue/3650

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/d3a18f06162b9585d2db936472b75fdbff37162d">d3a18f06</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-10-10T21:02:54Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">pam_sss: add try_cert_auth option

With this new option pam_sss can be configured to only do Smartcard
authentication or return an error if this is not possible.

Related to https://pagure.io/SSSD/sssd/issue/3650

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/49be8974b490c368d349752f3196af0c9ed28dd5">49be8974</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-10-10T21:02:57Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">pam_sss: add option require_cert_auth

With this new option pam_sss will wait until a Smartcard is available
and then try to authenticate with the help of the Smartcard.

Related https://pagure.io/SSSD/sssd/issue/3650

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/5cdb6968f407c7bcaba69f4892f51fd6426dddb2">5cdb6968</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-10-10T21:03:01Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">intg: require SC tests

Integration test for the new try_cert_auth and require_cert_auth option
for pam_sss.

Related to https://pagure.io/SSSD/sssd/issue/3650

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/46fd681a73ffef062cd027e7018e1a02d7a0a9df">46fd681a</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-10-10T21:06:16Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">p11_child: show PKCS#11 URI in debug output

Related to https://pagure.io/SSSD/sssd/issue/3814

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/f7b2152a4c3c816a5bc4226a0e01791313accef3">f7b2152a</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-10-10T21:06:21Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">p11_child: add PKCS#11 uri to restrict selection

Related to https://pagure.io/SSSD/sssd/issue/3814

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/725b65081d19da658b16338686c53dcf16d49de0">725b6508</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-10-10T21:06:26Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">PAM: add p11_uri option

Related to https://pagure.io/SSSD/sssd/issue/3814

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/4a22fb6bba6662ad628f6e17203e8ccf20eb9666">4a22fb6b</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-10-10T21:06:31Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">tests: add PKCS#11 URI tests

Related to https://pagure.io/SSSD/sssd/issue/3814

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/7a2e56d061085c155a51253bd612255a4d24cb57">7a2e56d0</a></strong>
<div>
<span>by Tomas Halman</span>
<i>at 2018-10-10T21:11:50Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">test_config: Test for invalid characker in domain

There was bug allowing forbidden characters in config file section name.
Bug has been fixed meantime but we decided to write the test to avoid
regeression.

Resolves:
https://pagure.io/SSSD/sssd/issue/3334

Reviewed-by: Michal Židek <mzidek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/dbd717fe5b7d8dd640b6ade435b49edb3db5280a">dbd717fe</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-10-10T21:14:01Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">PAM: return short name for files provider users

If the 'allow_missing_name' option is used with pam_sss and the user
name will be determined based on the certificate content and the mapping
rules the PAM responder will by default return the fully-qualified name
of the user which is then later used by other PAM modules as well.

For local users which are configured to use SSSD for Smartcard
authentication this might cause issues in other PAM modules because they
are not aware of the fully-qualified name and will treat the user as
unknown.

With this patch the PAM responder will return the short name for all
users handled by the files provider.

Related to https://pagure.io/SSSD/sssd/issue/3848

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/941e67b0bbb780aadb6461b60b4e3554dfb893db">941e67b0</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-10-15T19:56:58Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">TESTS: Add a test for whitespace trimming in netgroup entries

This is a unit test for commit dbb1abae6eaa9df24f61e3a9f855e2461a66a197

Reviewed-by: Tomas Halman <thalman@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/1e67da79ac23279752a1361a12afc2766cb3d4ad">1e67da79</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-10-15T19:58:48Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">TESTS: Add two basic multihost tests for the files provider

Reviewed-by: Michal Židek <mzidek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/7b3794fbe5e4f0888d4faeba12e6c5268f8cca42">7b3794fb</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-10-15T19:58:54Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">FILES: The files provider should not enumerate

Resolves:
https://pagure.io/SSSD/sssd/issue/3849

For reason I cannot explain now, the files provider always enumerates.
There is commit a60e6ec which implements this, but it's clearly wrong,
because then the plain getent passwd output contains duplicates from
nss_files and nss_sss:

$ getent passwd | sort
adm:x:3:4:adm:/var/adm:/sbin/nologin
adm:x:3:4:adm:/var/adm:/sbin/nologin
bin:x:1:1:bin:/bin:/sbin/nologin
bin:x:1:1:bin:/bin:/sbin/nologin
certuser:x:10329:10330::/home/certuser:/bin/bash
certuser:x:10329:10330::/home/certuser:/bin/bash
chrony:x:997:994::/var/lib/chrony:/sbin/nologin
chrony:x:997:994::/var/lib/chrony:/sbin/nologin
daemon:x:2:2:daemon:/sbin:/sbin/nologin
daemon:x:2:2:daemon:/sbin:/sbin/nologin

Reviewed-by: Michal Židek <mzidek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/91c608d0eb48435b5b5d2f3631a4bb2a40b8d519">91c608d0</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-10-15T20:02:55Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">p11_child: add OCSP check ot the OpenSSL version

Related to https://pagure.io/SSSD/sssd/issue/3489

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/3c096c9ad6dad911d035cfdd802b5dda4710fc68">3c096c9a</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-10-15T20:02:58Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">p11_child: add crl_file option for the OpenSSL build

In the NSS build a Certificate Revocation List (CRL) can just be added
to the NSS database. For OpenSSL a separate file is needed.

Related to https://pagure.io/SSSD/sssd/issue/3489

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/7794caec36e7142423491d90aaade7e49b9df1c1">7794caec</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-10-15T20:05:00Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">p11: Fix two instances of -Wmaybe-uninitialized in p11_child_openssl.c

If uri_str was passed to the p11_child and parsing the URI failed, then
modules would be uninitialized, but freed in the done handler with
p11_kit_modules_finalize_and_release()

Also, another warning is suppressed by setting the 's' variable to zero.
While it cannot happen that the variable will be uninitialized, we
should help the compiler by setting a value explicitly.

Reviewed-by: Sumit Bose <sbose@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/250e82252b53991e2902b292cfa6029ab28a10fb">250e8225</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-10-15T20:06:27Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sudo: use correct sbus interface

Internal dbus interfaces were renamed to shorter names in sbus2.

Resolves:
https://pagure.io/SSSD/sssd/issue/3854

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/8fbaf224193b9ca8b82a290bd52265c2f9b40315">8fbaf224</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-10-15T20:06:30Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sudo: fix error handling in sudosrv_refresh_rules_done

If sbus returns non-zero code then the output variables are not set and
therefore we access uninitialized memory.

Resolves:
https://pagure.io/SSSD/sssd/issue/3854

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/c74b430ba95d99b245b6347328024e4b4815b35e">c74b430b</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-10-15T20:06:33Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sbus: remove leftovers from previous implementation

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/05ba237af582c1ca3780e5fe06ab3320494efe52">05ba237a</a></strong>
<div>
<span>by Lukas Slebodnik</span>
<i>at 2018-10-15T20:11:20Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">CONFIGURE: Add minimal required version for p11-kit

There are few functions which were added in upstream p11-kit 0.23.3.
And there are compilation failures with older versions.

src/p11_child/p11_child_openssl.c: In function 'get_pkcs11_uri':
src/p11_child/p11_child_openssl.c:87:12: error: implicit declaration of function 'p11_kit_uri_get_slot_info' [-Werror=implicit-function-declaration]
     memcpy(p11_kit_uri_get_slot_info(uri), slot_info, sizeof(CK_SLOT_INFO));
            ^

src/p11_child/p11_child_openssl.c:88:5: error: implicit declaration of function 'p11_kit_uri_set_slot_id' [-Werror=implicit-function-declaration]
     p11_kit_uri_set_slot_id(uri, slot_id);
     ^

src/p11_child/p11_child_openssl.c: In function 'do_card':
src/p11_child/p11_child_openssl.c:767:35: error: implicit declaration of function 'p11_kit_uri_get_slot_id' [-Werror=implicit-function-declaration]
                     uri_slot_id = p11_kit_uri_get_slot_id(uri);
                                   ^
src/p11_child/p11_child_openssl.c:770:32: error: implicit declaration of function 'p11_kit_uri_match_slot_info' [-Werror=implicit-function-declaration]
                             || p11_kit_uri_match_slot_info(uri, &info) != 1) {
                                ^
Merges: https://pagure.io/SSSD/sssd/pull-request/3852

Reviewed-by: Sumit Bose <sbose@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/d143319bce8fc778df93fe7cd7ef4d03b7a3fc92">d143319b</a></strong>
<div>
<span>by Lukas Slebodnik</span>
<i>at 2018-10-15T20:13:08Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">SBUS: Silence warning maybe-uninitialized

It should not happen because function sbus_interface_find_property
should return NULL for access different than SBUS_PROPERTY_READABLE
or SBUS_PROPERTY_WRITABLE. And thus we would return ERR_SBUS_UNKNOWN_PROPERTY
from the function sbus_request_property.

src/sbus/interface/sbus_properties.c: In function 'sbus_request_property.isra.0':
src/sbus/interface/sbus_properties.c:360:14:
  error: 'type' may be used uninitialized in this function
  [-Werror=maybe-uninitialized]
     sbus_req = sbus_request_create(mem_ctx, conn, type, destination,
     ~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
                                    interface_name, property_name, path);
                                    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors

Merges: https://pagure.io/SSSD/sssd/pull-request/3851

Reviewed-by: Pavel Březina <pbrezina@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/46c483c09b85cecf8d1cc72618da993d8948c894">46c483c0</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-10-16T11:26:04Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">files: add session recording flag

If session recording is configured for a group the NSS ans PAM
responder rely on a attribute in the cache set by the backend to
determine is session recording is configured for the user or not. This
flag is typically set during the initgroups request.

Since the files provider does not have a dedicated initgroups request
the attribute must be set otherwise. This patch sets is for all users
after the files are reloaded.

Related to https://pagure.io/SSSD/sssd/issue/3855

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/fc29c3eb9750c5e7def4e1ab6eb18f4f5024f567">fc29c3eb</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-10-16T11:26:08Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">UTIL: Suppress Coverity warning

We recently added this code:
         if (domain_name != NULL
                &&  is_files_provider(find_domain_by_name(dom,
                                                          domain_name,
                                                          false)))

find_domain_by_name returns NULL if the domain_name can't be found. This
of course makes mostly sense for trusted domains that can appear and
disappear. And is_files_provider() didn't handle the situation where the
domain pointer was NULL and would directly dereference it.

This commit just adds a NULL check for the domain pointer so that
is_files_provider() returns 'false' if the domain pointer was NULL.

Another alternative might be to check the return value of
find_domain_by_name(), but I don't think it's worth the trouble.

Reviewed-by: Sumit Bose <sbose@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/f62f3b290cb5393bf9c4a9c2c424fd5c5185fe44">f62f3b29</a></strong>
<div>
<span>by Tomas Halman</span>
<i>at 2018-10-23T20:48:42Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">UTIL: move and rename sysdb_error_to_errno to utils

The function sysdb_error_to_errno() is used by multiple component,
so we want to move it to shared place. Function is renamed to
sss_ldb_error_to_errno() and moved to util/util_errors.c.

Old name is still #defined in sysdb.h and points to new
name.

Also few places where ldb error code was handled manually
has been updated to use sss_ldb_error_to_errno.

Resolves:
https://pagure.io/SSSD/sssd/issue/3802

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
Reviewed-by: Michal Židek <mzidek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/a0280715c360f85d7873cacb73a7b05ebefa3b2c">a0280715</a></strong>
<div>
<span>by Niranjan M.R</span>
<i>at 2018-10-23T20:52:09Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">testlib: Update update_resolv_conf() to decode str to bytes

Signed-off-by: Niranjan M.R <mrniranjan@redhat.com>

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/ac04d19fd6495c21e591564a6284fda96d672d36">ac04d19f</a></strong>
<div>
<span>by Niranjan M.R</span>
<i>at 2018-10-23T20:52:14Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">testlib: Replace Generic Exception with SSSDException and LdapException

Signed-off-by: Niranjan M.R <mrniranjan@redhat.com>

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/dd2e6f26e6e30f004bd51a943de2e0c93b1c6dbc">dd2e6f26</a></strong>
<div>
<span>by Madhuri Upadhye</span>
<i>at 2018-10-23T20:56:18Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">pytest: Add test cases for configuration validation

Test cases cover configuration validation for sssd.conf,
"config-check" option of the "sssctl" command to locate
problems in the configuration file.

Signed-off-by: Madhuri Upadhye <mupadhye@redhat.com>

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/f0603645f5ea5f707875807b4f815400f4b79e41">f0603645</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-10-24T09:56:48Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">PYSSS: Re-add the pysss.getgrouplist() interface

Related:
https://pagure.io/SSSD/sssd/issue/3493

Commit 0e211b8ba30c3adcdeef21ca1339b194cbfffb04 was supposed to remove
only the parts of the pysss API that relate to the local domain. But it
removed also the getgrouplist() method by accident. This method is very
important to IPA, so we need to add it back.

Reviewed-by: Alexander Bokovoy <abokovoy@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/b4063b2d3f1c700aa074f71d5db501e1bdfd6d2a">b4063b2d</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-10-25T09:57:13Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">ifp: fix typo causing a crash in FindByNameAndCertificate

Due to a typo in the recent refactoring the InfoPipe crashes in the
FindByNameAndCertificate request.

Additionally a state variable in set to the expected value.

Related to https://pagure.io/SSSD/sssd/issue/3863

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/ed476c870674d7fec3e5b0bb3176f83155601163">ed476c87</a></strong>
<div>
<span>by Tomas Halman</span>
<i>at 2018-11-05T19:28:14Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">DYNDNS: Drop support for legacy NSUPDATE

We should drop support for legacy versions of NSUPDATE that don't
support 'realm' option. The option 'realm' was added in
BIND 9.8.0a1.

Resolves:
https://pagure.io/SSSD/sssd/issue/2817

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/4760eae9b1b3ebb94fc5590cf5ba1a268e3120be">4760eae9</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-11-05T19:30:51Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sbus: allow access for sssd user

D-Bus allows access for root and euid by default, however when running
in non-root mode monitor continues to run as root but responsers as sssd
user. Therefore monitor euid != sssd user and the connection is terminated.

We must explicitly allow the connection for sssd user uid.

Resolves:
https://pagure.io/SSSD/sssd/issue/3871

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/0882793e4ba018073c2db9ab390bcdf16276b65f">0882793e</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-11-05T19:33:03Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">IFP: Use subreq, not req when calling RefreshRules_recv

This emits a failure when refreshing access control rules from e.g.
sssctl access-report.

Resolves:
https://pagure.io/SSSD/sssd/issue/3874

Reviewed-by: Pavel Březina <pbrezina@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/c42fb8de50435ae907811ae927fc8dc2ff6c3dd3">c42fb8de</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-11-07T20:34:32Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">CI: Make the c-ares suppression file more relaxed to prevent failures on Debian

Prevents tests running under valgrind from failing with:
==9189== 2 bytes in 1 blocks are possibly lost in loss record 1 of 195
==9189==    at 0x48357BF: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==9189==    by 0x60C7EF9: strdup (strdup.c:42)
==9189==    by 0x6E6C0EE: ares_init_options (in /usr/lib/x86_64-linux-gnu/libcares.so.2.2.0)
==9189==    by 0x5F96674: recreate_ares_channel (async_resolv.c:430)
==9189==    by 0x5F967D6: resolv_init (async_resolv.c:471)
==9189==    by 0x5F83147: be_res_init (data_provider_fo.c:884)
==9189==    by 0x5F83147: be_res_init (data_provider_fo.c:865)
==9189==    by 0x5F832B8: be_init_failover (data_provider_fo.c:70)
==9189==    by 0x127876: test_ipa_server_create_trusts_setup (test_ipa_subdomains_server.c:311)
==9189==    by 0x48541E2: ??? (in /usr/lib/x86_64-linux-gnu/libcmocka.so.0.5.1)
==9189==    by 0x4854A16: _cmocka_run_group_tests (in /usr/lib/x86_64-linux-gnu/libcmocka.so.0.5.1)
==9189==    by 0x1146CC: main (test_ipa_subdomains_server.c:999)

The failure is expected as the tests fork but the child processes never
clean up after themselves.

Reviewed-by: Sumit Bose <sbose@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/8007d6150a37c39881418e7f2b32129a5e4cb9e7">8007d615</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-11-07T20:38:30Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">INI: Return errno, not -1 on failure from sss_ini_get_stat

sss_ini_get_stat() has two branches for two libini versions. The newer
version directly returns EIO on failure, but the old version would have
returned the return value from fstat() directly. And fstat() returns -1
on failure but sets errno. This patch returns errno on failure and EOK
on success.

Reviewed-by: Michal Židek <mzidek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/4b52ed0610e399a0fe27cfab3601419acb6aff3d">4b52ed06</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-11-07T20:38:45Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">MONITOR: Don't check for pidfile if SSSD is already running

Related:
https://pagure.io/SSSD/sssd/issue/3862

The --genconf option of sssd is meant to be used to reload configuration from a
systemd socket-activated service. But it would only work if sssd was not
running, which defies its purpose.

Reviewed-by: Michal Židek <mzidek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/92b8f8d404bfe72abe8cd324f5569be5ba2c6db1">92b8f8d4</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-11-07T20:38:50Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">SSSD: Allow refreshing only certain section with --genconf

Related:
https://pagure.io/SSSD/sssd/issue/3862

Adds a new option --genconf-section for the sssd binary.  If this new
option --genconf-section is used, then only the section passed as this
option's value is refreshed.

Conversely, if this section no longer exists in the config file, then it
is removed from the confdb

Reviewed-by: Michal Židek <mzidek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/c53fc08a70679b181b0eff6422f199a51d527e67">c53fc08a</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-11-07T20:38:53Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">SYSTEMD: Re-read KCM configuration on systemctl restart kcm

Related:
https://pagure.io/SSSD/sssd/issue/3862

Uses the sssd command together with the --genconf-section=kcm option to
refresh the kcm configuration when the sssd-kcm systemd service is
restarted.

This allows the administrator to e.g. just drop a snippet to
/etc/sssd.conf.d/ or create the [kcm] section directly in the main sssd
config file, then just restart the sssd-kcm service for the changes to
apply.

Reviewed-by: Michal Židek <mzidek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/fc25224ea1a127081bf342053d1f189c10d3772d">fc25224e</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-11-07T20:38:55Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">TEST: Add a multihost test for sssd --genconf

Related:
https://pagure.io/SSSD/sssd/issue/3862

Adds a multihost test to make sure the genconf switch of sssd works in
general and even with sssd running.

Reviewed-by: Michal Židek <mzidek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/66da9d9db7bd85a17be51af610d9cfbbe4d4391c">66da9d9d</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-11-07T20:38:59Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">TESTS: Add a multihost test for changing sssd-kcm debug level by just restarting the KCM service

Related:
https://pagure.io/SSSD/sssd/issue/3862

Adds a test for the previous patches to make sure just restarting
sssd-kcm with changes to the [kcm] section applies the configuration
options from the kcm section.

Reviewed-by: Michal Židek <mzidek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/1ee12b05570fcfb8e4190c9ec704c5563138344d">1ee12b05</a></strong>
<div>
<span>by Lukas Slebodnik</span>
<i>at 2018-11-08T10:55:17Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">UTIL: Fix compilation with curl 7.62.0

The macro CURLE_SSL_CACERT is deprecated in upstream curl
since commit 3f3b26d6feb0667714902e836af608094235fca2.

  commit 3f3b26d6feb0667714902e836af608094235fca2
  Author: Han Han <hhan@thousandeyes.com>
  Date:   Wed Aug 22 11:13:32 2018 -0700

      ssl: deprecate CURLE_SSL_CACERT in favour of a unified error code

      Long live CURLE_PEER_FAILED_VERIFICATION

  sh$ git tag --contains 3f3b26d6feb0667714902e836af608094235fca2
  curl-7_62_0

It was not removed. It is just an alias to
CURLE_PEER_FAILED_VERIFICATION which causes compile time failures in
switch/case.

./src/util/tev_curl.c: In function 'curl_code2errno':
./src/util/tev_curl.c:113:5: error: duplicate case value
     case CURLE_PEER_FAILED_VERIFICATION:
     ^~~~
./src/util/tev_curl.c: 100:5: note: previously used here
     case CURLE_SSL_CACERT:
     ^~~~

Merges: https://pagure.io/SSSD/sssd/pull-request/3878

Resolves:
https://pagure.io/SSSD/sssd/issue/3875

Reviewed-by: Sumit Bose <sbose@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/4f824eca24e185e6463167b7bcc20d1398c60414">4f824eca</a></strong>
<div>
<span>by Lukas Slebodnik</span>
<i>at 2018-11-13T11:47:40Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">test_pac_responder: Skip test if pac responder is not installed

Merges: https://pagure.io/SSSD/sssd/pull-request/3881

Reviewed-by: Sumit Bose <sbose@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/fdbe67a88d1894471031b0aabc74bd1b29eef294">fdbe67a8</a></strong>
<div>
<span>by Lukas Slebodnik</span>
<i>at 2018-11-13T11:47:46Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">INTG: Show extra test summary info with pytest

It will show reasons why tests were skipped.
e.g.
  ====================== test session starts ========================
  platform linux -- Python 3.7.1, pytest-3.9.3, py-1.5.4,
                    pluggy-0.7.1 -- /usr/bin/python3
  cachedir: .pytest_cache
  rootdir: /dev/shm/sssd/src/tests/intg, inifile:
  collected 286 items / 285 deselected

  test_pac_responder.py::test_multithreaded_pac_client SKIPPED [100%]
  ==================== short test summary info ======================
  SKIP [1] test_pac_responder.py:108: No PAC responder, skipping

Merges: https://pagure.io/SSSD/sssd/pull-request/3881

Reviewed-by: Sumit Bose <sbose@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/55470b17eacdf97696b4736e9eb8bd2618601475">55470b17</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-11-13T11:48:13Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">pam_sss: return PAM_AUTHINFO_UNAVAIL if sc options are set

If pam_sss is called for PAM_USER root it currently returns
PAM_USER_UNKNOWN since SSSD does not handle root. To meet the documented
behavior if one to the sc options is used pam_sss should return
PAM_AUTHINFO_UNAVAIL in this case as well.

Related to https://pagure.io/SSSD/sssd/issue/3876

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/5e703d3d4be1aba1ed70ff0dcdbb0ec1cc07cb4a">5e703d3d</a></strong>
<div>
<span>by Lukas Slebodnik</span>
<i>at 2018-11-19T09:14:55Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">p11_child: Fix warning cast discards ‘const’ qualifier from pointer target type

The function process_responder had parameter port defined as const and
it was used in macro BIO_set_conn_port which cast 2nd parameter to
(char*)

  # define BIO_set_conn_port(b,port)     BIO_ctrl(b,BIO_C_SET_CONNECT,1, \
                                                 (char *)(port))

src/p11_child/p11_child_openssl.c: In function ‘process_responder’:
src/p11_child/p11_child_openssl.c:157:9: error: cast discards ‘const’ qualifier from pointer target type [-Werror=cast-qual]
         BIO_set_conn_port(cbio, port);
         ^~~~~~~~~~~~~~~~~

Merges: https://pagure.io/SSSD/sssd/pull-request/3880

Reviewed-by: Sumit Bose <sbose@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/f02714d6f0f5d9cdd504d5f0527849a6d6b88fab">f02714d6</a></strong>
<div>
<span>by Lukas Slebodnik</span>
<i>at 2018-11-19T09:17:40Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">CI: Modify suppression file for c-ares-1.15.0

Valgrind does not generate full stack trace for errors.
It is just limited amount of frames. Therefore we cannot see main
function with the new c-ares.

The suppression file generated with c-ares-1.14.0
{
   <insert_a_suppression_name_here>
   Memcheck:Leak
   match-leak-kinds: possible
   fun:malloc
   fun:strdup
   fun:ares_init_options
   fun:recreate_ares_channel
   fun:resolv_init
   fun:be_res_init
   fun:be_res_init
   fun:be_init_failover
   fun:test_ipa_server_create_trusts_setup
   obj:/usr/lib64/libcmocka.so.0.5.1
   fun:_cmocka_run_group_tests
   fun:main
}

The suppression file generated with c-ares-1.15.0
{
   <insert_a_suppression_name_here>
   Memcheck:Leak
   match-leak-kinds: possible
   fun:malloc
   fun:strdup
   obj:/usr/lib64/libcares.so.2.3.0
   obj:/usr/lib64/libcares.so.2.3.0
   fun:ares_init_options
   fun:recreate_ares_channel
   fun:resolv_init
   fun:be_res_init
   fun:be_res_init
   fun:be_init_failover
   fun:test_ipa_server_create_trusts_setup
   obj:/usr/lib64/libcmocka.so.0.5.1
   fun:_cmocka_run_group_tests
}

Merges: https://pagure.io/SSSD/sssd/pull-request/3884

Reviewed-by: Sumit Bose <sbose@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/6dcc34d09e0f583743dc4a74076eb30e505deed1">6dcc34d0</a></strong>
<div>
<span>by Niranjan M.R</span>
<i>at 2018-11-22T21:31:20Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">pytest/sudo: Modify fixture to restore sssd.conf

Modify set_case_sensitive_false fixture to restore sssd.conf
back to the original sssd.conf after running test_case_senitivity
test case

Signed-off-by: Niranjan M.R <mrniranjan@redhat.com>

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/ba87d78343a42e716a2ea32890b16f33c62002a2">ba87d783</a></strong>
<div>
<span>by Niranjan M.R</span>
<i>at 2018-11-22T21:31:23Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">pytest/sudo: Rename create_sudorule to case_sensitive_sudorule

Add del_sudo_rule function to delete the sudo rules
after test_sensitivity completes.

Signed-off-by: Niranjan M.R <mrniranjan@redhat.com>

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/4dcef883255b25f3a38cd67bcef3a6d3eb7d1c26">4dcef883</a></strong>
<div>
<span>by Niranjan M.R</span>
<i>at 2018-11-22T21:31:26Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">pytest/sudo: call case_sensitive_sudorule fixture instead of create_sudorule

Signed-off-by: Niranjan M.R <mrniranjan@redhat.com>

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/a5133f3ab384c2e3f37081bb73fdf705edc45ee6">a5133f3a</a></strong>
<div>
<span>by Niranjan M.R</span>
<i>at 2018-11-22T21:31:29Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">pytest/sudo: Add 2 fixtures set_entry_cache_sudo_timeout and generic_sudorule

set_entry_cache_sudo_timeout: this fixture adds entry_cache_sudo_timeout
to domain sections of sssd.conf
generic_sudorule: This is a generic sudo rule addding command /usr/bin/less
to be executed by posix user

generic_sudorule: Adds a generic sudo rule to access /usr/bin/less

Signed-off-by: Niranjan M.R <mrniranjan@redhat.com>

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/fa2106a7af9d70380c9694a1a15752f39531f6f0">fa2106a7</a></strong>
<div>
<span>by Niranjan M.R</span>
<i>at 2018-11-22T21:31:32Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">pytest/sudo: Add Testcase: sssd crashes when refreshing expired sudo rules.

Signed-off-by: Niranjan M.R <mrniranjan@redhat.com>

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/5c550e72ea9939139d2e9b0b7e3c6a534568d799">5c550e72</a></strong>
<div>
<span>by Niranjan M.R</span>
<i>at 2018-11-22T21:31:36Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">pytest: use ConfigParser() instead of SafeConfigParser()

fix the warnings of SafeConfigParser being deprectated

Signed-off-by: Niranjan M.R <mrniranjan@redhat.com>

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/406b731ddfbeb62623640cc37a7adc76af0a4b22">406b731d</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-11-22T21:40:31Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">nss: use enumeration context as talloc parent for cache req result

Otherwise we end up with memory leak since the result is never freed.

We need to convert nctx->*ent structures into talloc pointer so
we can use enum_ctx as parent.

Resolves:
https://pagure.io/SSSD/sssd/issue/3870

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/291071cb3c04eda7606d62bbff123a0a125c7d60">291071cb</a></strong>
<div>
<span>by Tomas Halman</span>
<i>at 2018-11-22T21:43:12Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">SSSCTL: user-show says that user is expired

sssctl user-show says that user is expired if the user comes from files
provider. This is ok because files user's expiration time is always set
to 0 but we should print a better, less confusing message.

The same change apply to groups.

Resolves:
https://pagure.io/SSSD/sssd/issue/3858

Reviewed-by: Pavel Březina <pbrezina@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/75696ddc84ab08c8c885dacc7796ebc8afc429ec">75696ddc</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-11-22T21:47:37Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">RESPONDER: Log failures from bind() and listen()

We've seen reports from users where SSSD (for a reason which is not
known at the moment) couldn't bind to the created socket. This patch
just logs the errno which should help in debugging issues like that in
the future.

Reviewed-by: Pavel Březina <pbrezina@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/6286f8120ac9986b418f4f08f26d6808cf028a9b">6286f812</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-11-22T21:48:46Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">p11_child(NSS): print key type in a debug message

NSS can handle EC keys automatically but a debug message indicating
which key type is used might be useful.

Related to https://pagure.io/SSSD/sssd/issue/3887

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/ef631f9e61e7a0e168cce9071470839a4c04114c">ef631f9e</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-11-22T21:48:51Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">pam_test_srv: set default value for SOFTHSM2_CONF

Currently the SOFTHSM2_CONF is not set by any fixture but some tests
sets them and other might rely on the setting done by a previous test.
This means that the tests have to run in a given order and depend on
each other.

To remove this dependency SOFTHSM2_CONF is set in the fixture to the
"default" SoftHSM2 configuration with one valid certificate. Any test
which needs a different setup must now set SOFTHSM2_CONF explicitly.

Related to https://pagure.io/SSSD/sssd/issue/3887

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/a0cdc3bdf0e7f8ef15997f269b6f1ca5cab85825">a0cdc3bd</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-11-22T21:48:54Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">tests: add ECC CA

To be able to test certificates with elliptic curve (EC) keys a new test
CA with this kind of keys is added.

Related to https://pagure.io/SSSD/sssd/issue/3887

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/a7421b5260cd2edd07ec5c0fefd240e76c5a0f03">a7421b52</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-11-22T21:48:58Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">test_pam_srv: add test for certificate with EC keys

Add an authentication test with a certificate with EC keys.

Related to https://pagure.io/SSSD/sssd/issue/3887

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/d64d9cfbe9dc44db04b253aa08c05e645e10708a">d64d9cfb</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-11-22T21:49:00Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">p11_child(openssl): add support for EC keys

Add support for EC keys to the OpenSSL version of p11_child. Please see
comments in the code for some technical details.

Related to https://pagure.io/SSSD/sssd/issue/3887

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/ad3356d105835718f57edb7844e1fed911770610">ad3356d1</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-11-22T21:49:03Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">utils: refactor ssh key extraction (OpenSSL)

Prepare the current code to allow adding other key types.

Related to https://pagure.io/SSSD/sssd/issue/3887

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/41c4661b6fd237b156606bfd0d8ca3edd5a16795">41c4661b</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-11-22T21:49:05Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">utils: add ec_pub_key_to_ssh() (OpenSSL)

Add EC key support for the OpenSSL version of the ssh key extraction
code.

Related to https://pagure.io/SSSD/sssd/issue/3887

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/4e627add38af409ec6a5023212677956babca1e7">4e627add</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-11-22T21:49:08Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">utils: refactor ssh key extraction (NSS)

Prepare the current code to allow adding other key types.

Related to https://pagure.io/SSSD/sssd/issue/3887

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/3906e5f41a00063127e07f5ca696a25eea2e8bb7">3906e5f4</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-11-22T21:49:11Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">utils: add ec_pub_key_to_ssh() (NSS)

Add EC key support for the NSS version of the ssh key extraction code.

Related to https://pagure.io/SSSD/sssd/issue/3887

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/53e6fdfd881f051898e85448832eafdd2ea09454">53e6fdfd</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-11-26T09:24:14Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">BUILD: Accept krb5 1.17 for building the PAC plugin

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/08bba3a6e3e4e21f2e20b71cca463d50420aa9ee">08bba3a6</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-11-26T09:24:18Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">tests: fix mocking krb5_creds in test_copy_ccache

To just test some ccache related functionality without talking to an
actual KDC to get the tickets some needed libkrb5 structs were mocked
based on tests from the MIT Kerberos source code. One struct member
(is_skey) was so far not regarded by libkrb5 for out test case. But a
recent fix for http://krbdev.mit.edu/rt/Ticket/Display.html?id=8718
changed this and we have to change the mocking.

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/1617f3e3dc90788ef50bbc7948c3870d08cc9c2d">1617f3e3</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-11-26T09:24:25Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">tests: increase p11_child_timeout

With recent version of valgrind some tests failed during a CI run with a
timeout. To avoid this the related p11_child_timeout is increased for
the affected tests.

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/09091b4b60456a989ecc8c3b6f76661a14c108ba">09091b4b</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-11-26T20:39:54Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">LDAP: minor refactoring in auth_send() to conform to our coding style

Related:
https://pagure.io/SSSD/sssd/issue/3451

A tevent _send() function should only return NULL on ENOMEM, otherwise
it should mark the request as failed but return the req pointer. This
was not much of an issue, before, but the next patch will add another
function call to the auth_send call which would make error handling
awkward.

Reviewed-by: Sumit Bose <sbose@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/57fc60c9dc77698cf824813c36eb0f90d767b315">57fc60c9</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-11-26T20:40:05Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">LDAP: Only authenticate the auth connection if we need to look up user information

Related:
https://pagure.io/SSSD/sssd/issue/3451

Commit add72860c7a7a2c418f4d8b6790b5caeaf7dfb7b initially addressed #3451 by
using the full sdap_cli_connect() request during LDAP authentication. This
was a good idea as it addressed the case where the authentication connection
must also look up some user information (typically with id_provider=proxy
where you don't know the DN to bind as during authentication), but this
approach also broke the use-case of id_provider=ldap and auth_provider=ldap
with ldap_sasl_auth=gssapi.

This is because (for reason I don't know) AD doesn't like if you use
both GSSAPI and startTLS on the same connection. But the code would
force TLS during the authentication as a general measure to not transmit
passwords in the clear, but then, the connection would also see that
ldap_sasl_auth=gssapi is set and also bind with GSSAPI.

This patch checks if the user DN is already known and if yes, then
doesn't authenticate the connection as the connection will then only be
used for the user simple bind.

Reviewed-by: Sumit Bose <sbose@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/6f113c7ddeaa5c82558e10118b499d22bf7a2b14">6f113c7d</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-11-26T20:40:14Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">LDAP: Log the encryption used during LDAP authentication

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/df9e4802c060fc21d38f238265805092352e5c95">df9e4802</a></strong>
<div>
<span>by Tomas Halman</span>
<i>at 2018-12-05T19:46:51Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">DYNDNS: Convert dyndns timer to be_ptask

We have API for periodic tasks and we want to
use it instead of tevent timers.

Resolves:
https://pagure.io/SSSD/sssd/issue/1944

Reviewed-by: Pavel Březina <pbrezina@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/f47940356462a3f477fe462e71d7680c959300db">f4794035</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-12-05T19:49:49Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sss_iface: prevent from using invalid names that start with digits

https://dbus.freedesktop.org/doc/dbus-specification.html#message-protocol-names

- Bus names that start with a colon (':') character are unique connection names. Other bus names are called well-known bus names.
- Bus names are composed of 1 or more elements separated by a period ('.') character. All elements must contain at least one character.
- Each element must only contain the ASCII characters "[A-Z][a-z][0-9]_-", with "-" discouraged in new bus names. Only elements that are part of a unique connection name may begin with a digit, elements in other bus names must not begin with a digit.
- Bus names must contain at least one '.' (period) character (and thus at least two elements).
- Bus names must not begin with a '.' (period) character.
- Bus names must not exceed the maximum name length (255).

Resolves:
https://pagure.io/SSSD/sssd/issue/3872

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/36255b893a8a55588309a7c5729560c48a30018d">36255b89</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2018-12-05T20:01:14Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">ci: add ability to run tests in jenkins

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/807bbce25ffedb6f0d2d61831b5d5133e11aa84a">807bbce2</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-12-05T20:02:39Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">PROXY: Copy the response to the caller

Resolves:
https://pagure.io/SSSD/sssd/issue/3892

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/7354e59e010197ab5be3440a0e2c24302298a237">7354e59e</a></strong>
<div>
<span>by George McCollister</span>
<i>at 2018-12-10T22:18:11Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">build: remove hardcoded samba include path

SMBCLIENT_FLAGS already provides the samba-4.0 include path. Remove the
hardcoded samba include path which causes cross-compilation warnings
such as:

cc1: warning: include location "/usr/include/samba-4.0" is unsafe for
cross-compilation [-Wpoison-system-directories]

These warnings are treated as errors on build systems such as
OpenEmbedded.

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/5565dd365e704f6ded4f95db5bfbefd5dffc888b">5565dd36</a></strong>
<div>
<span>by Tomas Halman</span>
<i>at 2018-12-10T22:20:39Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">DYNDNS: SSSD does not batch DDNS update requests

SSSD includes a 'send' command in between each record modification
and does not batch DDNS update requests. This is problematic in
complex AD environments because those requests may not be processed
by the same server.

Now forward zone update is done in two steps - one per
protocol family. If dyndns_update_per_family is set
to false, update is performed in single step.

Resolves:
https://pagure.io/SSSD/sssd/issue/3829

Reviewed-by: Pavel Březina <pbrezina@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/9096fc01cca8fcaeb19c36a27f3a9fa09d60772a">9096fc01</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-12-11T21:31:31Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">Revert "IPA: use forest name when looking up the Global Catalog"

This reverts commit 149174acae677d1e72a0da431bf0850d55f2ccb4.

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/62d671b874a66101c0f4bff39fc6d7f49cb8fca6">62d671b8</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-12-11T21:31:38Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">ipa: use only the global catalog service of the forest root

While creating the domains and sub-domains each domain gets a global
catalog services assigned but only one should be used because the global
catalog is by definition responsible for the whole forest so it does not
make sense to use a global catalog service for each domain and in the
worst case connect to the same GC multiple times.

In the AD provider this is simple because the GC service of the
configured domain AD_GC_SERVICE_NAME ("AD_GC") can be used. In the IPA
case all domains from the trusted forest are on the level of sub-domains
so we have to pick one. Since the forest root is linked from all domain
of the same forest it will be the most straight forward choice.

Related to https://pagure.io/SSSD/sssd/issue/3902

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/90f32399b4100ce39cf665649fde82d215e5eb49">90f32399</a></strong>
<div>
<span>by Tomas Halman</span>
<i>at 2018-12-11T21:51:20Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">nss: sssd returns '/' for emtpy home directories

For empty home directory in passwd file sssd returns "/". Sssd
should respect system behaviour and return the same as nsswitch
"files" module - return empty string.

Resolves:
https://pagure.io/SSSD/sssd/issue/3901

Reviewed-by: Simo Sorce <simo@redhat.com>
Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/d33eaac8761001af6ae7836c177bbdd6ac79fce9">d33eaac8</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-12-13T11:06:51Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">p11_child(openssl): do not free static memory

Coverity found a code path where we try to free a static array, this
patch removes it.

Related to https://pagure.io/SSSD/sssd/issue/3887

Reviewed-by: Michal Židek <mzidek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/e49e9f727e4960c8a0a2ed50488dac6e51ddf284">e49e9f72</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2018-12-13T11:09:13Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">krb5_child: fix permissions during SC auth

For PKINIT we might need access to the pcscd socket which by default is
only allowed for authenticated users. Since PKINIT is part of the
authentication and the user is not authenticated yet, we have to use
different privileges and can only drop it only after the TGT is
received. The fast_uid and fast_gid are the IDs the backend is running
with. This can be either root or the 'sssd' user. Root is allowed by
default and the 'sssd' user is allowed with the help of the
sssd-pcsc.rules policy-kit rule. So those IDs are a suitable choice. We
can only call switch_creds() because after the TGT is returned we have
to switch to the IDs of the user to store the TGT.

The final change to the IDs of the user is not only important for KCM
type credential caches but for file based ccache types like FILE or DIR
as well.

Related to https://pagure.io/SSSD/sssd/issue/3903

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/61e4ba58934b20a950255e05797aca25aadc1242">61e4ba58</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2018-12-13T11:14:06Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">NSS: Avoid changing the memory cache ownership away from the sssd user

Resolves:
https://pagure.io/SSSD/sssd/issue/3890

In case SSSD is compiled --with-sssd-user but run as root (which is the
default on RHEL and derivatives), then the memory cache will be owned by
the user that sssd_nss runs as, so root.

This conflicts with the packaging which specifies sssd.sssd as the owner. And
in turn, this means that users can't reliably assess the package integrity
using rpm -V.

This patch makes sure that the memory cache files are chowned to sssd.sssd
even if the nss responder runs as root.

Also, this patch changes the sssd_nss responder so that is becomes a member
of the supplementary sssd group. Even though in traditional UNIX sense,
a process running as root could write to a file owned by sssd:sssd, with
SELinux enforcing mode this becomes problematic as SELinux emits an error
such as:

type=AVC msg=audit(1543524888.125:1495): avc:  denied  { fsetid } for
pid=7706 comm="sssd_nss" capability=4  scontext=system_u:system_r:sssd_t:s0
tcontext=system_u:system_r:sssd_t:s0 tclass=capability

To make it possible for the sssd_nss process to write to the files, the
files are also made group-writable. The 'others' permission is still set
to read only.

Reviewed-by: Michal Židek <mzidek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/170625872a7d53c182ef095b4e5cba29f632c0c4">17062587</a></strong>
<div>
<span>by Alexey Tikhonov</span>
<i>at 2018-12-13T11:16:35Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">Fix error in hostname retrieval

Fix off-by-one error in gethostname() param

Resolves:
https://pagure.io/SSSD/sssd/issue/3865

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/15bde7dab466fc4f2719ce709de9dac7e1e10de8">15bde7da</a></strong>
<div>
<span>by Alexey Tikhonov</span>
<i>at 2019-01-06T19:12:55Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">util/tev_curl: Fix double free error in schedule_fd_processing()

Prevents deletion of already executed timer.

libcurl calls schedule_fd_processing every time it wants to have (new)
timer setup. And it happens quite a lot of times.

Being called schedule_fd_processing wants to delete previous timer before
creating new one. That's basically proper action. The problem is it sometimes
tries to delete non-existent (already executed and deleted by libtevent)
timer. This is "double free" case.

Proposed patch sets timer pointer in tcurl_ctx to NULL at the end of timer
handler (actually can be placed anywhere in the handler: handler called =>
timer deleted by libtevent). That prevents schedule_fd_processing from
freeing already freed memory. And I think it is good idea overall to not
have pointers to freed memory (dangling pointers). This should be safe
operation since all processes are single-thread proces

Resolves:
https://pagure.io/SSSD/sssd/issue/3917

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/814889a7f4691a135b617058c3ae876b54d5b226">814889a7</a></strong>
<div>
<span>by Tomas Halman</span>
<i>at 2019-01-06T19:17:19Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">ifp: extraAttributes is UnknownProperty

Attempting to get extraAttributes via SSSD's ifp fails.

Here I uncomment interface function for extraAttributes.
also right for querying this interface is changed to allow
this request.

Resolves:
https://pagure.io/SSSD/sssd/issue/3906

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/e4469fbdb3d5c53294c6514280ac75b847b3c61c">e4469fbd</a></strong>
<div>
<span>by Adam Williamson</span>
<i>at 2019-01-06T19:20:29Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sbus: use 120 second default timeout

As discussed in #1654537, first login to a system as a FreeIPA
domain user now usually causes an expensive SELinux operation
to happen; this can take longer than the default bus message
timeout of 25 seconds. To deal with this for now, let's use a
120 second default timeout; this is a big hammer, but unless we
can refactor things to use a longer timeout just for that one
call, or make the actual operation take less time, there's not
much else we can do.

Resolves:
https://bugzilla.redhat.com/show_bug.cgi?id=1654537

Resolves:
https://pagure.io/SSSD/sssd/issue/3909

Signed-off-by: Adam Williamson <awilliam@redhat.com>

Reviewed-by: Pavel Březina <pbrezina@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/02c15d40efe6dd9107528469904f1315fca37416">02c15d40</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2019-01-08T22:19:21Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">KCM: Deleting a non-existent ccache should not yield an error

Resolves:
https://pagure.io/SSSD/sssd/issue/3910

When the KCM destroy operation is called, it receives a name as an input. If
the name cannot be found, we would currently return KRB5_CC_NOTFOUND. But
other ccache types return KRB5_FCC_NOFILE in that case and e.g. utilities
like kdestroy special case KRB5_FCC_NOFILE to be non-fatal.

Reviewed-by: Tomas Halman <thalman@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/46e52b0361e1c36ae640400a4a683e03b2c4164b">46e52b03</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2019-01-08T22:19:45Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">TESTS: Add a test for deleting a non-existent ccache with KCM

Reviewed-by: Tomas Halman <thalman@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/f94881d4d34959231fedbaafd5f1fd6f5e9d8924">f94881d4</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2019-01-08T22:22:53Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">MAN: Explicitly state that not all generic domain options are supported for the files provider

Resolves:
https://pagure.io/SSSD/sssd/issue/3882

Reviewed-by: Tomas Halman <thalman@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/8e9e8011ce17860bec67a572e4c11a9178c03b8e">8e9e8011</a></strong>
<div>
<span>by Alexey Tikhonov</span>
<i>at 2019-01-10T21:19:46Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">CONFIG: validator rules & test

Add support of 'certmap' config section to validator rules

Resolves:
https://pagure.io/SSSD/sssd/issue/3845

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/bf248a3971b2794e0c82324081ac182dd74e2e9e">bf248a39</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2019-01-10T21:27:59Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">ci: add Fedora 29

Enables testing on Fedora 29 distro.

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/8a3517c5466c107f4d4e0970a1c33b51d6c762f8">8a3517c5</a></strong>
<div>
<span>by Michal Židek</span>
<i>at 2019-01-10T21:34:49Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">confdb: Always read snippet files

This patch removes the ldif with fallback configuration
and adds the fallback configuration as in-memory
INI snippet.

Fixes:
https://pagure.io/SSSD/sssd/issue/3439

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/b66f8dc3bd4e89c424bef5953aeb70742f9656dd">b66f8dc3</a></strong>
<div>
<span>by Michal Židek</span>
<i>at 2019-01-10T21:34:52Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">CONFDB: Remove old libini support

Remove code code that uses libini older then v1

Related:
https://pagure.io/SSSD/sssd/issue/3439

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/ea7ada6c0629df45348f699e30acc44194550801">ea7ada6c</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2019-01-10T21:39:52Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">idmap_sss: improve man page

The misleading in the idmap_sss man page is improved.

Related to https://pagure.io/SSSD/sssd/issue/3912

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/bc1e8ffd5cca74aa8408c1c6bce0a3cf42a0974b">bc1e8ffd</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2019-01-17T20:47:41Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sbus: do not use signature when copying dictionary entry

Resolves:
https://pagure.io/SSSD/sssd/issue/3921

When we open container for DBUS_TYPE_DICT_ENTRY, dbus expects the
signature to be NULL.

Reviewed-by: Sumit Bose <sbose@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/b3285f9f8a5eac3e4e70ed3bd6b74c15ad806e9e">b3285f9f</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2019-01-28T20:42:18Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">AD/IPA: Reset subdomain service name, not domain name

Related:
https://pagure.io/SSSD/sssd/issue/3911

Since commit 778f241e78241b0d6b8734148175f8dee804f494 the subdomain fail
over services use the "sd_" prefix. This was done to make it easier,
until the whole failover design works better with subdomains, to see
which services belong to the main domain from tools.

However, some parts of the code would still just use the domain name for
the failover service, which meant the service was not found, notably
when trying to reset services:

(Thu Dec 13 05:29:31 2018) [sssd[be[testrelm.test]]] [ipa_srv_ad_acct_retried] (0x0400): Subdomain re-set, will retry lookup
(Thu Dec 13 05:29:31 2018) [sssd[be[testrelm.test]]] [be_fo_reset_svc] (0x1000): Resetting all servers in service ipaad2016.test
(Thu Dec 13 05:29:31 2018) [sssd[be[testrelm.test]]] [be_fo_reset_svc] (0x0080): Cannot retrieve service [ipaad2016.test]

This patch switches to reading the service names from the ad_options and
the sdap_service structures that are contained within ad_options.

Reviewed-by: Tomas Halman <thalman@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/aaaa9a3e836e7b7af1ff0fc5058ddddfeef120a8">aaaa9a3e</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2019-01-28T20:42:41Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">IPA: Add explicit return after tevent_req_error

When working on another patch I realized that we don't use explicit
return after failing a request. This could be potentially fatal as the
code would continue, perhaps with data that is not defined.

Reviewed-by: Tomas Halman <thalman@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/484b48ff40d638af042f3b4d4a00f87a8a58a76c">484b48ff</a></strong>
<div>
<span>by Alexey Tikhonov</span>
<i>at 2019-01-28T20:45:30Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sss_client/common.c: fix Coverity issue

Usage of
strncpy(nssaddr.sun_path, socket_name, sizeof(nssaddr.sun_path))
1) confuses Coverity due to 3rd argument being equal to sizeof(1st)
2) again zeroes previously zeroed buffer
So replaced with strcpy()
This should be safe due to existing check of sizes.

Resolves:
https://pagure.io/SSSD/sssd/issue/3841

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/9959fbe70f8314a2b5ecf52a2e92bcd6a38bfefe">9959fbe7</a></strong>
<div>
<span>by Alexey Tikhonov</span>
<i>at 2019-01-28T20:45:34Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sss_client/common.c: fix off-by-one error in sizes check

`sizeof(nssaddr.sun_path)` being equal to `strlen(socket_name) + 1`
should be fine, not error.

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/bc92d36c96b4404353d991f42568b58cdf0a2d5a">bc92d36c</a></strong>
<div>
<span>by Alexey Tikhonov</span>
<i>at 2019-01-28T20:48:52Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sss_client/common.c: comment amended

Amended comment for `make_safe_fd()`

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/6e2df759de3b018c186900bde668da53ac675e10">6e2df759</a></strong>
<div>
<span>by Alexey Tikhonov</span>
<i>at 2019-01-28T20:48:59Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sss_client/nss_services.c: indentation fixed

Fixed indentation (tabs -> spaces) to follow "Coding guidelines"

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/08d5dabc50341034a41150018f8efd83555003c7">08d5dabc</a></strong>
<div>
<span>by Alexey Tikhonov</span>
<i>at 2019-01-28T20:49:01Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sss_client/nss_services.c: fixed incorrect mutex usage

Fixed "unlock mutex without previous lock" bug

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/0d96e175a4dc177d372cd56a25f155d80f369121">0d96e175</a></strong>
<div>
<span>by Alexey Tikhonov</span>
<i>at 2019-01-28T20:49:04Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sss_client: global unexported symbols made static

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/88c0c3fcd1d97bd499bb28c2065ba19d629fa4f7">88c0c3fc</a></strong>
<div>
<span>by Lukas Slebodnik</span>
<i>at 2019-01-29T19:55:43Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sss_cache: Do not fail for missing domains

The conf.db needn't exist(sssd has never been started) and in such situation
sss_cache failed when trying to invalidate all entries.

There is nothing to invalidate and therefore we are already in state
which we want to achieve with calling sss_cache.
No reason to fail.

Resolves:
https://pagure.io/SSSD/sssd/issue/3919

Merges: https://pagure.io/SSSD/sssd/pull-request/3926

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/325df4acae303efeabd96d2247fb5799c728536a">325df4ac</a></strong>
<div>
<span>by Lukas Slebodnik</span>
<i>at 2019-01-29T19:55:50Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">intg: Add test for sss_cache & shadow-utils use-case

Related to:
https://pagure.io/SSSD/sssd/issue/3919

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/71475f1ed78a65d78f75e5ca0fdc6e20cfdf2f39">71475f1e</a></strong>
<div>
<span>by Lukas Slebodnik</span>
<i>at 2019-01-29T19:55:53Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sss_cache: Do not fail if noting was cached

It might happen that we have some domains in conf.db but nothing
has been cached yet. sss_cache failed in such situation,

bash-4.4# sss_cache -E
No cache object matched the specified search
bash-4.4# echo $?
2

Because there is nothing to invalidate and so we are already in state
which we want to achieve with calling sss_cache.
There is no reason to fail.

We will still fail for invalidating particular entry. User might have a
typo in the name and should be informed about possible mistake.

bash-4.4# sss_cache -u test_user
No cache object matched the specified search
bash-4.4# echo $?
2

Resolves:
https://pagure.io/SSSD/sssd/issue/3919

Merges: https://pagure.io/SSSD/sssd/pull-request/3926

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/415094687e92789060626176c5ced31d4122692d">41509468</a></strong>
<div>
<span>by Lukas Slebodnik</span>
<i>at 2019-01-29T19:55:59Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">test_sss_cache: Add test case for invalidating missing entries

Related to:
https://pagure.io/SSSD/sssd/issue/3919

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/9b06c750b66ac675d3be19b15a60888153f11758">9b06c750</a></strong>
<div>
<span>by Lukas Slebodnik</span>
<i>at 2019-01-29T20:11:11Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">pyhbac-test: Do not use assertEquals

src/tests/pyhbac-test.py:163: DeprecationWarning: Please use assertEqual instead.
  self.assertEquals(el.__repr__(), u'<category 0 names [] groups []>')
src/tests/pyhbac-test.py:169: DeprecationWarning: Please use assertEqual instead.
  u'<category 1 names [foo] groups [bar, baz]>')

Merges: https://pagure.io/SSSD/sssd/pull-request/3927

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/c4db34c17a415abff585632d3f7547392e564cc2">c4db34c1</a></strong>
<div>
<span>by Lukas Slebodnik</span>
<i>at 2019-01-29T20:11:14Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">SSSDConfigTest: Do not use assertEquals

src/config/SSSDConfigTest.py:88: DeprecationWarning: Please use assertEqual instead.
  self.assertEquals(new_options['debug_level'][0], int)
src/config/SSSDConfigTest.py:91: DeprecationWarning: Please use assertEqual instead.
  self.assertEquals(new_options['command'][0], str)
src/config/SSSDConfigTest.py:94: DeprecationWarning: Please use assertEqual instead.
  self.assertEquals(new_options['reconnection_retries'][0], int)

+ many more

Merges: https://pagure.io/SSSD/sssd/pull-request/3927

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/769dc244771db5aebeada2f45fbb284b8076cc42">769dc244</a></strong>
<div>
<span>by Lukas Slebodnik</span>
<i>at 2019-01-29T20:11:16Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">SSSDConfig: Fix ResourceWarning unclosed file

/usr/lib64/python3.7/unittest/case.py:763:
    ResourceWarning: unclosed file <_io.TextIOWrapper name='src/config/testconfigs/sssd-invalid.conf'
                                    mode='r' encoding='UTF-8'>
  context = None
ResourceWarning: Enable tracemalloc to get the object allocation traceback
/usr/lib64/python3.7/unittest/case.py:763:
    ResourceWarning: unclosed file <_io.TextIOWrapper name='src/config/testconfigs/noparse.api.conf'
                                    mode='r' encoding='UTF-8'>
  context = None
ResourceWarning: Enable tracemalloc to get the object allocation traceback

Merges: https://pagure.io/SSSD/sssd/pull-request/3927

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/21bba050994aa59ab275a99dfa711b36d2900ebb">21bba050</a></strong>
<div>
<span>by Lukas Slebodnik</span>
<i>at 2019-01-29T20:11:18Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">SSSDConfigTest: Remove usage of failUnless

src/config/SSSDConfigTest.py:1855: DeprecationWarning: Please use assertTrue instead.
  self.failUnless(domain.get_name() in sssdconfig.list_domains())
src/config/SSSDConfigTest.py:1856: DeprecationWarning: Please use assertTrue instead.
  self.failUnless(domain.get_name() in sssdconfig.list_inactive_domains())
src/config/SSSDConfigTest.py:1585: DeprecationWarning: Please use assertTrue instead.
  self.failUnless(service.get_name() in sssdconfig.list_services())

Merges: https://pagure.io/SSSD/sssd/pull-request/3927

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/39b3b0e4a90d0624040319a177ebcb81568fc1b9">39b3b0e4</a></strong>
<div>
<span>by Lukas Slebodnik</span>
<i>at 2019-01-29T20:14:30Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">BUILD: Fix condition for building sssd-kcm man page

Merges: https://pagure.io/SSSD/sssd/pull-request/3928

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/f2a327f5a355b04b9aabc6d3b8c817cd0d82b534">f2a327f5</a></strong>
<div>
<span>by Lukas Slebodnik</span>
<i>at 2019-01-29T20:20:15Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">DIST: Do not use conditional include for template files

We do not use AC_CONFIG_FILES in configure.ac for replacing some values
at configure time because we need to replace them at build time in make.

And because such files are not listed in AC_CONFIG_FILES we need to have
them in the automake variable EXTRA_DIST. And they must not be included
conditionally. We want to have always in dist tarball besides. It must
not depend on the configuration of system which was used for generating
tarball

The automake documentation says:
  Still, sometimes there are files which must be distributed, but which
  are not covered in the automatic rules. These files should be listed
  in the EXTRA_DIST variable.

Merges: https://pagure.io/SSSD/sssd/pull-request/3929

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/afd23bd7fbb3dad0e80d9e77e94ca2abf67d19b0">afd23bd7</a></strong>
<div>
<span>by Lukas Slebodnik</span>
<i>at 2019-01-29T20:26:42Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">NSS: Do not use deprecated header files

In file included from src/util/cert/nss/cert.c:26:
/usr/include/nss3/key.h:9:9: note: #pragma message: key.h is deprecated. Please include keyhi.h instead.
 #pragma message("key.h is deprecated. Please include keyhi.h instead.")
         ^~~~~~~

Merges: https://pagure.io/SSSD/sssd/pull-request/3930

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/49c13e9aa84f1889cd2b50397b6a4f74c9dfba57">49c13e9a</a></strong>
<div>
<span>by Alexey Tikhonov</span>
<i>at 2019-01-31T21:06:50Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">providers/ldap: abort unsecure authentication requests

Abort LDAP authentication if the connection is not authenticated and
SDAP_DISABLE_AUTH_TLS is off.

Resolves: https://pagure.io/SSSD/sssd/issue/3889

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/53cc1187daee486bf4ba80dc3475944995ef4df6">53cc1187</a></strong>
<div>
<span>by Alexey Tikhonov</span>
<i>at 2019-01-31T21:06:53Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">providers/ldap: fixed check of ldap_get_option return value

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/a04d088d99f953c7b98c16cf23eedff62e9483bf">a04d088d</a></strong>
<div>
<span>by Alexey Tikhonov</span>
<i>at 2019-01-31T21:06:57Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">providers/ldap: init sasl_ssf in specific case

Set value of sasl ssf if ldap_get_option fails.

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/194438830cdd729e317c1e1baf93da7201dfd39b">19443883</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2019-02-05T20:56:26Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sbus: avoid using invalid stack point in SBUS_INTERFACE

SBUS_INTERFACE macros expanded as:
struct sbus_interface bus =
    ({ sbus_interface(
        "org.freedesktop.DBus",
        ((void *)0),
        (((const struct sbus_method[])
        {
            ({
                /* ... compile time check of function signature omitted */ ;
                sbus_method_sync(/* ... full list of params omitted */);
            }),
  ...

This however includes an issue that methods/properties/signals are returned
by value, however stored in sbus_interface as pointers. Once we return out
of the top-level block and assign resulting sbus_interface into 'bus' variable
those objects allocated on stack becomes invalid and can be overwritten by other
allocations on stack.

This patch overcomes this issue by changing declaration of SBUS_INTERFACE and
avoiding using this top-level block. This still keeps the declarative structure
and simplifies the code as it does not require any memory handling and
tests for successful allocations.

    const struct sbus_method __ ## varname ## _m[] = methods;                 \
    const struct sbus_signal __ ## varname ## _s[] = signals;                 \
    const struct sbus_property __ ## varname ## _p[] = properties;            \
    struct sbus_interface varname = SBUS_IFACE_ ## iface(                     \
        (__ ## varname ## _m),                                                \
        (__ ## varname ## _s),                                                \
        (__ ## varname ## _p)                                                 \
    )

Resolves:
https://pagure.io/SSSD/sssd/issue/3924

Reviewed-by: Alexey Tikhonov <atikhono@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/e185b039468ec27bbc905c61c57dffc5496af521">e185b039</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2019-02-05T20:58:12Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sbus: improve documentation of SBUS_INTERFACE

Reviewed-by: Alexey Tikhonov <atikhono@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/38ebae7e0ea889fa9022670a3e08e7352b624677">38ebae7e</a></strong>
<div>
<span>by Alexey Tikhonov</span>
<i>at 2019-02-05T21:01:21Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sbus/interface: fixed interface copy helpers

In `sbus_method_copy()` and other copy helpers there was code like:
```
copy = talloc_zero_array(mem_ctx, struct sbus_method, count + 1);
memcpy(copy, input, sizeof(struct sbus_method) * count + 1);
```
Copy of one byte of "sentinel" doesn't make a sense.
We can either rely on the fact that sentinel is zero-initialized struct
*and* `talloc_zero_array()` zero-initializes memory (so copying of
sentinel may be omitted at all) or just copy sentinel in a whole.
Opted for second option as more clear variant.

Reviewed-by: Pavel Březina <pbrezina@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/bb98486fb6c27b7207235cb77165b083b6ccba33">bb98486f</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2019-02-05T21:03:10Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">MULTIHOST: Do not use the deprecated namespace

This issue was causing warnings with the current pytest versions as
installed from pip.

See:
https://docs.pytest.org/en/latest/deprecations.html#pytest-namespace

Reviewed-by: Niranjan M.R <mrniranjan@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/ee9fdb08f73571683b272f06b6ea25630a2dbe01">ee9fdb08</a></strong>
<div>
<span>by Tomas Halman</span>
<i>at 2019-02-05T21:13:40Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">SSSCTL: user-checks does not show custom attributes

"sssctl user-checks" also query the IFP interface of sssd. The problem
is, that this command only displays POSIX user attrs, no matter if we
configured the InfoPipe for additional user attributes.

With this patch additional attributes are also printed out.

Resolves:
https://pagure.io/SSSD/sssd/issue/3866

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
Reviewed-by: Sumit Bose <sbose@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/2de3c5fb2490da0dabed0de498a8296db85a1e61">2de3c5fb</a></strong>
<div>
<span>by Lukas Slebodnik</span>
<i>at 2019-02-05T21:38:57Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sss_cache: Fail if unknown domain is passed in parameter

If sss_cache is called with --domain parameter we should fail
in case of unknown domain. It might be a typo and user should know
about such case.

Resolves:
https://pagure.io/SSSD/sssd/issue/3919

Merges: https://pagure.io/SSSD/sssd/pull-request/3940

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/159a2316b8d5560da5264022c598f1072f21bdba">159a2316</a></strong>
<div>
<span>by Lukas Slebodnik</span>
<i>at 2019-02-05T21:39:00Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">test_sss_cache: Add test case for wrong domain in parameter

Related to:
https://pagure.io/SSSD/sssd/issue/3919

Merges: https://pagure.io/SSSD/sssd/pull-request/3940

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/7133c7fc7f6dc561ad59d62a604b20316cb45e24">7133c7fc</a></strong>
<div>
<span>by Lukas Slebodnik</span>
<i>at 2019-02-05T21:42:36Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">Remove macro ZERO_STRUCT

It was used just on few places and we directly call memset on many
other places

[~/sssd]$ git grep memset | grep sizeof | wc -l
97

Merges: https://pagure.io/SSSD/sssd/pull-request/3941

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/334950e4b352ffa3a672f30f62b478c69690c830">334950e4</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2019-02-08T14:02:43Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">KCM: Return a valid tevent error code if a request cannot be created

Previously we were returning whatever was in 'ret' which is wrong,
typically it would have been EOK as returned from a previous successfull
call or even an uninitialized value.

Reviewed-by: Michal Židek <mzidek@redhat.com>
Reviewed-by: Simo Sorce <simo@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/7c441a13215dfd87f9facdaf5f6bcc19a25ec472">7c441a13</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2019-02-08T14:02:52Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">KCM: Allow representing ccaches with a NULL principal

Related:
https://pagure.io/SSSD/sssd/issue/3873

We need to make it possible to create an internal ccache representation
without passing in a principal. The principal is only assigned to the
ccache with krb5_cc_initialize(), but some programs like openssh use the
following sequence of calls:
    krb5_cc_new_unique
    krb5_cc_switch
    krb5_cc_initialize

Reviewed-by: Michal Židek <mzidek@redhat.com>
Reviewed-by: Simo Sorce <simo@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/d0eae0598cfb37e1e5aca10a0827b912f707d783">d0eae059</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2019-02-08T14:03:01Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">KCM: Create an empty ccache on switch to a non-existing one

Related:
https://pagure.io/SSSD/sssd/issue/3873

We need to make it possible to create an internal ccache representation
without passing in a principal. The principal is only assigned to the
ccache with krb5_cc_initialize(), but some programs like openssh use the
following sequence of calls:
    cc = krb5_cc_new_unique
    krb5_cc_switch(cc)
    krb5_cc_initialize(cc, principal)

Since switch changes the default ccache, we create a 'dummy' ccache with
krb5_cc_switch() and then the initialize call just fills in the details.

Reviewed-by: Simo Sorce <simo@redhat.com>
Reviewed-by: Michal Židek <mzidek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/ac95d4f0796f3f372f47f540afda7573adc63b39">ac95d4f0</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2019-02-08T14:03:08Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">TESTS: Add a multihost test for ssh credentials forwarding

Related:
https://pagure.io/SSSD/sssd/issue/3873

Reviewed-by: Michal Židek <mzidek@redhat.com>
Reviewed-by: Simo Sorce <simo@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/3eb99a171f59454fc2ec130b3e5052b3de5569a2">3eb99a17</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2019-02-10T20:20:14Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">PAM: use user name hint if any domain has set it

When using multiple domains the user name hint should be shown even if
only one domain has set the flag to have a consistent user experience.
Currently this would only be related to logins with GDM and activated
GDM Smartcard plugin.

Related to https://pagure.io/SSSD/sssd/issue/3949

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/12f74f8c98fac6a7eeb3937f623949bcb3adb547">12f74f8c</a></strong>
<div>
<span>by Alexey Tikhonov</span>
<i>at 2019-02-10T20:22:53Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">lib/cifs_idmap_sss: fixed unaligned mem access

Fixed following warning:
```
lib/cifs_idmap_sss/cifs_idmap_sss.c: In function ‘sss_sid_to_id’:
lib/cifs_idmap_sss/cifs_idmap_sss.c:221:47: warning: taking address
of packed member of ‘struct cifs_uxid’ may result in an unaligned
pointer value [-Waddress-of-packed-member]

err = sss_nss_getidbysid(sid, (uint32_t *)&cuxid->id.uid, &id_type);
```

Actually there are two issues:
1) Packed `cifs_uxid::id.uid` may be unaligned thus generating run time
error on some architectures (as compiler complains);
2) In theory size of `uid_t` may be different than size of `uint32_t`
thus resulting in corruption of `cifs_uxid` content.

Proposed patch is not ideal due to `(uid_t)uid` cast but solves most
of issues with minimal effor. Proper solution would require patching of
`sss_nss_getidbysid()` and all underlying functions for no good reason.

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/d409df3328545d8b326618f008a91f8f7d903a9b">d409df33</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2019-02-10T20:28:22Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">MAN: Add sssd-files(5) to the See Also section

Resolves:
https://pagure.io/SSSD/sssd/issue/3936

Reviewed-by: Michal Židek <mzidek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/e32920a9c7998476b596507762b28fcf81f6a456">e32920a9</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2019-02-16T21:52:09Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">utils: make N_ELEMENTS public

The calculation of the size of an array is used in the main code as
well, so it makes sense to move N_ELEMENTS to a common header to use it
more often.

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/e1ff063ffa4ad459bd0c8a9da56abf8379401dbe">e1ff063f</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2019-02-16T21:52:15Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">ad: replace ARRAY_SIZE with N_ELEMENTS

ARRAY_SIZE is taken from the Samba header file memory.h which is not
available as a public header in newer Samba versions anymore. This patch
replaces it with an internal macro.

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/d575d85c06093a560441aa2c177284bbe9891300">d575d85c</a></strong>
<div>
<span>by Alexey Tikhonov</span>
<i>at 2019-02-16T22:05:26Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">Util: fixed mistype in error string representation

Resolves: https://pagure.io/SSSD/sssd/issue/3916

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/138059b2cacfc82d7a72c46c5c944b25f0ed5e68">138059b2</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2019-02-16T22:14:46Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">ci: add Fedora Rawhide

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/05c6ed5508f3a7299fb46bd15cda32f05757e458">05c6ed55</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2019-02-16T22:18:29Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">TESTS: Add a simple integration test for retrieving the extraAttributes property

Related:
https://pagure.io/SSSD/sssd/issue/3906

Reviewed-by: Niranjan M.R <mrniranjan@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/e1755a00faf993e50db5dafa7671d88456a2a0ad">e1755a00</a></strong>
<div>
<span>by Tomas Halman</span>
<i>at 2019-02-16T22:21:43Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">ssh: sssd_ssh fails completely on p11_child timeout

When p11_child fails or timeout is reached, ssh login fails.
With this patch sssd_ssh proceeds to ssh key and password login.

Resolves:
https://pagure.io/SSSD/sssd/issue/3937

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
Reviewed-by: Sumit Bose <sbose@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/52c8336137a97af4843ff8074e7b3478f766f6d8">52c83361</a></strong>
<div>
<span>by Tomas Halman</span>
<i>at 2019-02-16T22:21:50Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">ssh: p11_child error message is too generic

We have only one error core for p11_child error. With this patch
new error ERR_P11_CHILD_TIMEOUT is introduced. It is then used
for better log message.

Resolves:
https://pagure.io/SSSD/sssd/issue/3937

Reviewed-by: Sumit Bose <sbose@redhat.com>
Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/938dd6c1a7ba3bc89e2f298365fbf378b82d6649">938dd6c1</a></strong>
<div>
<span>by mateusz</span>
<i>at 2019-02-16T22:25:12Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">Added note about default value of ad_gpo_map_batch parameter

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/85e363086fb7fd859c9b2eda408ec83addd79008">85e36308</a></strong>
<div>
<span>by Thorsten Scherf</span>
<i>at 2019-02-20T20:46:09Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">CONFIG: add missing ldap attributes for validation

https://pagure.io/SSSD/sssd/issue/3961

This patch adds missing 'ldap_host_*' attributes for config validation.

Reviewed-by: Alexey Tikhonov <atikhono@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/bc65ba9a07a924a58b13a0d5a935114ab72b7524">bc65ba9a</a></strong>
<div>
<span>by Victor Tapia</span>
<i>at 2019-02-26T21:45:37Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">GPO: Allow customization of GPO_CROND per OS

For both Debian and Ubuntu, the cron PAM service is named "cron" instead
of "crond", denying the use of the service by default.

This patch enables the HAVE_$OS (HAVE_DEBIAN/HAVE_FEDORA...) macros to select
the service name during build, allowing further customization if required.

Resolves:
https://bugs.launchpad.net/ubuntu/+source/sssd/+bug/1572908

Reviewed-by: Michal Židek <mzidek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/c01364341f22262ebc2012b102be872ec3f8bdf4">c0136434</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2019-02-26T21:47:53Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">responder: fix domain lookup refresh timeout

Currently the timeout to check for new domains is reset whenever there
is a request with an unknown domain name. If those requests happen more
then once a minute (the default timeout) the timeout is always reset
before it can expire and as a result the domain list is never refreshed.

If SSSD starts offline with an empty domain list the NSS responder might
never be able to resolve sub-domain users or groups even if the backend
has switched to the online state and already refreshed the domain list.

Related to https://pagure.io/SSSD/sssd/issue/3967

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/eaece8b2e360a9433573e9cdd09f2ab73995c55a">eaece8b2</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2019-02-26T21:52:30Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">ldap: add get_ldap_conn_from_sdom_pvt

Related:
https://pagure.io/SSSD/sssd/issue/2474

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/b2352a01f86f7c3d6fa4e405069383df9b1b84cb">b2352a01</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2019-02-26T21:52:35Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">ldap: prefer LDAP port during initgroups user lookup

The first step of an initgroups request is to lookup the user. When
using the AD provider the Global Catalog will be the preferred source.
But not all LDAP attributes of the user might be replicated to the
Global Catalog and as a result some of the missing attributes might be
removed from the cached user object.

Related to https://pagure.io/SSSD/sssd/issue/2474

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/3cb9a3db92247e02087f1dcd2af54fd083738646">3cb9a3db</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2019-02-26T21:52:36Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">ldap: user get_ldap_conn_from_sdom_pvt() where possible

Currently when trying to prefer LDAP port lookups over Global Catalog
searches some AD specific structs are needed in the common LDAP
provider code. By using get_ldap_conn_from_sdom_pvt() this can be
avoided and all AD specific details are handled inside
sdap_async_initgroups_ad.c now.

Related to https://pagure.io/SSSD/sssd/issue/2474

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/05350abdf2ab98770ca296b9485578218644a2a7">05350abd</a></strong>
<div>
<span>by Sumit Bose</span>
<i>at 2019-02-26T21:58:38Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">krb5_locator: always use port 88 for master KDC

If the kpasswdinfo file exists and the found IP address includes a port
number as well the master KDC lookup will use this port number which is
most probably wrong. Better use the default port 88 always for master
KDC lookups.

This patch also updates the man page for the locator plugin which was
quite outdated.

Related to https://pagure.io/SSSD/sssd/issue/3958

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/9ad7173ee23ee99b0806f2d20999fdd315c20c6b">9ad7173e</a></strong>
<div>
<span>by Alexey Tikhonov</span>
<i>at 2019-02-26T22:07:42Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">TESTS: fixed bug in guests startup function

`start-guest()` function always started `client` machine ignoring
supplied argument.

Reviewed-by: Pavel Březina <pbrezina@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/713e4f929326f106e8b5338a6d47b543a45f9d5b">713e4f92</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2019-02-26T22:09:12Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">TESTS: Don't fail when trying to create an OU that already exists

If a fixture that creates an OU was used twice, the second invocation
would fail, because the OU already exists. This patch ignores the
already exists error when adding an OU.

Reviewed-by: Michal Židek <mzidek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/686a8f5f1de8447aedd3c65715ff40118bb9908f">686a8f5f</a></strong>
<div>
<span>by Lukas Slebodnik</span>
<i>at 2019-02-26T22:16:50Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">test_files_provider: Do not use pytest fixtures as functions

test_files_provider.py::test_getpwnam_after_start
  src/tests/intg/test_files_provider.py:344: RemovedInPytest4Warning:
    Fixture "passwd_ops_setup" called directly. Fixtures are not meant
    to be called directly, are created automatically when test functions
    request them as parameters.
    See https://docs.pytest.org/en/latest/fixture.html for more information.

  src/tests/intg/test_files_provider.py:362: RemovedInPytest4Warning:
    Fixture "group_ops_setup" called directly. Fixtures are not meant
    to be called directly, are created automatically when test functions
    request them as parameters.
    See https://docs.pytest.org/en/latest/fixture.html for more information.
      return setup_gr_with_list(request, [GROUP1, CANARY_GR])

Resolves:
https://pagure.io/SSSD/sssd/issue/3942

Merges: https://pagure.io/SSSD/sssd/pull-request/3953

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/948cd08cd91cda79f6afa96f99241f26d84bf1bc">948cd08c</a></strong>
<div>
<span>by Lukas Slebodnik</span>
<i>at 2019-02-26T22:16:55Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">test_ldap: Do not uses pytest fixtures as functions

test_ldap.py::test_local_negative_timeout_enabled_by_default
  /usr/lib/python3.7/site-packages/_pytest/fixtures.py:827:
    RemovedInPytest4Warning: Fixture "passwd_ops_setup" called directly.
    Fixtures are not meant to be called directly, are created
    automatically when test functions request them as parameters. See
    https://docs.pytest.org/en/latest/fixture.html for more information.
    res = fixturefunc(**kwargs)
  /usr/lib/python3.7/site-packages/_pytest/fixtures.py:827:
    RemovedInPytest4Warning: Fixture "group_ops_setup" called directly.
    Fixtures are not meant to be called directly, are created
    automatically when test functions request them as parameters. See
    https://docs.pytest.org/en/latest/fixture.html for more information.
    res = fixturefunc(**kwargs)
  /usr/lib/python3.7/site-packages/_pytest/fixtures.py:827:
    RemovedInPytest4Warning: Fixture "user_and_group_rfc2307" called directly.
    Fixtures are not meant to be called directly, are created
    automatically when test functions request them as parameters. See
    https://docs.pytest.org/en/latest/fixture.html for more information.

Resolves:
https://pagure.io/SSSD/sssd/issue/3942

Merges: https://pagure.io/SSSD/sssd/pull-request/3953

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/5773463361bacd69c816ddc46a4fc069db7bd0f7">57734633</a></strong>
<div>
<span>by Lukas Slebodnik</span>
<i>at 2019-02-26T22:22:17Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">Revert "intg: Generate tmp dir with lowercase"

This reverts commit 929bb1170931d7aafac4dc0572ce18747c919a56.

It is already fixed in upstream.
Combination of python-urllib3-1.24 and python-requests-2.19
works well.

Resolves:
https://pagure.io/SSSD/sssd/issue/3276

Merges: https://pagure.io/SSSD/sssd/pull-request/3952

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/54d7175d018792fc676201bd1b9772b268f14922">54d7175d</a></strong>
<div>
<span>by Lukas Slebodnik</span>
<i>at 2019-02-26T22:25:10Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">ent_test: Update assertions for python 3.7.2

pwd and grp modules return different string in KeyError
since python 3.7.2

  sh-4.4$ python3 --version
  Python 3.7.1
  sh-4.4$ python3 -c 'import grp; grp.getgrnam("non-exist");'
  Traceback (most recent call last):
    File "<string>", line 1, in <module>
  KeyError: 'getgrnam(): name not found: non-exist'

  sh-4.4$ python3 --version
  Python 3.7.2
  sh-4.4$ python3 -c 'import grp; grp.getgrnam("non-exist");'
  Traceback (most recent call last):
    File "<string>", line 1, in <module>
  KeyError: "getgrnam(): name not found: 'non-exist'"

Merges: https://pagure.io/SSSD/sssd/pull-request/3951

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/ffd7536dfa402a6d0dec2fb0bb3e3a221f5f9aab">ffd7536d</a></strong>
<div>
<span>by Pavel Březina</span>
<i>at 2019-02-27T18:44:58Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sbus: terminated active ongoing request when reconnecting

Connection to the remote dbus server was lost. If there are any outgoing
requests they are waiting for a pretty long timeout. During this timeout
we kept chaining even new requests that come after successful reconnection
and these request were waiting for the timeout to ocurr as well because
they were chain to request that started before reconnection.

Now, we terminated all active outgoing request that have a key associated
so we can immediately start sending new requests.

Resolves:
https://pagure.io/SSSD/sssd/issue/3907

Reviewed-by: Alexey Tikhonov <atikhono@redhat.com>
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/6a8e906ecedb9c0b86bd5ea08b63d3484def6b52">6a8e906e</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2019-02-27T19:04:20Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">Updating translations for the 2.1 release
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/3aee2b3a83e3a7c0fda11927fd84424e4d473fa4">3aee2b3a</a></strong>
<div>
<span>by Jakub Hrozek</span>
<i>at 2019-02-27T19:11:12Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">Updating the version for the 2.1.0 release
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/41820e4fe3ce6e0913db1b4e9701d5a615cfe1b2">41820e4f</a></strong>
<div>
<span>by Timo Aaltonen</span>
<i>at 2019-05-27T09:04:22Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">Merge tag 'sssd-1_16_4' into exp

Tagging the 1.16.4 release
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/33090ef079ceea88c9bc6db0bc460304010b55e5">33090ef0</a></strong>
<div>
<span>by Timo Aaltonen</span>
<i>at 2019-05-27T09:04:35Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">Merge branch 'experimental' into exp
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/0540daf449923fa119e145f49926fd067d7ecb3d">0540daf4</a></strong>
<div>
<span>by Timo Aaltonen</span>
<i>at 2019-05-27T09:05:46Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">bump the version
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/8622e22df4d5011647cc1609771f36c52282ffe2">8622e22d</a></strong>
<div>
<span>by Timo Aaltonen</span>
<i>at 2019-05-27T09:55:49Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">fix changelog whitespace
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/488c638db2f50f2418fcc0cad1279817c05cb163">488c638d</a></strong>
<div>
<span>by Timo Aaltonen</span>
<i>at 2019-05-27T09:59:43Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sssd-tools.install: Local domain support is deprecated and not built by default anymore, so drop the files.
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/c0156c40831757a3984804fed532e22e4763442d">c0156c40</a></strong>
<div>
<span>by Timo Aaltonen</span>
<i>at 2019-05-27T10:09:56Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">control, sssd-common.install: Secrets responder is dropped, deprecated.
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/63551dbf831343b9a9be120cec6d2a0aac954ebd">63551dbf</a></strong>
<div>
<span>by Timo Aaltonen</span>
<i>at 2019-05-27T10:10:26Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">control: Add ldap-utils to build-depends, tests need it.
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/78d546320441cb87aa9ad35fd5d7411b4619e371">78d54632</a></strong>
<div>
<span>by Timo Aaltonen</span>
<i>at 2019-05-27T10:34:56Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">sssd-common.install: Add new internal libs for iface/sbus.
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/32bd57736bdb0310dcb468e9cf85bd813a996e23">32bd5773</a></strong>
<div>
<span>by Timo Aaltonen</span>
<i>at 2019-05-27T10:39:25Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">fix-whitespace-test.diff: Fix ignoring the debian dir.
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/fc016d5866889dd66788ca830b3b8887859d1871">fc016d58</a></strong>
<div>
<span>by Timo Aaltonen</span>
<i>at 2019-05-27T10:55:37Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">rules: Update the clean target.
</pre>
</li>
<li>
<strong><a href="https://salsa.debian.org/sssd-team/sssd/commit/74e0e0d4142b7fc7645c0f7a4bb32268a43db689">74e0e0d4</a></strong>
<div>
<span>by Timo Aaltonen</span>
<i>at 2019-05-27T10:55:44Z</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap; margin: 0;">releasing package sssd version 2.1.0-1
</pre>
</li>
</ul>
<h4>18 changed files:</h4>
<ul>
<li class="file-stats">
<a href="#0c2c5c506cef0ed2d3ad2b7e45e97980b66f5520">
<span class="new-file">
+
Jenkinsfile
</span>
</a>
</li>
<li class="file-stats">
<a href="#d5b4de16d947214ec306bd57bed1bd23a939b5f9">
Makefile.am
</a>
</li>
<li class="file-stats">
<a href="#87db583be5c13c1f7b3c958b10e03d67b6a2ca06">
configure.ac
</a>
</li>
<li class="file-stats">
<a href="#49d80aa598751b3e8c23a3bbb4e7e9c03aa770b6">
contrib/ci/deps.sh
</a>
</li>
<li class="file-stats">
<a href="#d348d65f630a357f2aeaa78fc64043f57caa4cb0">
contrib/ci/sssd.supp
</a>
</li>
<li class="file-stats">
<a href="#b8d57aa4a09effcbac8deeffe8aea9131499424f">
contrib/sssd.spec.in
</a>
</li>
<li class="file-stats">
<a href="#3d2f764ef24a4af4d2f718f6385375e433c0eeb7">
<span class="new-file">
+
contrib/test-suite/README.md
</span>
</a>
</li>
<li class="file-stats">
<a href="#828a4acdc5cebd16badf8e6b765e91b963ef8b24">
<span class="new-file">
+
contrib/test-suite/run-client.sh
</span>
</a>
</li>
<li class="file-stats">
<a href="#cf3808699e58bfbc32dc1065a7eccca70ea9039d">
<span class="new-file">
+
contrib/test-suite/run.sh
</span>
</a>
</li>
<li class="file-stats">
<a href="#9c96da0e9f91d7d8937b69b524702c106258f0d1">
debian/changelog
</a>
</li>
<li class="file-stats">
<a href="#58ef006ab62b83b4bec5d81fe5b32c3b4c2d1cc2">
debian/control
</a>
</li>
<li class="file-stats">
<a href="#71e9a10ddb48495f37869d68ebcb95ed481cc26f">
<span class="new-file">
+
debian/patches/fix-whitespace-test.diff
</span>
</a>
</li>
<li class="file-stats">
<a href="#bc34014ab4b9a49dd7a27bdd8d352912607c3a96">
debian/patches/series
</a>
</li>
<li class="file-stats">
<a href="#8756c63497c8dc39f7773438edf53b220c773f67">
debian/rules
</a>
</li>
<li class="file-stats">
<a href="#888968d8696ab3370bf567424c1013cb2ad00c45">
debian/sssd-common.install
</a>
</li>
<li class="file-stats">
<a href="#0514a851978ed12f08de653a802230e7d9f7c47a">
debian/sssd-tools.install
</a>
</li>
<li class="file-stats">
<a href="#1f35254eafd260ef7262c1165537916497a1d165">
po/POTFILES.in
</a>
</li>
<li class="file-stats">
<a href="#4e573a66c66b45b45a1e180cad791738ed22cdd2">
po/bg.po
</a>
</li>
</ul>
<h5>The diff was not included because it is too large.</h5>

</div>
<div class="footer" style="margin-top: 10px;">
<p style="font-size: small; color: #777;">

<br>
<a href="https://salsa.debian.org/sssd-team/sssd/compare/0f5034094c829aa0e66374e20c774d32336c8293...74e0e0d4142b7fc7645c0f7a4bb32268a43db689">View it on GitLab</a>.
<br>
You're receiving this email because of your account on salsa.debian.org.
If you'd like to receive fewer emails, you can
adjust your notification settings.


</p>
</div>
</body>
</html>