Bug 514770

Summary: GER, Paged Results: remove per-entry response control
Product: [Retired] 389 Reporter: Noriko Hosoi <nhosoi>
Component: Security - Access Control (GER)Assignee: Noriko Hosoi <nhosoi>
Status: CLOSED CURRENTRELEASE QA Contact: Viktor Ashirov <vashirov>
Severity: medium Docs Contact:
Priority: low    
Version: 1.2.1CC: jgalipea, nkinder, rmeggins
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-12-07 16:37:59 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 434914, 517385    
Attachments:
Description Flags
git patch file for result.c none

Description Noriko Hosoi 2009-07-30 17:15:54 UTC
Description of problem:
Currently, Simple Paged Results and GER returns response control per-entry.
Since per entry-response controls (except for persistent search EntryChange 
controls) are ignored by the ldapsearch client, we are getting rid of the
unnecessary write_controls calls for Simple Paged Results.

Comment 1 Noriko Hosoi 2009-07-30 17:36:48 UTC
(In reply to comment #0)
> we are getting rid of the
> unnecessary write_controls calls for Simple Paged Results.  
and GER...

Comment 2 Noriko Hosoi 2009-07-30 17:45:30 UTC
Created attachment 355712 [details]
git patch file for result.c

File: ldap/servers/slapd/result.c

Fix description: Getting rid of the unnecessary write_controls calls for GER and Simple Paged Results.

Note: both GER and Simple Paged Results add its control to the result controls maintained in the "operation" and they are sent to the client via send_ldap_result.

Comment 3 Noriko Hosoi 2009-07-30 18:43:44 UTC
Thanks to Nathan for the review.

Pushed to master.

$ git merge paged
Updating 7c52ad5..7a4fce4
Fast forward
 ldap/servers/slapd/result.c |   30 ------------------------------
 1 files changed, 0 insertions(+), 30 deletions(-)
$ git push
Counting objects: 11, done.
Delta compression using 4 threads.
Compressing objects: 100% (6/6), done.
Writing objects: 100% (6/6), 649 bytes, done.
Total 6 (delta 4), reused 0 (delta 0)
To ssh://git.fedorahosted.org/git/389/ds.git
   7c52ad5..7a4fce4  master -> master

Comment 4 Jenny Severance 2010-06-07 18:15:16 UTC
can you please add steps to verify this bug? Thanks!

Comment 5 Noriko Hosoi 2010-06-07 19:03:27 UTC
(In reply to comment #4)
> can you please add steps to verify this bug? Thanks!    

I'm afraid there is no way to verify this bug with a black box test.  It's a pure server internal issue -- getting rid of unnecessary return control.  Since GER and Simple Paged Results acceptance tests are passing 100%, I think we could say this bug is verified.

Comment 6 Jenny Severance 2010-06-07 19:29:30 UTC
Thanks Noriko - marking verified.