FIX: Cleanup fields
authorJanis Streib <janis@dogcraft.de>
Sat, 23 Aug 2014 10:44:34 +0000 (12:44 +0200)
committerJanis Streib <janis@dogcraft.de>
Sat, 23 Aug 2014 10:44:34 +0000 (12:44 +0200)
30 files changed:
src/org/cacert/gigi/CertificateProfile.java
src/org/cacert/gigi/Digest.java
src/org/cacert/gigi/Domain.java
src/org/cacert/gigi/Gigi.java
src/org/cacert/gigi/GigiApiException.java
src/org/cacert/gigi/GigiConfig.java
src/org/cacert/gigi/User.java
src/org/cacert/gigi/crypto/SMIME.java
src/org/cacert/gigi/database/DatabaseConnection.java
src/org/cacert/gigi/email/TestEmailProvider.java
src/org/cacert/gigi/localisation/Language.java
src/org/cacert/gigi/output/CertificateIterable.java
src/org/cacert/gigi/output/ClientCSRGenerate.java
src/org/cacert/gigi/output/DateSelector.java
src/org/cacert/gigi/output/Form.java
src/org/cacert/gigi/output/Menu.java
src/org/cacert/gigi/output/MenuItem.java
src/org/cacert/gigi/output/template/HashAlgorithms.java
src/org/cacert/gigi/output/template/Template.java
src/org/cacert/gigi/output/template/TemplateBlock.java
src/org/cacert/gigi/pages/account/CertificateIssueForm.java
src/org/cacert/gigi/pages/account/Certificates.java
src/org/cacert/gigi/pages/account/ChangeForm.java
src/org/cacert/gigi/pages/account/MailAddForm.java
src/org/cacert/gigi/pages/main/Signup.java
src/org/cacert/gigi/pages/wot/AssuranceForm.java
src/org/cacert/gigi/pages/wot/AssurePage.java
src/org/cacert/gigi/util/Job.java
src/org/cacert/gigi/util/PasswordStrengthChecker.java
src/org/cacert/gigi/util/RandomToken.java

index d2e6b27784344aba482a749648236c7d456e6389..845fa6acb04d93eb53028c11fad148349e487f66 100644 (file)
@@ -9,15 +9,15 @@ import org.cacert.gigi.database.DatabaseConnection;
 
 public class CertificateProfile {
 
-    final int id;
+    private final int id;
 
-    final String keyName;
+    private final String keyName;
 
-    final String visibleName;
+    private final String visibleName;
 
-    static HashMap<String, CertificateProfile> byName = new HashMap<>();
+    private static HashMap<String, CertificateProfile> byName = new HashMap<>();
 
-    static HashMap<Integer, CertificateProfile> byId = new HashMap<>();
+    private static HashMap<Integer, CertificateProfile> byId = new HashMap<>();
 
     private CertificateProfile(int id, String keyName, String visibleName) {
         this.id = id;
index bf7cfee60f8a0b2aaa50e1ad5c9d45d7148ca335..8c4644f72dc9dd38a6397634a4ab00d57b30520d 100644 (file)
@@ -3,7 +3,7 @@ package org.cacert.gigi;
 public enum Digest {
     SHA256("Currently recommended, because the other algorithms" + " might break on some older versions of the GnuTLS library" + " (older than 3.x) still shipped in Debian for example."), SHA384(null), SHA512("Highest protection against hash collision attacks of the algorithms offered here.");
 
-    final String exp;
+    private final String exp;
 
     private Digest(String explanation) {
         exp = explanation;
index 0adf6a4c2d31683f3b3da15a4299d7c0f48ff299..538081d8d37143f3e924c2579d57162894d9996c 100644 (file)
@@ -8,11 +8,11 @@ import org.cacert.gigi.database.DatabaseConnection;
 
 public class Domain {
 
-    User owner;
+    private User owner;
 
-    String suffix;
+    private String suffix;
 
-    int id;
+    private int id;
 
     public Domain(int id) throws SQLException {
         PreparedStatement ps = DatabaseConnection.getInstance().prepare("SELECT memid, domain FROM `domains` WHERE id=? AND deleted IS NULL");
index 20a2b1ca3e72d7ab323a90cad3df42ca095182a4..0fead124456ddb1e2131eb2fb20bec33b6b509bc 100644 (file)
@@ -48,7 +48,7 @@ public class Gigi extends HttpServlet {
 
     private HashMap<String, Page> pages = new HashMap<String, Page>();
 
-    Menu m;
+    private Menu m;
 
     public Gigi(Properties conf) {
         DatabaseConnection.init(conf);
index 8851ec4412efaf4f848a4ed142de11344bbc1291..a79877396061d9134fdbc8e16eb0145a6c923a5b 100644 (file)
@@ -8,9 +8,9 @@ import org.cacert.gigi.localisation.Language;
 
 public class GigiApiException extends Exception {
 
-    SQLException e;
+    private SQLException e;
 
-    LinkedList<String> messages = new LinkedList<>();
+    private LinkedList<String> messages = new LinkedList<>();
 
     public GigiApiException(SQLException e) {
         super(e);
index 4c36302abaca89c52ef66ba8ca32681686562bd5..3a1b9eed6642aadbbf4d733e9b3440a118ff248e 100644 (file)
@@ -15,11 +15,11 @@ public class GigiConfig {
 
     public static final String GIGI_CONFIG_VERSION = "GigiConfigV1.0";
 
-    byte[] cacerts;
+    private byte[] cacerts;
 
-    byte[] keystore;
+    private byte[] keystore;
 
-    Properties mainProps = new Properties();
+    private Properties mainProps = new Properties();
 
     private char[] keystorpw;
 
index b3fdb26ceaf23b6bb2cd2d36a1a4d3ca7a1e972f..eebf9317e2b4839aadc894fd0f86363787e17ab2 100644 (file)
@@ -14,11 +14,11 @@ public class User {
 
     private int id;
 
-    Name name = new Name(null, null, null, null);
+    private Name name = new Name(null, null, null, null);
 
-    Date dob;
+    private Date dob;
 
-    String email;
+    private String email;
 
     public User(int id) {
         this.id = id;
index f3ac0f766b6f00e9eff6960abdf675f592e0f9ff..9b6573409892a42a38d23fc46e7cfda727cf3191 100644 (file)
@@ -74,7 +74,7 @@ public class SMIME {
         mimeEncode(contents, Base64.getEncoder().encodeToString(bOut.toByteArray()).replaceAll("(.{64})(?=.)", "$1\n"), to);
     }
 
-    static Random r = new Random();
+    private static Random r = new Random();
 
     private static void mimeEncode(String contents, String signature, PrintWriter to) {
         String boundary = generateBoundary(contents, null);
index 389a82cfe84bb43fbe1b0a1b8ac39448389778e0..21d701cb4bc4048589fcd845040e5660a405912d 100644 (file)
@@ -5,21 +5,21 @@ import java.sql.DriverManager;
 import java.sql.PreparedStatement;
 import java.sql.ResultSet;
 import java.sql.SQLException;
+import java.sql.Statement;
 import java.util.HashMap;
 import java.util.Properties;
-import java.sql.Statement;
 
 public class DatabaseConnection {
 
     public static final int CONNECTION_TIMEOUT = 24 * 60 * 60;
 
-    Connection c;
+    private Connection c;
 
-    HashMap<String, PreparedStatement> statements = new HashMap<String, PreparedStatement>();
+    private HashMap<String, PreparedStatement> statements = new HashMap<String, PreparedStatement>();
 
     private static Properties credentials;
 
-    Statement adHoc;
+    private Statement adHoc;
 
     public DatabaseConnection() {
         try {
@@ -54,7 +54,7 @@ public class DatabaseConnection {
         return statement;
     }
 
-    long lastAction = System.currentTimeMillis();
+    private long lastAction = System.currentTimeMillis();
 
     private void ensureOpen() {
         if (System.currentTimeMillis() - lastAction > CONNECTION_TIMEOUT * 1000L) {
@@ -79,7 +79,7 @@ public class DatabaseConnection {
         return id;
     }
 
-    static ThreadLocal<DatabaseConnection> instances = new ThreadLocal<DatabaseConnection>() {
+    private static ThreadLocal<DatabaseConnection> instances = new ThreadLocal<DatabaseConnection>() {
 
         @Override
         protected DatabaseConnection initialValue() {
index caf29661e4df6565bd1e70d4c813bd95e77b3ee6..7beaa9a98001bf190cbe79546cc8ea3601473a69 100644 (file)
@@ -10,13 +10,13 @@ import java.util.Properties;
 
 class TestEmailProvider extends EmailProvider {
 
-    ServerSocket servs;
+    private ServerSocket servs;
 
-    Socket client;
+    private Socket client;
 
-    DataOutputStream out;
+    private DataOutputStream out;
 
-    DataInputStream in;
+    private DataInputStream in;
 
     protected TestEmailProvider(Properties props) {
         try {
index dc99d9a8a6ace810037a99881f8a28a26436e55c..4f7cd17a807bb0d9b33a8b654fbd532f4aefd36a 100644 (file)
@@ -19,9 +19,9 @@ public class Language {
 
     private static HashMap<String, Language> langs = new HashMap<String, Language>();
 
-    HashMap<String, String> translations = new HashMap<String, String>();
+    private HashMap<String, String> translations = new HashMap<String, String>();
 
-    Locale l;
+    private Locale l;
 
     protected Language(String language) throws ParserConfigurationException, IOException, SAXException {
         if (language.contains("_")) {
index 62e7f7057541a474389960c3028d357260156cf1..41d046e72020f0bf35a435283b76d11f82b7b954 100644 (file)
@@ -9,13 +9,13 @@ import org.cacert.gigi.output.template.IterableDataset;
 
 public class CertificateIterable implements IterableDataset {
 
-    Certificate[] certificates;
+    private Certificate[] certificates;
 
     public CertificateIterable(Certificate[] certificates) {
         this.certificates = certificates;
     }
 
-    int i = 0;
+    private int i = 0;
 
     @Override
     public boolean next(Language l, Map<String, Object> vars) {
index 4885440ccb5b29c3eb220f72e9712ba89015c2d0..20d1c68f04d985c4983d1075190ec1056bc18d7a 100644 (file)
@@ -12,9 +12,9 @@ import org.cacert.gigi.util.ServerConstants;
 
 public class ClientCSRGenerate {
 
-    static Template normal;
+    private static Template normal;
 
-    static Template IE;
+    private static Template IE;
     static {
         normal = new Template(ClientCSRGenerate.class.getResource("ClientCSRGenerate.templ"));
         IE = new Template(ClientCSRGenerate.class.getResource("ClientCSRGenerateIE.templ"));
index cf42c220abc782dab32240a4f3846f6287af1dfc..442dc773e820351c2d29b6a66e722a6e14b0c770 100644 (file)
@@ -13,7 +13,7 @@ import org.cacert.gigi.localisation.Language;
 
 public class DateSelector implements Outputable {
 
-    String[] names;
+    private String[] names;
 
     public DateSelector(String day, String month, String year) {
         this.names = new String[] {
@@ -21,11 +21,11 @@ public class DateSelector implements Outputable {
         };
     }
 
-    int day;
+    private int day;
 
-    int month;
+    private int month;
 
-    int year;
+    private int year;
 
     @Override
     public void output(PrintWriter out, Language l, Map<String, Object> vars) {
index a4c65a488c70310b311627247e0484df22c9c362..a6374e532b74c0b2b57f50015bbdd878fc7c9d56 100644 (file)
@@ -16,7 +16,7 @@ public abstract class Form implements Outputable {
 
     public static final String CSRF_FIELD = "csrf";
 
-    String csrf;
+    private String csrf;
 
     public Form(HttpServletRequest hsr) {
         csrf = RandomToken.generateToken(32);
index aefdbea66f9ced245bc0f3a0017ab2cc364896f6..629154cc4476b783179da5380bac6fa837327b5b 100644 (file)
@@ -7,9 +7,9 @@ import org.cacert.gigi.localisation.Language;
 
 public class Menu implements Outputable {
 
-    String menuName;
+    private String menuName;
 
-    String id;
+    private String id;
 
     private IMenuItem[] content;
 
index 25e1b8cece0042124ad1fef1c0d900a3f1bcf1c3..fc6be1505bbe0031fff3bbac877a661c54c32173 100644 (file)
@@ -7,9 +7,9 @@ import org.cacert.gigi.localisation.Language;
 
 public class MenuItem implements Outputable, IMenuItem {
 
-    final String href;
+    private final String href;
 
-    final String name;
+    private final String name;
 
     public MenuItem(String href, String name) {
         this.href = href;
index 37b1ca705511e15d93a9db96e9c72238f5db89bf..ee2a4b14281ff379598d245181c592b51b5586da 100644 (file)
@@ -7,9 +7,9 @@ import org.cacert.gigi.localisation.Language;
 
 public class HashAlgorithms implements IterableDataset {
 
-    int i = 0;
+    private int i = 0;
 
-    Digest selected;
+    private Digest selected;
 
     public HashAlgorithms(Digest selected) {
         this.selected = selected;
index c7d9a066c6b0ab1a45381e9e950199db1dcfe44a..ac59f3135ef75a11639710a81983be75ab990c53 100644 (file)
@@ -21,11 +21,11 @@ import org.cacert.gigi.util.HTMLEncoder;
 
 public class Template implements Outputable {
 
-    TemplateBlock data;
+    private TemplateBlock data;
 
-    long lastLoaded;
+    private long lastLoaded;
 
-    File source;
+    private File source;
 
     private static final Pattern CONTROL_PATTERN = Pattern.compile(" ?([a-z]+)\\(\\$([^)]+)\\) ?\\{ ?");
 
@@ -134,6 +134,7 @@ public class Template implements Outputable {
         return null;
     }
 
+    @Override
     public void output(PrintWriter out, Language l, Map<String, Object> vars) {
         if (source != null && DevelLauncher.DEVEL) {
             if (lastLoaded < source.lastModified()) {
index 2eb87e7b2e35f7fa4684407ea579449f2adee1cd..6b30cf33f7559d81570fcf3483e4e71dabc8064c 100644 (file)
@@ -8,9 +8,9 @@ import org.cacert.gigi.output.Outputable;
 
 class TemplateBlock implements Outputable {
 
-    String[] contents;
+    private String[] contents;
 
-    Outputable[] vars;
+    private Outputable[] vars;
 
     public TemplateBlock(String[] contents, Outputable[] vars) {
         this.contents = contents;
index 184b87860ca8a0388a1ea36ddc52f02237f6767f..0ec43af5c5f032d4c4f5889f54ea4a861f9ffb1f 100644 (file)
@@ -94,23 +94,23 @@ public class CertificateIssueForm extends Form {
             1, 3, 6, 1, 5, 5, 7, 3, 9
     });
 
-    User u;
+    private User u;
 
     private CSRType csrType;
 
-    String csr;
+    private String csr;
 
-    String spkacChallenge;
+    private String spkacChallenge;
 
     public String CN = DEFAULT_CN;
 
-    Set<SubjectAlternateName> SANs = new LinkedHashSet<>();
+    private Set<SubjectAlternateName> SANs = new LinkedHashSet<>();
 
-    Digest selectedDigest = Digest.getDefault();
+    private Digest selectedDigest = Digest.getDefault();
 
-    boolean login;
+    private boolean login;
 
-    CertificateProfile profile = CertificateProfile.getById(1);
+    private CertificateProfile profile = CertificateProfile.getById(1);
 
     public CertificateIssueForm(HttpServletRequest hsr) {
         super(hsr);
@@ -118,7 +118,7 @@ public class CertificateIssueForm extends Form {
         spkacChallenge = RandomToken.generateToken(16);
     }
 
-    Certificate result;
+    private Certificate result;
 
     public Certificate getResult() {
         return result;
index 66f8e90fcfbbe229f2580711180c00426f1da36d..0293d86afe6ae460b825b7bc532aa310173d8658 100644 (file)
@@ -22,7 +22,7 @@ import org.cacert.gigi.util.PEM;
 
 public class Certificates extends Page {
 
-    Template certDisplay = new Template(Certificates.class.getResource("CertificateDisplay.templ"));
+    private Template certDisplay = new Template(Certificates.class.getResource("CertificateDisplay.templ"));
 
     public static final String PATH = "/account/certs";
 
@@ -85,7 +85,7 @@ public class Certificates extends Page {
         return true;
     }
 
-    Template certTable = new Template(CertificateIterable.class.getResource("CertificateTable.templ"));
+    private Template certTable = new Template(CertificateIterable.class.getResource("CertificateTable.templ"));
 
     @Override
     public void doGet(HttpServletRequest req, HttpServletResponse resp) throws IOException {
index 8a7689079f0023767f6def090411f02a9fb5d144..f43d62d02b0f5a69e7731f251a8fc66487d71ef9 100644 (file)
@@ -14,7 +14,7 @@ import org.cacert.gigi.pages.Page;
 
 public class ChangeForm extends Form {
 
-    User target;
+    private User target;
 
     public ChangeForm(HttpServletRequest hsr, User target) {
         super(hsr);
index 44634de1ea575cddc100a6708b2570d22e563a1c..3500faaeb8e080cbe00ee0fbba4fe655487174c8 100644 (file)
@@ -21,7 +21,7 @@ public class MailAddForm extends Form {
         t = new Template(ChangePasswordPage.class.getResource("MailAddForm.templ"));
     }
 
-    User target;
+    private User target;
 
     public MailAddForm(HttpServletRequest hsr, User target) {
         super(hsr);
index c9b77f6125c71d2ecb178eb9d275031bae672af9..a7a1a88a9c1681a9a32483a42bc18eca1e99851b 100644 (file)
@@ -2,10 +2,10 @@ package org.cacert.gigi.pages.main;
 
 import java.io.IOException;
 import java.io.PrintWriter;
+import java.sql.Date;
 import java.sql.PreparedStatement;
 import java.sql.ResultSet;
 import java.sql.SQLException;
-import java.sql.Date;
 import java.util.HashMap;
 import java.util.Map;
 
@@ -26,9 +26,9 @@ import org.cacert.gigi.util.PasswordStrengthChecker;
 
 public class Signup extends Form {
 
-    User buildup = new User();
+    private User buildup = new User();
 
-    Template t;
+    private Template t;
 
     boolean general = true, country = true, regional = true, radius = true;
 
index e3dfe54c106f04b05b2635f259a092ac3c79a9d8..01c4ea0e33edd5afb5e2080a47941bf5d6d649fd 100644 (file)
@@ -20,9 +20,9 @@ import org.cacert.gigi.util.Notary.AssuranceResult;
 
 public class AssuranceForm extends Form {
 
-    User assuree;
+    private User assuree;
 
-    static final Template templ;
+    private static final Template templ;
     static {
         templ = new Template(AssuranceForm.class.getResource("AssuranceForm.templ"));
     }
@@ -93,4 +93,8 @@ public class AssuranceForm extends Form {
         return false;
     }
 
+    public User getAssuree() {
+        return assuree;
+    }
+
 }
index 0e9bc4196354848c487f3219dfed546c9af7b5ad..d7074d520104a7b184ce35e778ba28cfa6f976a0 100644 (file)
@@ -58,7 +58,7 @@ public class AssurePage extends Page {
             out.println(translate(req, check.getMessage()));
             return;
         }
-        if (form == null || form.assuree.getId() != mid) {
+        if (form == null || form.getAssuree().getId() != mid) {
             form = new AssuranceForm(req, mid);
         }
 
@@ -78,7 +78,7 @@ public class AssurePage extends Page {
             }
 
             AssuranceForm form = Form.getForm(req, AssuranceForm.class);
-            if (mid != form.assuree.getId()) {
+            if (mid != form.getAssuree().getId()) {
                 return;
             }
             if (form.submit(out, req)) {
index 13e6c7e0c1c5c768f0888ee7c3c977b4a016a8e4..9ec4612986ff498e9a8a5d18c773f879f3730cae 100644 (file)
@@ -9,7 +9,7 @@ import org.cacert.gigi.database.DatabaseConnection;
 
 public class Job {
 
-    int id;
+    private int id;
 
     private Job(int id) {
         this.id = id;
index e52c1dd07d080a5544fa3426bb3ed8a96e2d50e9..7df2e220c7ed1d2d5e594546ddc64be64eefdb9b 100644 (file)
@@ -7,15 +7,15 @@ import org.cacert.gigi.User;
 
 public class PasswordStrengthChecker {
 
-    static Pattern digits = Pattern.compile("\\d");
+    private static Pattern digits = Pattern.compile("\\d");
 
-    static Pattern lower = Pattern.compile("[a-z]");
+    private static Pattern lower = Pattern.compile("[a-z]");
 
-    static Pattern upper = Pattern.compile("[A-Z]");
+    private static Pattern upper = Pattern.compile("[A-Z]");
 
-    static Pattern whitespace = Pattern.compile("\\s");
+    private static Pattern whitespace = Pattern.compile("\\s");
 
-    static Pattern special = Pattern.compile("(?!\\s)\\W");
+    private static Pattern special = Pattern.compile("(?!\\s)\\W");
 
     private PasswordStrengthChecker() {}
 
index 0c1035d6f9813b5f0234c9af8b10487325de4feb..8a11c8f096abcb6868dd15f517382d4636f1a589 100644 (file)
@@ -4,7 +4,7 @@ import java.security.SecureRandom;
 
 public class RandomToken {
 
-    static SecureRandom sr = new SecureRandom();
+    private static SecureRandom sr = new SecureRandom();
 
     public static String generateToken(int length) {
         StringBuffer token = new StringBuffer();