From d8a0f1e6e9d2420be31200cf0554336e84eec843 Mon Sep 17 00:00:00 2001 From: James Moger <james.moger@gitblit.com> Date: Fri, 23 Nov 2012 10:38:38 -0500 Subject: [PATCH] Revised Gitblit GO certificate generation to use new X509 utility functions --- src/com/gitblit/MakeCertificate.java | 111 +++++++++++++++++++++---------------------------------- 1 files changed, 43 insertions(+), 68 deletions(-) diff --git a/src/com/gitblit/MakeCertificate.java b/src/com/gitblit/MakeCertificate.java index 14e749d..e3c39ff 100644 --- a/src/com/gitblit/MakeCertificate.java +++ b/src/com/gitblit/MakeCertificate.java @@ -29,8 +29,6 @@ import javax.security.auth.x500.X500Principal; -import org.bouncycastle.asn1.x500.X500NameBuilder; -import org.bouncycastle.asn1.x500.style.BCStyle; import org.bouncycastle.cert.X509v3CertificateBuilder; import org.bouncycastle.cert.jcajce.JcaX509CertificateConverter; import org.bouncycastle.cert.jcajce.JcaX509v3CertificateBuilder; @@ -41,10 +39,17 @@ import com.beust.jcommander.Parameter; import com.beust.jcommander.ParameterException; import com.beust.jcommander.Parameters; +import com.gitblit.utils.TimeUtils; +/** + * Utility class to generate self-signed certificates. + * + * @author James Moger + * + */ public class MakeCertificate { - private final static FileSettings fileSettings = new FileSettings(); + private static final String BC = org.bouncycastle.jce.provider.BouncyCastleProvider.PROVIDER_NAME; public static void main(String... args) { Params params = new Params(); @@ -52,104 +57,74 @@ try { jc.parse(args); } catch (ParameterException t) { + System.err.println(t.getMessage()); jc.usage(); } - File keystore = new File("keystore"); - generateSelfSignedCertificate(params.alias, keystore, params.storePassword, params.subject); + File keystore = new File("serverKeyStore.jks"); + generateSelfSignedCertificate(params.hostname, keystore, params.storePassword, + params.subject); } - - public static void generateSelfSignedCertificate(String hostname, File keystore, String keystorePassword) { + + public static void generateSelfSignedCertificate(String hostname, File keystore, + String keystorePassword, String info) { try { Security.addProvider(new org.bouncycastle.jce.provider.BouncyCastleProvider()); - final String BC = org.bouncycastle.jce.provider.BouncyCastleProvider.PROVIDER_NAME; - - KeyPairGenerator kpGen = KeyPairGenerator.getInstance("RSA", "BC"); - kpGen.initialize(1024, new SecureRandom()); - KeyPair pair = kpGen.generateKeyPair(); - - // Generate self-signed certificate - X500NameBuilder builder = new X500NameBuilder(BCStyle.INSTANCE); - builder.addRDN(BCStyle.OU, Constants.NAME); - builder.addRDN(BCStyle.O, Constants.NAME); - builder.addRDN(BCStyle.CN, hostname); - - Date notBefore = new Date(System.currentTimeMillis() - 1*24*60*60*1000l); - Date notAfter = new Date(System.currentTimeMillis() + 10*365*24*60*60*1000l); - BigInteger serial = BigInteger.valueOf(System.currentTimeMillis()); - - X509v3CertificateBuilder certGen = new JcaX509v3CertificateBuilder(builder.build(), serial, notBefore, notAfter, builder.build(), pair.getPublic()); - ContentSigner sigGen = new JcaContentSignerBuilder("SHA256WithRSAEncryption").setProvider(BC).build(pair.getPrivate()); - X509Certificate cert = new JcaX509CertificateConverter().setProvider(BC).getCertificate(certGen.build(sigGen)); - cert.checkValidity(new Date()); - cert.verify(cert.getPublicKey()); - - // Save to keystore - KeyStore store = KeyStore.getInstance("JKS"); - if (keystore.exists()) { - FileInputStream fis = new FileInputStream(keystore); - store.load(fis, keystorePassword.toCharArray()); - } else { - store.load(null); - } - store.setKeyEntry(hostname, pair.getPrivate(), keystorePassword.toCharArray(), new java.security.cert.Certificate[] { cert }); - store.store(new FileOutputStream(keystore), keystorePassword.toCharArray()); - } catch (Throwable t) { - t.printStackTrace(); - throw new RuntimeException("Failed to generate self-signed certificate!", t); - } - } - - public static void generateSelfSignedCertificate(String hostname, File keystore, String keystorePassword, String info) { - try { - Security.addProvider(new org.bouncycastle.jce.provider.BouncyCastleProvider()); - - final String BC = org.bouncycastle.jce.provider.BouncyCastleProvider.PROVIDER_NAME; - - KeyPairGenerator kpGen = KeyPairGenerator.getInstance("RSA", "BC"); + KeyPairGenerator kpGen = KeyPairGenerator.getInstance("RSA", BC); kpGen.initialize(1024, new SecureRandom()); KeyPair pair = kpGen.generateKeyPair(); // Generate self-signed certificate X500Principal principal = new X500Principal(info); - - Date notBefore = new Date(System.currentTimeMillis() - 1*24*60*60*1000l); - Date notAfter = new Date(System.currentTimeMillis() + 10*365*24*60*60*1000l); + + Date notBefore = new Date(System.currentTimeMillis() - TimeUtils.ONEDAY); + Date notAfter = new Date(System.currentTimeMillis() + 10 * TimeUtils.ONEYEAR); BigInteger serial = BigInteger.valueOf(System.currentTimeMillis()); - X509v3CertificateBuilder certGen = new JcaX509v3CertificateBuilder(principal, serial, notBefore, notAfter, principal, pair.getPublic()); - ContentSigner sigGen = new JcaContentSignerBuilder("SHA256WithRSAEncryption").setProvider(BC).build(pair.getPrivate()); - X509Certificate cert = new JcaX509CertificateConverter().setProvider(BC).getCertificate(certGen.build(sigGen)); + X509v3CertificateBuilder certGen = new JcaX509v3CertificateBuilder(principal, serial, + notBefore, notAfter, principal, pair.getPublic()); + ContentSigner sigGen = new JcaContentSignerBuilder("SHA256WithRSAEncryption") + .setProvider(BC).build(pair.getPrivate()); + X509Certificate cert = new JcaX509CertificateConverter().setProvider(BC) + .getCertificate(certGen.build(sigGen)); cert.checkValidity(new Date()); cert.verify(cert.getPublicKey()); - // Save to keystore + // Save to keystore KeyStore store = KeyStore.getInstance("JKS"); if (keystore.exists()) { FileInputStream fis = new FileInputStream(keystore); store.load(fis, keystorePassword.toCharArray()); + fis.close(); } else { store.load(null); } - store.setKeyEntry(hostname, pair.getPrivate(), keystorePassword.toCharArray(), new java.security.cert.Certificate[] { cert }); - store.store(new FileOutputStream(keystore), keystorePassword.toCharArray()); + store.setKeyEntry(hostname, pair.getPrivate(), keystorePassword.toCharArray(), + new java.security.cert.Certificate[] { cert }); + FileOutputStream fos = new FileOutputStream(keystore); + store.store(fos, keystorePassword.toCharArray()); + fos.close(); } catch (Throwable t) { t.printStackTrace(); throw new RuntimeException("Failed to generate self-signed certificate!", t); } } - + + /** + * JCommander Parameters class for MakeCertificate. + */ @Parameters(separators = " ") private static class Params { - @Parameter(names = { "--alias" }, description = "Server alias", required = true) - public String alias = null; - + private static final FileSettings FILESETTINGS = new FileSettings(Constants.PROPERTIES_FILE); + + @Parameter(names = { "--hostname" }, description = "Server Hostname", required = true) + public String hostname; + @Parameter(names = { "--subject" }, description = "Certificate subject", required = true) - public String subject = null; - + public String subject; @Parameter(names = "--storePassword", description = "Password for SSL (https) keystore.") - public String storePassword = fileSettings.getString(Keys.server.storePassword, ""); + public String storePassword = FILESETTINGS.getString(Keys.server.storePassword, ""); } } -- Gitblit v1.9.1