From b74b209b6453e0734b4df1b226d77e640f7edbbc Mon Sep 17 00:00:00 2001
From: david_page <david_page@localhost>
Date: Fri, 05 Oct 2007 18:11:14 +0000
Subject: [PATCH] issue 466 (partial) CryptoManager more TODO: i18n cleanup

---
 opends/src/messages/messages/core.properties                 |   18 +++++++--
 opends/src/server/org/opends/server/types/CryptoManager.java |   65 +++++++++++++++-----------------
 2 files changed, 44 insertions(+), 39 deletions(-)

diff --git a/opends/src/messages/messages/core.properties b/opends/src/messages/messages/core.properties
index 9720add..2253e8e 100644
--- a/opends/src/messages/messages/core.properties
+++ b/opends/src/messages/messages/core.properties
@@ -1676,10 +1676,20 @@
  implementation %s is not indexed in backend %s.  Backend initialization \
  for this group implementation may take a very long time to complete
 SEVERE_ERR_CRYPTOMGR_CANNOT_GET_PREFERRED_DIGEST_661=CryptoManager cannot get \
- preferred digest:  %s
+ the preferred digest:  %s
 SEVERE_ERR_CRYPTOMGR_CANNOT_GET_PREFERRED_MAC_ENGINE_662=CryptoManager cannot \
- get preferred MAC engine:  %s
+ get the preferred MAC engine:  %s
 SEVERE_ERR_CRYPTOMGR_CANNOT_GET_PREFERRED_ENCRYPTION_CIPHER_663=CryptoManager \
- cannot get preferred encryption cipher:  %s
+ cannot get the preferred encryption cipher:  %s
 SEVERE_ERR_CRYPTOMGR_CANNOT_GET_PREFERRED_KEY_WRAPPING_CIPHER_664=CryptoManager \
- cannot get preferred key wrapping cipher:  %s
+ cannot get the preferred key wrapping cipher:  %s
+SEVERE_ERR_CRYPTOMGR_FAILED_TO_INITIATE_INSTANCE_KEY_GENERATION_665=CryptoManager \
+ failed to add entry "%s" to initiate instance key generation
+SEVERE_ERR_CRYPTOMGR_FAILED_TO_RETRIEVE_INSTANCE_CERTIFICATE_666=CryptoManager \
+ failed to retrieve entry "%s" (the instance-key-pair public-key certificate):  %s
+SEVERE_ERR_CRYPTOMGR_FAILED_TO_COMPUTE_INSTANCE_KEY_IDENTIFIER_667=CryptoManager \
+ failed to compute an instance key identifier:  %s
+SEVERE_ERR_CRYPTOMGR_FAILED_TO_ADD_INSTANCE_KEY_ENTRY_TO_ADS_668=CryptoManager \
+ failed to add entry "%s"
+SEVERE_ERR_CRYPTOMGR_FAILED_TO_PUBLISH_INSTANCE_KEY_ENTRY_669=CryptoManager \
+ failed to publish the instance-key-pair public-key-certificate entry in ADS:  %s
diff --git a/opends/src/server/org/opends/server/types/CryptoManager.java b/opends/src/server/org/opends/server/types/CryptoManager.java
index 3b1d0d4..89f57cb 100644
--- a/opends/src/server/org/opends/server/types/CryptoManager.java
+++ b/opends/src/server/org/opends/server/types/CryptoManager.java
@@ -109,6 +109,7 @@
   private static AttributeType attrInitVectorLength;
   private static AttributeType attrKeyLength;
   private static AttributeType attrCompromisedTime;
+  private static ObjectClass   ocCertRequest;
   private static ObjectClass   ocInstanceKey;
   private static ObjectClass   ocCipherKey;
   private static ObjectClass   ocMacKey;
@@ -208,6 +209,8 @@
            ConfigConstants.ATTR_CRYPTO_KEY_LENGTH_BITS);
       attrCompromisedTime = DirectoryServer.getAttributeType(
            ConfigConstants.ATTR_CRYPTO_KEY_COMPROMISED_TIME);
+      ocCertRequest = DirectoryServer.getObjectClass(
+              "ds-cfg-self-signed-cert-request"); // TODO: conf-const
       ocInstanceKey = DirectoryServer.getObjectClass(
            ConfigConstants.OC_CRYPTO_INSTANCE_KEY);
       ocCipherKey = DirectoryServer.getObjectClass(
@@ -391,21 +394,18 @@
         catch (DirectoryException ex) {
           if (0 == i
                   && ResultCode.NO_SUCH_OBJECT == ex.getResultCode()){
-            final Entry e = new Entry(entryDN, null, null, null);
-            final AttributeType ocAttrType
-                    = DirectoryServer.getAttributeType("objectclass");
-            e.addObjectClass(new AttributeValue(ocAttrType, "top"));
-            e.addObjectClass(new AttributeValue(ocAttrType,
-                    "ds-cfg-self-signed-cert-request"));
-            AddOperation addOperation = icc.processAdd(e.getDN(),
-                    e.getObjectClasses(),
-                    e.getUserAttributes(),
-                    e.getOperationalAttributes());
+            final Entry entry = new Entry(entryDN, null, null, null);
+            entry.addObjectClass(DirectoryServer.getTopObjectClass());
+            entry.addObjectClass(ocCertRequest);
+            AddOperation addOperation = icc.processAdd(entry.getDN(),
+                    entry.getObjectClasses(),
+                    entry.getUserAttributes(),
+                    entry.getOperationalAttributes());
             if (ResultCode.SUCCESS != addOperation.getResultCode()) {
               throw new DirectoryException(
                       addOperation.getResultCode(),
-                      Message.raw("Failed to add entry %s.",
-                              e.getDN().toString()));
+         ERR_CRYPTOMGR_FAILED_TO_INITIATE_INSTANCE_KEY_GENERATION.get(
+                 entry.getDN().toString()));
             }
           }
           else {
@@ -419,10 +419,8 @@
         TRACER.debugCaught(DebugLogLevel.ERROR, ex);
       }
       throw new CryptoManagerException(
-              // TODO: i18n
-              Message.raw("Failed to retrieve %s:  "
-                      + getExceptionMessage(ex).toString(),
-                      entryDN.toString()), ex);
+            ERR_CRYPTOMGR_FAILED_TO_RETRIEVE_INSTANCE_CERTIFICATE.get(
+                    getExceptionMessage(ex), entryDN.toString()), ex);
     }
     return(certificate);
   }
@@ -468,10 +466,8 @@
         TRACER.debugCaught(DebugLogLevel.ERROR, ex);
       }
       throw new CryptoManagerException(
-              // TODO: i18n
-            Message.raw("Failed to get MessageDigest instance for" +
-                    " %s:  " + getExceptionMessage(ex).toString(),
-                      mdAlgorithmName), ex);
+          ERR_CRYPTOMGR_FAILED_TO_COMPUTE_INSTANCE_KEY_IDENTIFIER.get(
+                  getExceptionMessage(ex)), ex);
     }
     return StaticUtils.bytesToHexNoSpace(
          md.digest(instanceKeyCertificate));
@@ -538,9 +534,9 @@
                       FILTER_OC_INSTANCE_KEY),
               requestedAttributes);
       if (0 == searchOp.getSearchEntries().size()) {
-        final Entry e = new Entry(entryDN, null, null, null);
-        e.addObjectClass(DirectoryServer.getTopObjectClass());
-        e.addObjectClass(ocInstanceKey);
+        final Entry entry = new Entry(entryDN, null, null, null);
+        entry.addObjectClass(DirectoryServer.getTopObjectClass());
+        entry.addObjectClass(ocInstanceKey);
         // Add the key ID attribute.
         final LinkedHashSet<AttributeValue> keyIDValueSet =
                 new LinkedHashSet<AttributeValue>(1);
@@ -549,7 +545,8 @@
                 attrKeyID,
                 attrKeyID.getNameOrOID(),
                 keyIDValueSet);
-        e.addAttribute(keyIDAttr, new ArrayList<AttributeValue>(0));
+        entry.addAttribute(keyIDAttr,
+                new ArrayList<AttributeValue>(0));
         // Add the public key certificate attribute.
         final LinkedHashSet<AttributeValue> certificateValueSet =
                 new LinkedHashSet<AttributeValue>(1);
@@ -565,18 +562,18 @@
                 attrPublicKeyCertificate.getNameOrOID(),
                 certificateOptions,
                 certificateValueSet);
-        e.addAttribute(certificateAttr,
+        entry.addAttribute(certificateAttr,
                 new ArrayList<AttributeValue>(0));
 
-        AddOperation addOperation = icc.processAdd(e.getDN(),
-                e.getObjectClasses(),
-                e.getUserAttributes(),
-                e.getOperationalAttributes());
+        AddOperation addOperation = icc.processAdd(entry.getDN(),
+                entry.getObjectClasses(),
+                entry.getUserAttributes(),
+                entry.getOperationalAttributes());
         if (ResultCode.SUCCESS != addOperation.getResultCode()) {
           throw new DirectoryException(
                   addOperation.getResultCode(),
-                  Message.raw("Failed to add entry %s.",
-                          e.getDN().toString()));
+            ERR_CRYPTOMGR_FAILED_TO_ADD_INSTANCE_KEY_ENTRY_TO_ADS.get(
+                    entry.getDN().toString()));
         }
       }
     } catch (DirectoryException ex) {
@@ -584,10 +581,8 @@
         TRACER.debugCaught(DebugLogLevel.ERROR, ex);
       }
       throw new CryptoManagerException(
-              // TODO: i18n
-              Message.raw("Failed to publish %s:  "
-                      + getExceptionMessage(ex).toString(),
-                      entryDN.toString()), ex);
+              ERR_CRYPTOMGR_FAILED_TO_PUBLISH_INSTANCE_KEY_ENTRY.get(
+                      getExceptionMessage(ex)), ex);
     }
   }
 

--
Gitblit v1.10.0