diff --git a/lib/puppet/ssl/certificate_factory.rb b/lib/puppet/ssl/certificate_factory.rb
index 3d8190c88..cd51ff5d7 100644
--- a/lib/puppet/ssl/certificate_factory.rb
+++ b/lib/puppet/ssl/certificate_factory.rb
@@ -1,165 +1,166 @@
 require 'puppet/ssl'
 
 # The tedious class that does all the manipulations to the
 # certificate to correctly sign it.  Yay.
 module Puppet::SSL::CertificateFactory
   # How we convert from various units to the required seconds.
   UNITMAP = {
     "y" => 365 * 24 * 60 * 60,
     "d" => 24 * 60 * 60,
     "h" => 60 * 60,
     "s" => 1
   }
 
   def self.build(cert_type, csr, issuer, serial)
     # Work out if we can even build the requested type of certificate.
     build_extensions = "build_#{cert_type.to_s}_extensions"
     respond_to?(build_extensions) or
       raise ArgumentError, "#{cert_type.to_s} is an invalid certificate type!"
 
     # set up the certificate, and start building the content.
     cert = OpenSSL::X509::Certificate.new
 
     cert.version    = 2 # X509v3
     cert.subject    = csr.content.subject
     cert.issuer     = issuer.subject
     cert.public_key = csr.content.public_key
     cert.serial     = serial
 
     # Make the certificate valid as of yesterday, because so many people's
     # clocks are out of sync.  This gives one more day of validity than people
     # might expect, but is better than making every person who has a messed up
     # clock fail, and better than having every cert we generate expire a day
     # before the user expected it to when they asked for "one year".
     cert.not_before = Time.now - (60*60*24)
     cert.not_after  = Time.now + ttl
 
     add_extensions_to(cert, csr, issuer, send(build_extensions))
 
     return cert
   end
 
   private
 
   def self.add_extensions_to(cert, csr, issuer, extensions)
     ef = OpenSSL::X509::ExtensionFactory.
       new(cert, issuer.is_a?(OpenSSL::X509::Request) ? cert : issuer)
 
     # Extract the requested extensions from the CSR.
     requested_exts = csr.request_extensions.inject({}) do |hash, re|
       hash[re["oid"]] = [re["value"], re["critical"]]
       hash
     end
 
     # Produce our final set of extensions.  We deliberately order these to
     # build the way we want:
     # 1. "safe" default values, like the comment, that no one cares about.
     # 2. request extensions, from the CSR
     # 3. extensions based on the type we are generating
     # 4. overrides, which we always want to have in their form
     #
     # This ordering *is* security-critical, but we want to allow the user
     # enough rope to shoot themselves in the foot, if they want to ignore our
     # advice and externally approve a CSR that sets the basicConstraints.
     #
     # Swapping the order of 2 and 3 would ensure that you couldn't slip a
     # certificate through where the CA constraint was true, though, if
     # something went wrong up there. --daniel 2011-10-11
     defaults = { "nsComment" => "Puppet Ruby/OpenSSL Internal Certificate" }
     override = { "subjectKeyIdentifier" => "hash" }
 
     exts = [defaults, requested_exts, extensions, override].
       inject({}) {|ret, val| ret.merge(val) }
 
     cert.extensions = exts.map do |oid, val|
       val, crit = *val
       val       = val.join(', ') unless val.is_a? String
 
       # Enforce the X509v3 rules about subjectAltName being critical:
       # specifically, it SHOULD NOT be critical if we have a subject, which we
       # always do. --daniel 2011-10-18
       crit = false if oid == "subjectAltName"
 
       # val can be either a string, or [string, critical], and this does the
       # right thing regardless of what we get passed.
       ef.create_ext(oid, val, crit)
     end
   end
 
   # TTL for new certificates in seconds. If config param :ca_ttl is set,
   # use that, otherwise use :ca_days for backwards compatibility
   def self.ttl
     ttl = Puppet.settings[:ca_ttl]
 
     return ttl unless ttl.is_a?(String)
 
     raise ArgumentError, "Invalid ca_ttl #{ttl}" unless ttl =~ /^(\d+)(y|d|h|s)$/
 
     $1.to_i * UNITMAP[$2]
   end
 
   # Woot! We're a CA.
   def self.build_ca_extensions
     {
       # This was accidentally omitted in the previous version of this code: an
       # effort was made to add it last, but that actually managed to avoid
       # adding it to the certificate at all.
       #
       # We have some sort of bug, which means that when we add it we get a
       # complaint that the issuer keyid can't be fetched, which breaks all
       # sorts of things in our test suite and, e.g., bootstrapping the CA.
       #
       # http://tools.ietf.org/html/rfc5280#section-4.2.1.1 says that, to be a
       # conforming CA we MAY omit the field if we are self-signed, which I
       # think gives us a pass in the specific case.
       #
       # It also notes that we MAY derive the ID from the subject and serial
       # number of the issuer, or from the key ID, and we definitely have the
       # former data, should we want to restore this...
       #
       # Anyway, preserving this bug means we don't risk breaking anything in
       # the field, even though it would be nice to have. --daniel 2011-10-11
       #
       # "authorityKeyIdentifier" => "keyid:always,issuer:always",
       "keyUsage"               => [%w{cRLSign keyCertSign}, true],
       "basicConstraints"       => ["CA:TRUE", true],
     }
   end
 
   # We're a terminal CA, probably not self-signed.
   def self.build_terminalsubca_extensions
     {
       "keyUsage"         => [%w{cRLSign keyCertSign}, true],
       "basicConstraints" => ["CA:TRUE,pathlen:0", true],
     }
   end
 
   # We're a normal server.
   def self.build_server_extensions
     {
       "keyUsage"         => [%w{digitalSignature keyEncipherment}, true],
-      "extendedKeyUsage" => [%w{serverAuth clientAuth emailProtection}, true],
+      "extendedKeyUsage" => [%w{serverAuth clientAuth}, true],
       "basicConstraints" => ["CA:FALSE", true],
     }
   end
 
   # Um, no idea.
   def self.build_ocsp_extensions
     {
       "keyUsage"         => [%w{nonRepudiation digitalSignature}, true],
       "extendedKeyUsage" => [%w{serverAuth OCSPSigning}, true],
       "basicConstraints" => ["CA:FALSE", true],
     }
   end
 
   # Normal client.
   def self.build_client_extensions
     {
       "keyUsage"         => [%w{nonRepudiation digitalSignature keyEncipherment}, true],
+      # We don't seem to use this, but that seems much more reasonable here...
       "extendedKeyUsage" => [%w{clientAuth emailProtection}, true],
       "basicConstraints" => ["CA:FALSE", true],
       "nsCertType"       => "client,email",
     }
   end
 end