group.of.nepali.translators team mailing list archive
-
group.of.nepali.translators team
-
Mailing list archive
-
Message #35382
[Bug 1874526] Re: [landscape] Substitute oidc conf in service file
This bug was fixed in the package sosreport - 3.9-1ubuntu0.19.10.3
---------------
sosreport (3.9-1ubuntu0.19.10.3) eoan; urgency=medium
* d/p/0003-lshw-command.patch: (LP: #1871494)
- Add `lshw` command to hardware plugin
[Eric Desrochers]
* d/p/0004-lds-substitute-oidc-conf.patch:
- landscape substitute oidc conf
in service file (LP: #1874526)
[Eric Desrochers]
* d/tests/{control,simple.sh}: (LP: #1865212)
- Add testsuite "simple.sh". A port of the travis tests to bash
provided by upstream.
-- Heather Lemon <heather.lemon@xxxxxxxxxxxxx> Wed, 29 Apr 2020
13:47:47 +0000
** Changed in: sosreport (Ubuntu Eoan)
Status: Fix Committed => Fix Released
** Changed in: sosreport (Ubuntu Bionic)
Status: Fix Committed => Fix Released
--
You received this bug notification because you are a member of नेपाली
भाषा समायोजकहरुको समूह, which is subscribed to Xenial.
Matching subscriptions: Ubuntu 16.04 Bugs
https://bugs.launchpad.net/bugs/1874526
Title:
[landscape] Substitute oidc conf in service file
Status in sosreport package in Ubuntu:
Fix Released
Status in sosreport source package in Xenial:
Won't Fix
Status in sosreport source package in Bionic:
Fix Released
Status in sosreport source package in Eoan:
Fix Released
Status in sosreport source package in Focal:
Fix Released
Status in sosreport source package in Groovy:
Fix Released
Bug description:
[Impact]
Landscape has added the ability to connect to OIDC.
The plugin should be updated to obfuscate the sensitive information.
https://docs.ubuntu.com/landscape/en/onprem-auth#openid-connect-
support
[Test Case]
* Install sosreport
* Install landscape-client and/or landscape-server (to make sure sosreport's landscape plugin will be triggered) from the Landscape PPA -> https://launchpad.net/~landscape
* Manually append or create files: "/etc/landscape/service.conf" & "/etc/landscape/service.conf.old" (No need to have a fully functionnal landscape setup, just the package installed (for triggering purposes) and then you can create and add the parameter by hand)
* Add the following in both "/etc/landscape/service.conf" & "/etc/landscape/service.conf.old":
oidc-client-secret = secret-test
oidc-client-id = id-test
* Execute sosreport "sosreport -a"
* Make sure landscape plugin was exercise.
* Extract archive and make sure both "oidc-client-id" & "oidc-client-secret" are subsituted in files "/etc/landscape/service.conf" & "/etc/landscape/service.conf.old" as it should (if present).
Expected result (path_to_sosreport/etc/landscape/service.conf*)
oidc-client-secret = [********]
oidc-client-id = [********]
Extra testing (sanity check):
* Look under "sos_reports" for full report.
* Look under "sos_logs" for warnings/errors.
$ grep -v "INFO:" sos_logs/sos.log
* Run "simple.sh": A quick port of the travis tests to bash. Generating various type of sosreports collection.
https://raw.githubusercontent.com/sosreport/sos/master
[Regression]
No regression expected, we don't change/impact core functionnalities
nor affect other plugins. If something happens it will be isolate to
the landscape plugin itself only.
Worse case the OID substitution won't work as expected (corner case)
and will reveal OID sensible information, but it is very unlikely to
happen as it will be intensively tested during the testing phase, and
the substitute mechanism in place has been proven to work for the same
configuration files in the landscape plugin already.
[Other Informations]
Upstream bug:
https://github.com/sosreport/sos/issues/2023
Upstream PR:
https://github.com/sosreport/sos/pull/2025
Upstream commit:
https://github.com/sosreport/sos/pull/2025/commits/0c4d821e26e1206a0b99f427b572931ba2fd9bb5
To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/sosreport/+bug/1874526/+subscriptions