#50139 Ticket 50122 - Fix incorrect path spec
Closed 3 years ago by spichugi. Opened 5 years ago by firstyear.
firstyear/389-ds-base 50122-missing-path  into  master

file modified
+2 -2
@@ -191,9 +191,9 @@ 

          return

  

      try:

-         selinux.restorecon(slapd[path], recursive=True)

+         selinux.restorecon(path, recursive=True)

      except:

-         log.debug("Failed to run restorecon on: " + slapd[path])

+         log.debug("Failed to run restorecon on: " + path)

  

  def selinux_label_port(port, remove_label=False):

      """

Bug Description: Due to human error, I missed a path spec in
a change I made

Fix Description: Fix the path

https://pagure.io/389-ds-base/issue/50122

Author: William Brown william@blackhats.net.au

Review by: ???

rebased onto 4b8b470

5 years ago

Pull-Request has been merged by firstyear

5 years ago

This broke the installer, but fixing it via a different ticket 50165

389-ds-base is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in 389-ds-base's github repository.

This pull request has been cloned to Github as issue and is available here:
- https://github.com/389ds/389-ds-base/issues/3198

If you want to continue to work on the PR, please navigate to the github issue,
download the patch from the attachments and file a new pull request.

Thank you for understanding. We apologize for all inconvenience.

Pull-Request has been closed by spichugi

3 years ago
Metadata