[Pkg-sssd-devel] [Git][sssd-team/sssd][master] 29 commits: CI: Build srpm fix for illegal version tag '-'

Wed Nov 23 09:13:17 GMT 2022

Timo Aaltonen pushed to branch master at Debian SSSD packaging / sssd

Commits:
587fe968 by Justin Stephenson at 2022-10-12T14:22:20+02:00
CI: Build srpm fix for illegal version tag '-'

error: line 45: Illegal char '-' (0x2d) in: Version: sssd-2-7

Reviewed-by: Iker Pedrosa <ipedrosa at redhat.com>
Reviewed-by: Pavel Březina <pbrezina at redhat.com>
(cherry picked from commit ab89455be398f4688f95489b805c8ae204ba6f0d)

- - - - -
e6f94e25 by Alexey Tikhonov at 2022-10-13T21:00:16+02:00
IPA: "trusted user not found" isn't an error

Reviewed-by: Pavel Březina <pbrezina at redhat.com>
Reviewed-by: Sumit Bose <sbose at redhat.com>
(cherry picked from commit a5403f7896a685d09f6449915d5216b6b1224bcb)

- - - - -
a6d52145 by Pavel Březina at 2022-10-13T21:00:44+02:00
confdb: avoid syslog message when no domains are enabled

This syslog message would also appear when calling other tools like
sss_cache which is confusing. We return specific error code instead
and let the error be syslogged in the monitor in monitor.c:main (this
is already implemented).

Resolves: https://github.com/SSSD/sssd/issues/6387

:fixes: A regression when running sss_cache when no SSSD domain is
  enabled would produce a syslog critical message was fixed.

Reviewed-by: Alejandro López <allopez at redhat.com>
Reviewed-by: Sumit Bose <sbose at redhat.com>
(cherry picked from commit b38fdc8185fcd6a2e5d4b483d3119964f9922070)

- - - - -
1df02947 by Pavel Březina at 2022-10-13T21:00:44+02:00
monitor: read all enabled domains in add_implicit_services

Reading sssd/domains option is no longer sufficient since domains
can be enabled through domain/enabled.

Reviewed-by: Alejandro López <allopez at redhat.com>
Reviewed-by: Sumit Bose <sbose at redhat.com>
(cherry picked from commit 4da861368b88c03b22993f95de5b508ad5637c25)

- - - - -
a6312c46 by Pavel Březina at 2022-10-13T21:00:44+02:00
sss_cache: use ERR_NO_DOMAIN_ENABLED instead of ENOENT

Reviewed-by: Alejandro López <allopez at redhat.com>
Reviewed-by: Sumit Bose <sbose at redhat.com>
(cherry picked from commit 64c22dd1c4f79f953b879fc167b535de928f4bfd)

- - - - -
ab7cbb5f by Pavel Březina at 2022-10-13T21:00:44+02:00
confdb: chande debug level when no domain are found in confdb_get_domains

We print the error as fatal error in the monitor to make sure the
message is correctly visible. However, the error is not fatal for tools
like sss_cache and it should not be printed there by default.

Since the tools have default debug level set to SSSDBG_FATAL_FAILURE, it
is sufficient to just drop the level to critical.

Resolves: https://github.com/SSSD/sssd/issues/6387

Reviewed-by: Alejandro López <allopez at redhat.com>
Reviewed-by: Sumit Bose <sbose at redhat.com>
(cherry picked from commit df55b1f16ea2fb5e56f0fe69419904d50b7e2476)

- - - - -
62276737 by aborah-sudo at 2022-10-20T11:04:12+02:00
Tests: port proxy_provider/rfc2307

https://gitlab.cee.redhat.com/sssd/sssd-qe/-/tree/RHEL8.6/client/proxy_provider/rfc2307

Reviewed-by: Steeve Goveas <sgoveas at redhat.com>
Reviewed-by: Sumit Bose <sbose at redhat.com>
(cherry picked from commit 6c0ae779fca1450d3256e9d718164dfb75880cdf)

- - - - -
cfaa06b0 by Alexey Tikhonov at 2022-11-01T17:15:51+01:00
CFG RULES: allow 'fallback_to_nss' option

Reviewed-by: Alejandro López <allopez at redhat.com>
Reviewed-by: Iker Pedrosa <ipedrosa at redhat.com>
(cherry picked from commit 8a2fd06b2928189bcf0c3373c9e38437063b5241)

- - - - -
e0026e38 by Alexey Tikhonov at 2022-11-01T17:19:14+01:00
SYSDB: pre-existence of MPG group in the cache isn't an error

Reviewed-by: Justin Stephenson <jstephen at redhat.com>
Reviewed-by: Sumit Bose <sbose at redhat.com>
(cherry picked from commit 71466a8dbdb1d755ace15680cc2b4b11b68a0573)

- - - - -
7bf97190 by Pavel Březina at 2022-11-01T17:37:24+01:00
ci: enable ci for sssd-2-8 branch

Reviewed-by: Iker Pedrosa <ipedrosa at redhat.com>
Reviewed-by: Justin Stephenson <jstephen at redhat.com>
(cherry picked from commit 62458d49090ae94b1d65ac0642ec6e3b3cd75a05)

- - - - -
40ccfd6c by Pavel Březina at 2022-11-01T17:37:24+01:00
ci: switch to actions/checkout at v3

checkout at v3 runs on node16 so it will remove deprecation warning:
https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/

Reviewed-by: Iker Pedrosa <ipedrosa at redhat.com>
Reviewed-by: Justin Stephenson <jstephen at redhat.com>
(cherry picked from commit c526acbaea0b1cd1e0a554820f860e2e20e2ae6d)

- - - - -
9aee8164 by Pavel Březina at 2022-11-01T17:37:24+01:00
ci: use GITHUB_OUTPUT instead of set-output

https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

Reviewed-by: Iker Pedrosa <ipedrosa at redhat.com>
Reviewed-by: Justin Stephenson <jstephen at redhat.com>
(cherry picked from commit 770bf7bf3afada064df8ed9d47445d912d5f0a4c)

- - - - -
a7853d51 by Pavel Březina at 2022-11-01T17:37:24+01:00
ci: switch to actions/upload-artifact at v3

upload-artifact at v3 runs on node16 so it will remove deprecation warning:
https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/

Reviewed-by: Iker Pedrosa <ipedrosa at redhat.com>
Reviewed-by: Justin Stephenson <jstephen at redhat.com>
(cherry picked from commit b2d193b9370eb7d2bd9ffb713e58cbd936f411b7)

- - - - -
6d7f1fba by Alejandro López at 2022-11-02T15:58:41+01:00
CACHE_REQ: Do not use timestamp optimization on "files" provider.

Avoid requesting only the latest updates when using the "files"
provider as it only updates the cache if /etc/files or /etc/group
is touched.

Added a test for this situation.

Resolves: https://github.com/SSSD/sssd/issues/6398

Reviewed-by: Alexey Tikhonov <atikhono at redhat.com>
Reviewed-by: Justin Stephenson <jstephen at redhat.com>
(cherry picked from commit 2070000a3b80e2dc13bd19b3da99efaa08c34fac)

- - - - -
c61e1098 by Justin Stephenson at 2022-11-03T11:08:27+01:00
Analyzer: Optimize list verbose output

Modify the analyzer to parse the responder log file in one pass. This
avoids repeated parsing of a single log file. This operation will now
store log lines in a dictionary on a single pass then format and print
the output accordingly. Does not affect 'list' or 'show' output.

Reviewed-by: Alexey Tikhonov <atikhono at redhat.com>
Reviewed-by: Tomáš Halman <thalman at redhat.com>
(cherry picked from commit 4c678cbb4ace4be413847fa297bce0c7c8cc42b6)

- - - - -
129979a3 by Alejandro López at 2022-11-03T11:15:43+01:00
Cache: String has to be duplicated instead of copied

input->name.attr has to be duplicated when not NULL.
Just copying the pointer is not a good thing.

Reviewed-by: Alexey Tikhonov <atikhono at redhat.com>
Reviewed-by: Iker Pedrosa <ipedrosa at redhat.com>
(cherry picked from commit 214540e72a6743e1112b35d562efc595594b1a82)

- - - - -
1377bca9 by Alejandro López at 2022-11-03T11:15:43+01:00
CACHE_REQ: Initialize domain with NULL

Variable domain is compared to NULL before checking sss_parse_inp_recv()'s
return value. Let's initialize it to NULL.

Warning: sss_parse_inp_recv() modifies domain's value when returning EOK
or ERR_DOMAIN_NOT_FOUND. The situation is not perfect but is better than
before.

Reviewed-by: Alexey Tikhonov <atikhono at redhat.com>
Reviewed-by: Iker Pedrosa <ipedrosa at redhat.com>
(cherry picked from commit 719c92533c52aad091c8b58e7a82b9e3b88f8e62)

- - - - -
1562df03 by Alejandro López at 2022-11-03T11:15:43+01:00
CACHE_REQ: Do not return duplicated values.

For each domain, a multi-domain search is done and all the results for
each domain are returned each time. This produces several copies of the
same results.
Function cache_req_input_parsed() decides whether to launch a multi- or
single-domain search based on the domain the user requested. If the user
didn’t request a domain (such as in a call to ListByName()), a
multi-domain search is launched, even if we are only looking for the
current domain. In that case a single-domain search is enough.

Resolves: https://github.com/SSSD/sssd/issues/6360

Reviewed-by: Alexey Tikhonov <atikhono at redhat.com>
Reviewed-by: Iker Pedrosa <ipedrosa at redhat.com>
(cherry picked from commit 8f7c35eae22aaa9649db1cf2d07d17d8c4b92e82)

- - - - -
82c69b7f by Alejandro López at 2022-11-03T11:15:43+01:00
TESTS: Correct ListByAttr()'s test

Now that ListByName() and ListBy Attr are fixed, the test needs to
be corrected as it was explicitly avoiding that case.

Reviewed-by: Alexey Tikhonov <atikhono at redhat.com>
Reviewed-by: Iker Pedrosa <ipedrosa at redhat.com>
(cherry picked from commit a97962826efdf5c51fed7553e761a803c48f6675)

- - - - -
13e841de by Alejandro López at 2022-11-03T11:15:43+01:00
CACHE_REQ: Consider the domain when looking names in the cache

Using a fully-qualified name based on the expresion the user provided,
when looking for names in the cache.

Resolves: https://github.com/SSSD/sssd/issues/6361

Reviewed-by: Alexey Tikhonov <atikhono at redhat.com>
Reviewed-by: Iker Pedrosa <ipedrosa at redhat.com>
(cherry picked from commit 569dca5f24e2e92aa86c7a2800f772e29211fc31)

- - - - -
7f583fad by Alejandro López at 2022-11-03T11:15:43+01:00
TESTS: New test for D-Bus' ListByName()

Added a test for the D-Bus function ListByName().

Reviewed-by: Alexey Tikhonov <atikhono at redhat.com>
Reviewed-by: Iker Pedrosa <ipedrosa at redhat.com>
(cherry picked from commit 09895561a34592de1d58af211db1725388d86d5e)

- - - - -
0f670188 by Alejandro López at 2022-11-03T11:15:43+01:00
CACHE_REQ: Use a const struct in cache_req_data_create()

The structure data is not modified in the function. The const modifier
can be used.

Reviewed-by: Alexey Tikhonov <atikhono at redhat.com>
Reviewed-by: Iker Pedrosa <ipedrosa at redhat.com>
(cherry picked from commit c3453e4e82796c5ae578671f25568c74919cad13)

- - - - -
cfa11294 by 김인수 at 2022-11-04T11:29:21+01:00
po: update translations

(Korean) currently translated at 100.0% (663 of 663 strings)
Translation: SSSD/SSSD-2-8
Translate-URL: https://translate.fedoraproject.org/projects/sssd/sssd-2-8/ko/

po: update translations

(Polish) currently translated at 100.0% (663 of 663 strings)
Translation: SSSD/SSSD-2-8
Translate-URL: https://translate.fedoraproject.org/projects/sssd/sssd-2-8/pl/

po: update translations

(Russian) currently translated at 100.0% (663 of 663 strings)
Translation: SSSD/SSSD-2-8
Translate-URL: https://translate.fedoraproject.org/projects/sssd/sssd-2-8/ru/

po: update translations

(Ukrainian) currently translated at 100.0% (663 of 663 strings)
Translation: SSSD/SSSD-2-8
Translate-URL: https://translate.fedoraproject.org/projects/sssd/sssd-2-8/uk/

po: update translations

(Polish) currently translated at 100.0% (663 of 663 strings)
Translation: SSSD/SSSD-2-8
Translate-URL: https://translate.fedoraproject.org/projects/sssd/sssd-2-8/pl/

po: update translations

(Korean) currently translated at 100.0% (663 of 663 strings)
Translation: SSSD/SSSD-2-8
Translate-URL: https://translate.fedoraproject.org/projects/sssd/sssd-2-8/ko/

- - - - -
59685cff by Shridhar Gadekar at 2022-11-04T11:29:24+01:00
Tests: GSSAPI ssh login failing due to a missing directive

Adding the missing pxssh module import

Reviewed-by: Iker Pedrosa <ipedrosa at redhat.com>
Reviewed-by: Jakub Vávra <jvavra at redhat.com>
(cherry picked from commit 90f860b277074df1861ad4110df35eee7c83862d)

- - - - -
59973c0b by Pavel Březina at 2022-11-04T11:37:10+01:00
pot: update translations

- - - - -
a18ef88e by Pavel Březina at 2022-11-04T12:02:30+01:00
Release sssd-2.8.1

- - - - -
d690c2c6 by Timo Aaltonen at 2022-11-23T10:07:24+02:00
Merge branch 'upstream'

- - - - -
1fa6693e by Timo Aaltonen at 2022-11-23T10:07:57+02:00
version bump

- - - - -
53508df4 by Timo Aaltonen at 2022-11-23T10:10:46+02:00
releasing package sssd version 2.8.1-1

- - - - -

30 changed files:

- .github/actions/build-sssd-srpm/action.yml
- .github/workflows/analyze-target.yml
- .github/workflows/ci.yml
- .github/workflows/copr_build.yml
- .github/workflows/copr_cleanup.yml
- .github/workflows/coverity.yml
- .github/workflows/static-code-analysis.yml
- contrib/ci/get-matrix.py
- debian/changelog
- po/bg.po
- po/ca.po
- po/cs.po
- po/de.po
- po/es.po
- po/eu.po
- po/fi.po
- po/fr.po
- po/hu.po
- po/id.po
- po/it.po
- po/ja.po
- po/ka.po
- po/ko.po
- po/nb.po
- po/nl.po
- po/pl.po
- po/pt.po
- po/pt_BR.po
- po/ru.po
- po/sssd.pot

The diff was not included because it is too large.

View it on GitLab: https://salsa.debian.org/sssd-team/sssd/-/compare/82285406c7d9908080277f51086a47925ce7359c...53508df49e30afb44d0b233e093d3ff8bbbcdd03

-- 
View it on GitLab: https://salsa.debian.org/sssd-team/sssd/-/compare/82285406c7d9908080277f51086a47925ce7359c...53508df49e30afb44d0b233e093d3ff8bbbcdd03
You're receiving this email because of your account on salsa.debian.org.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://alioth-lists.debian.net/pipermail/pkg-sssd-devel/attachments/20221123/5dd9c825/attachment-0001.htm>