* GET instead of POST for listsub. Also add a 'Back...' link.

[citadel.git] / webcit / listsub.c

diff --git a/webcit/listsub.c b/webcit/listsub.c
index 40b87a162e6a020e108519a9f4d3fc58b666f587..b94279530e5f568124e75decbd4f6b5282a4fbc3 100644 (file)
--- a/webcit/listsub.c
+++ b/webcit/listsub.c
@@ -81,7 +81,8 @@ void do_listsub(void)
                                "without your consent.<BR><BR>"
                                "Please click on the link which is being "
                                "e-mailed to you and your subscription will "
-                               "be confirmed.<BR></CENTER>\n"
+                               "be confirmed.<BR>\n"
+                               "<A HREF=\"/listsub\">Back...</A></CENTER>\n"
                        );
                }
                else {
@@ -117,7 +118,8 @@ void do_listsub(void)
                                "lists without your consent.<BR><BR>"
                                "Please click on the link which is being "
                                "e-mailed to you and your unsubscription will "
-                               "be confirmed.<BR></CENTER>\n"
+                               "be confirmed.<BR>\n"
+                               "<A HREF=\"/listsub\">Back...</A></CENTER>\n"
                        );
                }
                else {
@@ -141,7 +143,17 @@ void do_listsub(void)
                        wprintf("<CENTER><H1>Confirmation successful!</H1>");
                }
                else {
-                       wprintf("<CENTER><H1>Confirmation failed.</H1>");
+                       wprintf("<CENTER><H1>Confirmation failed.</H1>"
+                               "This could mean one of two things:<UL>\n"
+                               "<LI>You waited too long to confirm your "
+                               "subscribe/unsubscribe request (the "
+                               "confirmation link is only valid for three "
+                               "days)\n<LI>You have <i>already</i> "
+                               "successfully confirmed your "
+                               "subscribe/unsubscribe request and are "
+                               "attempting to do it again.</UL>\n"
+                               "The error returned by the server was: "
+                       );
                }
                wprintf("%s</CENTER><BR>\n", &buf[4]);
        }
@@ -150,7 +162,7 @@ void do_listsub(void)
         * Any other (invalid) command causes the form to be displayed
         */
        else {
-FORM:          wprintf("<FORM METHOD=\"POST\" ACTION=\"/listsub\">\n"
+FORM:          wprintf("<FORM METHOD=\"GET\" ACTION=\"/listsub\">\n"
                        "<TABLE BORDER=0>\n"
                );