From d13a1d8a8506438a1f6b27f96f41e0ecaf612852 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Felix=20D=C3=B6rre?= Date: Thu, 15 Sep 2016 09:50:37 +0200 Subject: [PATCH] fix: certlogin. There was a "toLower" needed instead of an "toUpper" Change-Id: Ie233b6e920ec486a7e59d100681e86856bc7485c --- src/org/cacert/gigi/Gigi.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/org/cacert/gigi/Gigi.java b/src/org/cacert/gigi/Gigi.java index 23f10df9..7f6b3c18 100644 --- a/src/org/cacert/gigi/Gigi.java +++ b/src/org/cacert/gigi/Gigi.java @@ -361,7 +361,7 @@ public final class Gigi extends HttpServlet { if (clientSerial != null) { X509Certificate[] cert = (X509Certificate[]) req.getAttribute("javax.servlet.request.X509Certificate"); if (cert == null || cert[0] == null// - || !cert[0].getSerialNumber().toString(16).toUpperCase().equals(clientSerial) // + || !cert[0].getSerialNumber().toString(16).toLowerCase().equals(clientSerial) // || !cert[0].getIssuerDN().equals(hs.getAttribute(CERT_ISSUER))) { hs.invalidate(); resp.sendError(403, "Certificate mismatch."); -- 2.39.2