paged results control is not working in some cases when we have a subsuffix.

[389-ds-bot]

Cloned from Pagure issue: https://pagure.io/389-ds-base/issue/47824

• Created at 2014-06-24 20:46:02 by gparente (@germanparente)
• Closed as Fixed
• Assigned to nhosoi (@nhosoi)
• Associated bugzillas
  • https://bugzilla.redhat.com/show_bug.cgi?id=1117975
  • https://bugzilla.redhat.com/show_bug.cgi?id=1112729

---

If a request is sent to server using paged searches and the suffix has a subsuffix, the results corresponding to the subsuffix are not shown in the query when the number of records per page is less than cardinal of the main suffix.

Hope it will be clear in the steps to reproduce:

• create a directory instance.
• create suffix o=redhat
• create subsuffix ou=subsuffix (parent o=redhat).

import a few entries as in attachment (see redhat.ldif and subsuffix.ldif).

ldapsearch the entire suffix with root entry as base dn and subtree scope. You will see all the entries.

Now use option "-E pr=/noprompt.

if is less than number of records in main suffix, the subsuffix entries will not appear in the request result.

if is greater or equal than number of records in the main suffix, we should see all the entries as requested.

[389-ds-bot]

Comment from gparente (@germanparente) at 2014-06-24 20:48:48

attachment
redhat.ldif

[389-ds-bot]

Comment from gparente (@germanparente) at 2014-06-24 20:48:58

attachment
subsuffix.ldif

[389-ds-bot]

Comment from nhosoi (@nhosoi) at 2014-07-10 00:28:20

Ticket has been cloned to Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1117975

[389-ds-bot]

Comment from nhosoi (@nhosoi) at 2014-07-10 23:46:13

git patch file (master)
0001-Ticket-47824-paged-results-control-is-not-working-in.patch

[389-ds-bot]

Comment from mreynolds (@mreynolds389) at 2014-07-15 23:23:30

Looks good. For the CI test, even if we are not using the tmp directory we should still clear it, because someone might copy/paste the script for a new testcase and that new testcase might use the tmp directory:

    # clear the tmp directory
    standalone.clearTmpDir(__file__)

[389-ds-bot]

Comment from nhosoi (@nhosoi) at 2014-07-16 00:59:14

git patch file (master) -- revised following the advice from Mark.
0001-Ticket-47824-CI-test-add-test-case-for-ticket-47824.patch

[389-ds-bot]

Comment from nhosoi (@nhosoi) at 2014-07-16 01:02:45

Reviewed by Mark (Thank you!!)

Pushed to master:
eb5fde9..b116d7e master -> master
commit b116d7e
Ticket 47824 - paged results control is not working in some cases when we have a subsuffix.

b116d7e..8b305c1 master -> master
commit 8b305c1
Ticket 47824 - CI test: add test case for ticket 47824

[389-ds-bot]

Comment from nhosoi (@nhosoi) at 2014-08-06 04:59:40

Changing the target milestone to 1.2.11.

Pushed to 389-ds-base-1.3.2:
7c5d8f2..0bc9016 389-ds-base-1.3.2 -> 389-ds-base-1.3.2
commit 0bc9016

Pushed to 389-ds-base-1.2.11:
c6ad00b..9674195 389-ds-base-1.2.11 -> 389-ds-base-1.2.11
commit 9674195

[389-ds-bot]

Comment from spichugi (@droideck) at 2016-07-20 14:42:11

attachment
0001-Ticket-47824-Remove-CI-test-from-tickets-and-add-log.patch

[389-ds-bot]

Comment from spichugi (@droideck) at 2016-07-20 19:41:15

To ssh://git.fedorahosted.org/git/389/ds.git

Pushed to master:
bff1dd4..e3a7705 master -> master
commit e3a7705
Author: Simon Pichugin droideck@redhat.com
Date: Wed Jul 20 10:10:55 2016 +0200

[389-ds-bot]

Comment from nhosoi (@nhosoi) at 2017-02-11 23:08:30

Metadata Update from @nhosoi:

• Issue assigned to nhosoi
• Issue set to the milestone: 1.2.11.30