]> WPIA git - gigi.git/commitdiff
Merge "upd: get enough space between radio/checkbox and following text"
authorMarcus Mängel <m.maengel@inopiae.de>
Sun, 8 Sep 2019 14:03:28 +0000 (16:03 +0200)
committerGerrit Code Review <gigi-system@dogcraft.de>
Sun, 8 Sep 2019 14:03:28 +0000 (16:03 +0200)
src/club/wpia/gigi/database/SQLFileManager.java
src/club/wpia/gigi/pages/LoginPage.templ
src/club/wpia/gigi/pages/MainPageNotLogin.templ
src/club/wpia/gigi/pages/RootCertPage.templ
src/club/wpia/gigi/pages/main/KeyCompromiseForm.templ
src/club/wpia/gigi/pages/main/Signup.java
src/club/wpia/gigi/pages/orga/AffiliationForm.templ
src/club/wpia/gigi/ping/PingerDaemon.java
tests/club/wpia/gigi/ping/TestTiming.java

index 3364924744e1e94bab48ab5cf664453c2cabf569..16f8fa21587b702081baf01a095111806005a3b2 100644 (file)
@@ -42,7 +42,7 @@ public class SQLFileManager {
                 }
             }
         }
-        sql = sql.replaceAll("--[^\n]*\n", "\n");
+        sql = sql.replaceAll("(^|\n)--[^\n]*\n", "\n");
         sql = sql.replaceAll("#[^\n]*\n", "\n");
         String[] stmts = sql.split(";");
         Pattern p = Pattern.compile("\\s*DROP TABLE IF EXISTS \"([^\"]+)\"");
index 621d9f8511c54b10692b01f6de14ed53d6e67e4e..d08ab021dd018e301fe821171963983cdb64d04a 100644 (file)
@@ -4,5 +4,5 @@
 <label for="username"><?=_Email Address?>:</label><input class="form-control" type='text' name="username" required/><br />
 <label for="password"><?=_Password?>:</label><input class="form-control" type='password' name='password' required/><br />
 <button type='submit' name="process" value="Login" class="btn btn-primary"><?=_Login?></button><br /><br />
-<p class='smalltext'><?=_If you are having trouble with your username or password, please visit our !(/kb/lostPassword)knowledge base!'</a>' for more information?></p>
+<p class='smalltext'><?=_If you are having trouble with your username or password, please visit our !(/kb/lostPassword)FAQ!'</a>' for more information?></p>
 </div>
index 2e3ef1a3a4730fefeb9127c0de5c5cec71272079..4e604009e22c5e25c15124a93c09ccb7a16473d3 100644 (file)
@@ -12,5 +12,5 @@ On the other hand, server certificates can be used to establish secure internet
 
 <hr/>
 
-<p><?=_For general documentation and help, please visit the ${appName} !(/kb/gigi)knowledge base!'</a>'.?>
+<p><?=_For general documentation and help, please visit the ${appName} !(/kb/gigi)FAQ!'</a>'.?>
 <?=_For specific policies, see the ${appName} !(/policy)Policies page!'</a>'.?></p>
\ No newline at end of file
index 899470ec688551f4415ab0420c14c6b5046bd062..10b20f8ca44a446309e9f378fe43683d9280490e 100644 (file)
@@ -2,7 +2,7 @@
 <a href="?pem">PEM</a> <a href="?cer">DER</a></p>
 <p><?=_A p7b file with all intermediate certificates is available for download here:?><br/>
 <a href="?bundle"><?=$bundle?></a></p>
-<p><?=_Find information how to add the root and intermediate certificates to the truststore of your browser or operating system in our !(/kb/truststores)knowledge base!'</a>'.?></p>
+<p><?=_Find information how to add the root and intermediate certificates to the truststore of your browser or operating system in our !(/kb/truststores)FAQ!'</a>'.?></p>
 <p>
 <?=_A full list of all DER-encoded intermediate certificates is provided below:?>
 </p>
index 9aadd3a0aaa05201ecc8a14ddda8611847e812d0..8a8d1ad6a99620afec3b7f228f3e8ea6cf6f5e90 100644 (file)
@@ -1,14 +1,9 @@
 <p>
-<?=_This form allows you to report a certificate whose private key has been compromised.?>
-<?=_You require to identify the certificate you need to report.?>
-<?=_You may upload the certificate (as PEM or DER) or may identify the certificate by serial.?>
+<?=_This form allows you to report a certificate whose private key has been compromised. You are required to identify the certificate you need to report. You may upload the certificate (as PEM or DER) or may identify the certificate by serial.?>
 </p>
 
 <p>
-<?=_Additionally you need to prove that you have access to the private key.?>
-<?=_There are also two possibilities available here:?>
-<?=_Either you may upload the plain private key (as PEM or DER) or you may only sign a given message with the according private key.?>
-<?=_You may create a fitting signature with this command:?>
+<?=_Additionally you need to prove that you have access to the private key. There are also two possibilities available here: Either you may upload the plain private key (as PEM or DER) or you may only sign a given message with the according private key. You may create a fitting signature with this command:?>
 </p>
 <p>
 <code data-challenge="<?=$challenge?>">
@@ -40,8 +35,7 @@ printf '%s' '<?=$challengePrefix?><?=$challenge?>' | openssl dgst -sha256 -sign
   </tr>
   <tr>
     <td colspan="4">
-      <?=_You may provide information on how the private key was compromised to help the certificate owner prevent further key compromises.?>
-      <?=_You can indicate that this information should not be sent to the certificate owner, but only be visible to ${appName} staff, by checking the checkbox below.?>
+      <?=_You may provide information on how the private key was compromised to help the certificate owner prevent further key compromises. You can indicate that this information should not be sent to the certificate owner, but only be visible to ${appName} staff, by checking the checkbox below.?>
       <p>
       <label for="confidential"><?=_Don't send the message to the certificate owner?></label>
       <input type='checkbox' name='confidential' id='confidential'>
index 8cfaa2c498765e298a795331d62220306f702bd7..60d85be68404c6e5d341b445e9d90f5296f25d25 100644 (file)
@@ -53,7 +53,7 @@ public class Signup extends Form {
         vars.put("name", ni);
         vars.put("dob", myDoB);
         vars.put("email", HTMLEncoder.encodeHTML(email));
-        vars.put("helpOnNames", new SprintfCommand("Help on Names {0}in the knowledge base{1}", Arrays.asList("!(/kb/names", "!'</a>")));
+        vars.put("helpOnNames", new SprintfCommand("Help on Names in our {0}FAQ{1}", Arrays.asList("!(/kb/names", "!'</a>")));
         vars.put("csrf", getCSRFToken());
         vars.put("dobmin", User.MINIMUM_AGE + "");
         vars.put("countryCode", cs);
index 6d8c4f2ba38162e685fc637e2f221820309cf668..de3969b05c160b93f2616c2b560e5b47162e5b11 100644 (file)
@@ -10,7 +10,7 @@
     <td><?=$name?></td>
     <td><?=$e-mail?></td>
     <td><?=$master?></td>
-    <td><button class="btn btn-warning btn-confirm" data-confirm="<?=_Do you really want to delete the affiliation of ${e-mail} to this organisation??>" data-reply="<?=_Cancel?>,<?=_Delete?>" type="submit" name="del" value="<?=$e-mail?>">X</button> </td>
+    <td><button class="btn btn-warning btn-confirm" data-confirm="<?=_Do you really want to delete the affiliation of ${name} (${e-mail}) to this organisation??>" data-reply="<?=_Cancel?>,<?=_Delete?>" type="submit" name="del" value="<?=$e-mail?>">X</button> </td>
   </tr>
 <? } ?>
   <tr>
index 895e8c831f099de7758efd1f3b590e604291e2bc..dee4e90c8d47f18278e7cbfde4c219647fa2c173 100644 (file)
@@ -201,7 +201,7 @@ public class PingerDaemon extends Thread {
                         HashMap<String, Object> vars = new HashMap<>();
                         vars.put("valid", target.isVerified());
                         vars.put("domain", target.getSuffix());
-                        vars.put("domainLink", "https://" + ServerConstants.getHostNamePortSecure(Host.WWW) + "/" + EditDomain.PATH + target.getId());
+                        vars.put("domainLink", "https://" + ServerConstants.getHostNamePortSecure(Host.WWW) + EditDomain.PATH + target.getId());
                         vars.put("certs", new ArrayIterable<Certificate>(cs) {
 
                             @Override
index f2b6fe100f4b31fd73ed06318249a432fe4881b0..2c36b03b3e3ba2bcbe849c808a51c5ddef69ab26 100644 (file)
@@ -23,10 +23,13 @@ import club.wpia.gigi.dbObjects.Domain;
 import club.wpia.gigi.dbObjects.DomainPingConfiguration;
 import club.wpia.gigi.dbObjects.DomainPingExecution;
 import club.wpia.gigi.dbObjects.DomainPingType;
+import club.wpia.gigi.pages.account.domain.EditDomain;
 import club.wpia.gigi.ping.DomainPinger.PingState;
 import club.wpia.gigi.testUtils.PingTest;
 import club.wpia.gigi.testUtils.TestEmailReceiver.TestMail;
 import club.wpia.gigi.util.RandomToken;
+import club.wpia.gigi.util.ServerConstants;
+import club.wpia.gigi.util.ServerConstants.Host;
 import club.wpia.gigi.util.SimpleSigner;
 
 public class TestTiming extends PingTest {
@@ -109,6 +112,7 @@ public class TestTiming extends PingTest {
                 pd.executeNeededPings(new Date(System.currentTimeMillis() + 15 * 24 * 60 * 60L * 1000));
                 // The user receives another warning mail.
                 mail = getMailReceiver().receive(u.getEmail());
+                assertThat(mail.getMessage(), CoreMatchers.containsString("https://" + ServerConstants.getHostNamePortSecure(Host.WWW) + EditDomain.PATH + d.getId()));
                 assertThat(mail.getMessage(), CoreMatchers.containsString(d.getSuffix()));
                 assertThat(mail.getMessage(), CoreMatchers.containsString(c.getSerial()));
                 // And when the revocation is carried out