Bug 533979 (ipa-fixes)

Summary: TRACKING bug for fixes IPA team will provide in one chunk for Dogtag
Product: [Retired] Dogtag Certificate System Reporter: Dmitri Pal <dpal>
Component: Certificate ManagerAssignee: Christina Fu <cfu>
Status: CLOSED EOL QA Contact: Ben Levenson <benl>
Severity: medium Docs Contact:
Priority: urgent    
Version: 1.3CC: dpal, jgalipea
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: 2020-03-27 20:05:27 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: 531940, 531953, 532092, 533054, 533287    
Bug Blocks:    
Attachments:
Description Flags
large patch which the other individual patches were broken out from
none
code committed for John Dennis none

Description Dmitri Pal 2009-11-09 22:58:54 UTC
Bugs that IPA team will fix:

533287
533054
533052
532092
531953
531940
531934

Comment 1 John Dennis 2009-11-12 16:49:50 UTC
Created attachment 369259 [details]
large patch which the other individual patches were broken out from

Comment 2 John Dennis 2009-11-12 16:59:20 UTC
I have provided patches for the following:

531937
531940
531953
532092
533052
533054
533287

Each of these bugs an individual patch attached to them as per the CS teams request. This will allow only the relevant change to be considered when reviewing the bug solution.

This leaves only 531934. We can make progress without 531934, but the others are blocking us. I'll provide a patch for 531934 later.

Comment 3 John Dennis 2009-11-12 17:03:19 UTC
The large patch in comment #1 is provided for only for sanity checking. The individual patches attached to the individual bug reports were broken out from the large patch. After applying all the individual patches the result should match the large patch. I provided the large patch as insurance I didn't make any mistakes when breaking out the smaller patches.

Comment 4 Andrew Wnuk 2009-11-18 00:48:10 UTC
bug #531937 - nothing to review, fix will be provided in bug #533376
bug #531940 - reviewed
bug #531953 - reviewed
bug #532092 - nothing to review, fix provided in bug #531953
bug #533052 - reviewed

Comment 5 Andrew Wnuk 2009-11-18 19:03:52 UTC
bug #533054 - reviewed
bug #533287 - nothing to review, fix provided in bug #531953

Comment 6 Andrew Wnuk 2009-11-18 19:06:31 UTC
Combined patches provided in bugs 531940, 533054, 533052, and 533054 are identical with the big patch attached to bug 533979.

Comment 7 Andrew Wnuk 2009-11-25 23:29:59 UTC
Created attachment 373874 [details]
code committed for John Dennis

This is John Dennis patch. It does not break CS build. Unit testing is responsibility of John Dennis.

Comment 8 Andrew Wnuk 2009-11-25 23:38:00 UTC
svn commit pki/base/common/src/com/netscape/cms/servlet/profile/ProfileSubmitServlet.java
Sending        pki/base/common/src/com/netscape/cms/servlet/profile/ProfileSubmitServlet.java
Transmitting file data .
Committed revision 858.

svn commit pki/base/common/src/com/netscape/cms/servlet/common/GenUnexpectedErrorTemplateFiller.java
Sending        pki/base/common/src/com/netscape/cms/servlet/common/GenUnexpectedErrorTemplateFiller.java
Transmitting file data .
Committed revision 859.

svn commit pki/base/common/src/com/netscape/cms/servlet/base/CMSServlet.java
Sending        pki/base/common/src/com/netscape/cms/servlet/base/CMSServlet.java
Transmitting file data .
Committed revision 860.