Merge pull request #53 from RUB-NDS/CertificateCN

ic0ns · web-flow · commit 2016a3b05bb7 · 2019-05-13T17:52:24.000+02:00
correct common name in certificate report
diff --git a/src/main/java/de/rub/nds/tlsscanner/probe/certificate/CertificateReportGenerator.java b/src/main/java/de/rub/nds/tlsscanner/probe/certificate/CertificateReportGenerator.java
@@ -90,7 +90,7 @@ private static void setCommonNames(CertificateReportImplementation report,
         if (x500name != null) {
             RDN[] rdNs = x500name.getRDNs(BCStyle.CN);
             for (int i = 0; i < rdNs.length; i++) {
-                commonNames.append(IETFUtils.valueToString(rdNs[i]));
+                commonNames.append(IETFUtils.valueToString(rdNs[i].getFirst().getValue()));
                 if (i < rdNs.length - 1) {
                     commonNames.append(" ,");
                 }

Original file line number	Diff line number	Diff line change
`@@ -90,7 +90,7 @@ private static void setCommonNames(CertificateReportImplementation report,`
`90`	`90`	`if (x500name != null) {`
`91`	`91`	`RDN[] rdNs = x500name.getRDNs(BCStyle.CN);`
`92`	`92`	`for (int i = 0; i < rdNs.length; i++) {`
`93`		`- commonNames.append(IETFUtils.valueToString(rdNs[i]));`
	`93`	`+ commonNames.append(IETFUtils.valueToString(rdNs[i].getFirst().getValue()));`
`94`	`94`	`if (i < rdNs.length - 1) {`
`95`	`95`	`commonNames.append(" ,");`
`96`	`96`	`}`