Commits on Source (30)
-
Mark Reynolds authored
Description: Add replication functionality to UI. Cleaned up various UI interactions and page loading Added console logging for all CLI commands https://pagure.io/389-ds-base/issue/49926 Reviewed by: ?
-
Mark Reynolds authored
-
Simon Pichugin authored
Description: Make standard schema attributes and objectclasses non-editable. Attributes table should contain a string representation of the syntax attributes, and if you point a cursor on them you can see an ID. MAY and MUST attributes list shouldn't contain empty lines. Make WebUI form and CLI functionality consistent. Fix formatting and forloop lint issues. https://pagure.io/389-ds-base/issue/49928 Reviewed by: mreynolds, wibrown, mhonek (Thanks!)
-
Mark Reynolds authored
Description: Removed subpackage and cleaned up some compiler warnings related to ticket 49915 https://pagure.io/389-ds-base/issue/49979 Reviewed by: mreynolds
-
Mark Reynolds authored
Description: Fix single_space char pointer to actually point to a space https://pagure.io/389-ds-base/issue/49979
-
Amita Sharma authored
Description: We need to have the requirement token specified in the init.py file of each test suit in order to get the test cases mapped to these requirements in our test system. https://pagure.io/389-ds-base/issue/49929 Reviewed by: Viktor
-
Mark Reynolds authored
Description: Add a web browser console logging function to enforce a consistent format and hide password values for CLI commands. https://pagure.io/389-ds-base/issue/49978 Reviewed by: ?
-
William Brown authored
Lib389 uses a .replace rather than a path join, causing dirsrv to be replaced in the path. This breaks if the prefix is "/opt/dirsrv". This should use path join instead. https://pagure.io/389-ds-base/issue/49939 Author: William Brown <william@blackhats.net.au>
-
William Brown authored
The package maintainer at SUSE kindly pointed out a number of issues detected by rpmlint. Of the three issues, this resolves two of them. The first is calling setgroups with an empty group list before setgid so that we guarantee we only have the target user group remaining in our permission set when we drop privileges. The second is to remove a superfluous shebang line from python https://pagure.io/389-ds-base/issue/49975 Author: William Brown <william@blackhats.net.au>
-
Mark Reynolds authored
Description: Fixed some minor issues found in the JS. Also improved the replication enabling/disabling design/flow. https://pagure.io/389-ds-base/issue/49926 Reviewed by: ?
-
Mark Reynolds authored
Description: The server has not supported backup/restore of a single backend for a long time now. All references to it should be remove from the CLI and man pages. https://pagure.io/389-ds-base/issue/49856 Reviewed by: tbordaz & spichugi(Thanks!!)
-
Mark Reynolds authored
Description: dscreate was not setting the selinux labels on the ports, so if you specified a non-standard port the instance would not start. This fix sets/removes selinux labels during instance creation and removal Also moved ds_selinux_port_query & ds_selinux_enabled to the legacy tools package as they are only used by setup-ds.pl https://pagure.io/389-ds-base/issue/49814 Reviewed by: firstyear & mhonek (Thanks!!)
-
Mark Reynolds authored
Description: Skip labelling ports that use the the standard port numbers (389, 636). https://pagure.io/389-ds-base/issue/49814 Reviewed by: mreynolds(one line commit rule)
-
Mark Reynolds authored
Description: The first commit for this ticket introduced a compiler warning because it was missing the proper header file. https://pagure.io/389-ds-base/issue/49975 Reviewed by: mreynolds(one line commit rule)
-
Thierry Bordaz authored
Bug Description: During a MODRDN the DN cache is updated to replace source DN with the target DN (modrdn_rename_entry_update_indexes) If later a failure occurs (for example if BETXN_POSTOP fails) and the txn is aborted, the target DN (for the specific entryID) remains in the DN cache. If the entry is returned in a search, to build the DN there is a lookup of the DN cache with the entryID. It retrieves the target DN rather than the source DN Fix Description: In case of failure of the operation, the entry (from the entryID) need to be cleared from the DN cache https://pagure.io/389-ds-base/issue/49967 Reviewed by: Mark Reynolds Platforms tested: F27 Flag Day: no Doc impact: no
-
Thierry Bordaz authored
Bug Description: when adding 'memberof' to a member entry, the update may fail (invalid schema, db errors...). The error is reported at upper level. But in case of MODRDN the error is lost in memberof_moddn_attr_list where returned code of memberof_modop_one_replace_r is not tested Fix Description: Report a failure in memberof_moddn_attr_list as soon as memberof_modop_one_replace_r fails https://pagure.io/389-ds-base/issue/49985 Reviewed by: Simon Pichugi, William Brown Platforms tested: F27 Flag Day: no Doc impact: no
-
German Parente authored
Bug Description: seen at customer site, as first request to ldap database is the RUV, if the suffix provided in the command line does not exist or it's not replicated, we have an error message that it's regarding the RUV: ds-replcheck -D "cn=directory manager" -w secret12 -b "o=ipaca" -r ldap://ipamaster.germanparente.local:389 -m ldap://ipareplica.germanparente.local Performing online report... Connecting to servers... Gathering Master's RUV... Error: Failed to get Master RUV entry: {'desc': 'No such object'} Fix Description: add function to validate suffix exists and it's replicated https://pagure.io/389-ds-base/issue/49997 Author: German Parente <gparente@redhat.com> Review by: ???
-
Mark Reynolds authored
Bug Description: ----------------- At server startup the server's internal operatons performed by bootstrapping occurred before the thread data was initialized. This caused random values in the logging counters [1]. It was also observed that nested operations(and nested-nested operations, etc) were not properly logged [2]. Fix Description: ----------------- [1] Move the thread initialization higher up in main() [2] Changed the way we log nested internal operations. Instead, we keep the internal op number the same for nested ops but also now display the nested level. https://pagure.io/389-ds-base/issue/49995 Reviewed by: lkrispenz, amsharma, firstyear (Thanks!)
-
Simon Pichugin authored
Description: Refactor the existing 389-console Cockpit plugin structure so it uses React, Babel, Eslint. Add Makefile targets and refactor spec-file accordingly. Add python3-policycoreutils, npm and nodejs to BuildRequires. Put blank Plugins page which work in React and uses Cockpit API. Reviewed by: mreynolds, vashirov, mhonek (Thanks!) https://pagure.io/389-ds-base/issue/49999
-
Mark Reynolds authored
Description: Need to add missing dependency in specfile for python3-selinux https://pagure.io/389-ds-base/issue/49814 Reviewed by: mreynolds(one line commit rule)
-
Mark Reynolds authored
Description: Cleanup spec file https://pagure.io/389-ds-base/issue/49814 Reviewed by: mreynolds(one line commit rule)
-
Simon Pichugin authored
Description: Change Makefile so it is run only when there is no files cockpit_dist and node_modules. Add target for creating a tarball for Koji system https://pagure.io/389-ds-base/issue/49999 Reviewed by: mreynolds, mhonek (Thanks!)
-
Mark Reynolds authored
Description: Fix regression from previous commit
-
Thierry Bordaz authored
Bug Description: setgroups fails when not called by superuser. In case of prefix deployment DS is started by regular user Fix Description: Only superuser calls setgroups https://pagure.io/389-ds-base/issue/49975 Reviewed by: Mark Reynolds, Ludwig Krispenz and William Brown Platforms tested: F27 Flag Day: no Doc impact: no
-
German Parente authored
Bug Description: This is something we have very often in IPA context and customers are very often asking why there are errors in the logs: [31/Oct/2018:05:52:23.436616394 -0400] - ERR - NSACLPlugin - acl_parse - The ACL target cn=groups,cn=compat,dc=cgparente,dc=local does not exist [31/Oct/2018:05:52:23.438951763 -0400] - ERR - NSACLPlugin - acl_parse - The ACL target cn=computers,cn=compat,dc=cgparente,dc=local does not exist Fix Description: just log WARN instead of ERR https://pagure.io/389-ds-base/issue/50013 Author: German Parente <gparente@redhat.com> Review by: ???
-
Mark Reynolds authored
Bug Description: Schema parsing assumed X-ORIGIN was always in this format "X-ORIGIN '", but it can also be in other formats like: "X-ORIGIN (". So when it did not contain the original format we got list index errors. Fix Description: Loosen the format to " X-ORIGIN " which all the formats. Also: improve from UI schema error messages updated create_test for python3 https://pagure.io/389-ds-base/issue/50004 Reviewed by: firstyear & spichugi(Thanks!!)
-
Simon Pichugin authored
Description: Move plugins.es6 to index.es6 so we can properly navigate using old nav-bar. New React entry points should be added to the index.es6. Add a make target for manual Cockpit UI building: 'make 389-console' and a target to node_modules.mk 'eslint-fix'. Fix a few CSS issues. https://pagure.io/389-ds-base/issue/49999 Reviewed by: mreynolds, mhonek (Thanks!)
-
Thierry Bordaz authored
Bug Description: During a MODRDN of a group member, referential integrity will update the groups containing this member. Under specific conditions, the MODRDN can fail (err=1). on MODRDN Referential integrity checks if the original DN of the target MODRDN entry is member of a given group. If it is then it updates the group. The returned code of the group update is using the variable 'rc'. It does a normalized DN comparison to compare original DN with members DN, to determine if a group needs to be updated. If the group does not need to be updated, 'rc' is not set. The bug is that it uses 'rc' to normalize the DN and if the group is not updated the returned code reflects the normalization returned code rather that the group update. The bug is hit in specific conditions One of the evaluated group contains more than 128 members the last member (last value) of the group is not the moved entry the last member (last value) of the group is a DN value that contains escaped chars Fix Description: Use a local variable to check the result of the DN normalization https://pagure.io/389-ds-base/issue/50020 Reviewed by: Simon Pichugin, Mark Reynolds (thanks) Platforms tested: F27 Flag Day: no
-
Thierry Bordaz authored
Bug Description: During a dse update (config, schema,..) the dse callback will process the mods but can also modify them (SLAPI_MODIFY_MODS) leaving only ignored attributes. A consequence is that later audit logging will only log the ignored attributes. Fix Description: Save a copy of the orignal mods before the dse callback and restore them when dse callback completes. https://pagure.io/389-ds-base/issue/50026 Reviewed by: Ludwig Krispenz Platforms tested: F27 Flag Day: no Doc impact: no
-
Mark Reynolds authored