Univention Bugzilla – Bug 43695
Add links to root certificate and the revocation list into the side menu
Last modified: 2017-04-04 18:29:45 CEST
In UCS 4.1, there are two entries on the UCS startsite for the root certificate and the revocation list. These links should be best viewed in the side menu in an additional submenu "Certificates". These menu entries need only to be visible on the master/backup system.
Alternatively they could also be shown as portal entries. But your suggestion might be better.
If we implement comment #0 we have to remove the old entries from the ucs-overview.
(In reply to Florian Best from comment #2) > If we implement comment #0 we have to remove the old entries from the > ucs-overview. This is already done by blacklisting these keys in the UCR module
(In reply to Dirk Wiesenthal from comment #3) > (In reply to Florian Best from comment #2) > > If we implement comment #0 we have to remove the old entries from the > > ucs-overview. > > This is already done by blacklisting these keys in the UCR module I don't think blacklisting is a good idea because we need to keep this code forever then. Unsetting is better.
The menu entries need to link to the master (via tools.status('ldap_master')). As this will only work in scenarios where the DNS can be resolved, please make sure that on the roles domaincontroller_master and domaincontroller_backup (via tools.status('server_role')) the link is relative. The menu entries can be added to management/univention-web/js/hooks/default_menu_entries.js. The previous UCR variables entries can simply be removed from services/univention-apache/08univention-apache.inst and in the postinst they can be unset as Florian said. (In reply to Florian Best from comment #4) > (In reply to Dirk Wiesenthal from comment #3) > > (In reply to Florian Best from comment #2) > > > If we implement comment #0 we have to remove the old entries from the > > > ucs-overview. > > > > This is already done by blacklisting these keys in the UCR module > I don't think blacklisting is a good idea because we need to keep this code > forever then. Unsetting is better. I would vote for both, i.e., blacklisting + unsetting.
The ucr entries will get unset now: svn commit: 77579 + 77625 build: 9.0.3-2A~4.2.0.201703131135 The certificates-menu has been added: svn commit: 77580 + 77629 build: 1.0.36-5A~4.2.0.201703131139
REOPEN: This changed the prior synchronous call of umc/hooks! to an asynchronous require(). The umcConfig.callback() expects that umc/hooks! is executed before the callback is executed which is not anymore the case.
The translation is missing.
(In reply to Florian Best from comment #8) > The translation is missing. Ah no, I confused en.po with de.po in your commit.
REOPEN: Please change into: if [ -n "$keys" ]; then ucr unset $keys; fi if you don't escape $keys in the first /bin/dash will fail with: sh: 9: [: ucs/web/overview/entries/admin/.../description: unexpected operator
I've made the changes suggested by Florian in Comment #10. Thanks for the hint. svn commit: 77735 build: 9.0.3-3A~4.2.0.201703151500
(In reply to Florian Best from comment #7) > REOPEN: This changed the prior synchronous call of umc/hooks! to an > asynchronous require(). The umcConfig.callback() expects that umc/hooks! is > executed before the callback is executed which is not anymore the case. We solved this differently by providing login.onLogin() / login.onLogout() / login.addLink().
OK: local/relative links on a DC Master/Backup OK: absolute links with FQDN to the DC Master on a DC Slave OK: Code-Review REOPEN: Changelog entry missing
The changelog entry has now been added. svn commit: 77893
OK: Changelog entry
UCS 4.2 has been released: https://docs.software-univention.de/release-notes-4.2-0-en.html https://docs.software-univention.de/release-notes-4.2-0-de.html If this error occurs again, please use "Clone This Bug".