From: Christian Weiske <cweiske@cweiske.de>
Date: Wed, 11 May 2011 04:44:49 +0000 (+0200)
Subject: do not use links but POST form to change SSL settings. Proper web apps should never... 
X-Git-Tag: v0.98.0~58
X-Git-Url: https://gitweb.fluxo.info/?a=commitdiff_plain;h=9d91f1bee197f8f23547eb81080f2fff22cb49db;p=semanticscuttle.git

do not use links but POST form to change SSL settings. Proper web apps should never change data with GET requests
---

diff --git a/data/templates/editprofile-sslclientcerts.tpl.php b/data/templates/editprofile-sslclientcerts.tpl.php
index c43def4..246fb73 100644
--- a/data/templates/editprofile-sslclientcerts.tpl.php
+++ b/data/templates/editprofile-sslclientcerts.tpl.php
@@ -13,7 +13,13 @@
  <tbody>
  <?php foreach($sslClientCerts as $cert) { ?>
   <tr <?php if ($cert->isCurrent()) { echo 'class="ssl-current"'; } ?>>
-   <td><a href="#FIXME">delete</a></td>
+   <td>
+    <form method="post" action="#FIXME">
+     <button type="submit" name="action" value="deleteClientCert">
+      <?php echo T_('delete'); ?>
+     </button>
+    </form>
+   </td>
    <td><?php echo htmlspecialchars($cert->sslSerial); ?></td>
    <td><?php echo htmlspecialchars($cert->sslName); ?></td>
    <td><?php echo htmlspecialchars($cert->sslEmail); ?></td>
@@ -31,9 +37,11 @@
   <p><?php echo T_('Your current certificate is already registered with your account.'); ?></p>
  <?php } else { ?>
   <p>
-   <a href="#FIXME">
-    <?php echo T_('Register current certificate to automatically login.'); ?>
-   </a>
+   <form method="post" action="#FIXME">
+    <button type="submit" name="action" value="registerCurrentCert">
+     <?php echo T_('Register current certificate to automatically login.'); ?>
+    </button>
+   </form>
   </p>
  <?php } ?>
 <?php } else { ?>