Bug 689866

Summary: ns-newpwpolicy.pl needs to use the new DN format
Product: [Retired] 389 Reporter: Noriko Hosoi <nhosoi>
Component: Command Line UtilitiesAssignee: Noriko Hosoi <nhosoi>
Status: CLOSED CURRENTRELEASE QA Contact: Viktor Ashirov <vashirov>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 1.2.8CC: amsharma, rmeggins
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 689895 (view as bug list) Environment:
Last Closed: 2015-12-07 16:53:01 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: 639035, 656390, 689895    
Attachments:
Description Flags
git patch file (master) nhosoi: review?, rmeggins: review+

Description Noriko Hosoi 2011-03-22 17:13:35 UTC 
Description of problem:
The command-line utility ns-newpwpolicy.pl (template-ns-newpwpolicy.pl.in) hasn't been modified to adjust to the new DN format.

It escapes only ',' in the nested DN, in which '=' also needs to be escaped.
Comment 1 Noriko Hosoi 2011-03-22 17:19:28 UTC 
Created attachment 486852 [details]
git patch file (master)

Description:  template-ns-newpwpolicy.pl.in hasn't been modified
to adjust to the new DN format.  It escapes only ',' in the nested
DN, in which '=' also needs to be escaped.  This patch fixed it.
Comment 2 Noriko Hosoi 2011-03-22 19:19:22 UTC 
Reviewed by Rich (Thank you!!!)

Pushed to master.

$ git merge 689866
Updating c772157..09bc822
Fast-forward
 .../src/scripts/template-ns-newpwpolicy.pl.in      |   20 +++++++++++---------
 1 files changed, 11 insertions(+), 9 deletions(-)

$ git push
Counting objects: 13, done.
Delta compression using up to 4 threads.
Compressing objects: 100% (6/6), done.
Writing objects: 100% (7/7), 845 bytes, done.
Total 7 (delta 5), reused 0 (delta 0)
To ssh://git.fedorahosted.org/git/389/ds.git
   c772157..09bc822  master -> master

Cherry-picked commit 09bc8226f17e58afd4fd1f4c3c73214ecc33b85e and pushed to 389-ds-base-1.2.8 branch:

$ git cherry-pick 09bc8226f17e58afd4fd1f4c3c73214ecc33b85e
Finished one cherry-pick.
[ds128-local 27624b8] Bug 689866 - ns-newpwpolicy.pl needs to use the new DN format
 1 files changed, 11 insertions(+), 9 deletions(-)


$ git push origin ds128-local:389-ds-base-1.2.8
Counting objects: 13, done.
Delta compression using up to 4 threads.
Compressing objects: 100% (6/6), done.
Writing objects: 100% (7/7), 844 bytes, done.
Total 7 (delta 5), reused 0 (delta 0)
To ssh://git.fedorahosted.org/git/389/ds.git
   df7c57c..27624b8  ds128-local -> 389-ds-base-1.2.8
Comment 4 Amita Sharma 2011-04-07 07:23:14 UTC 
Tested the bug by executing the pwpolicy test case passes 100%, 
bug is verified.